entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

lisibilité code stats de temps de résolution (#86006) #1060

Merged
fpeters merged 2 commits from wip/86006-resolution-time-code into main 2024-01-26 08:44:50 +01:00
Conversation 0 Commits 2 Files Changed 1 +2 -1
fpeters commented 2024-01-22 21:05:52 +01:00
Owner
Copy Link

cf https://dev.entrouvert.org/issues/86006?#note-1

cf https://dev.entrouvert.org/issues/86006?#note-1
fpeters changed title from WIP: lisibilité code stats de temps de résolution (#86006) to lisibilité code stats de temps de résolution (#86006) 2024-01-22 21:49:23 +01:00
vdeniaud approved these changes 2024-01-23 14:14:21 +01:00
wcs/sql.py Outdated
@ -2366,3 +2366,3 @@
results = cur.fetchall()
return [res_time for row in results if (res_time := row[1].total_seconds()) > 0]
return [row[1].total_seconds() for row in results if row[1].total_seconds() >= 0]
vdeniaud commented 2024-01-23 14:11:58 +01:00
Owner
Copy Link

Marrant, la récupération de row[1].total_seconds() dans une variable res_time c'était aussi affaire de lisibilité (car souvent row[index] on ne sait pas ce que c'est, ici ça va vu qu'il n'y a qu'une colonne référencée 5 lignes plus haut).

En tout cas un vrai gain de lisibilité pour l'ensemble de wcs ça serait de voir pour utiliser https://www.psycopg.org/docs/extras.html#namedtuple-cursor partout.

Marrant, la récupération de `row[1].total_seconds()` dans une variable `res_time` c'était aussi affaire de lisibilité (car souvent row[index] on ne sait pas ce que c'est, ici ça va vu qu'il n'y a qu'une colonne référencée 5 lignes plus haut). En tout cas un vrai gain de lisibilité pour l'ensemble de wcs ça serait de voir pour utiliser https://www.psycopg.org/docs/extras.html#namedtuple-cursor partout.
fpeters commented 2024-01-23 14:44:35 +01:00
Author
Owner
Copy Link

Marrant, la récupération de row[1].total_seconds() dans une variable res_time c'était aussi affaire de lisibilité (car souvent row[index] on ne sait pas ce que c'est, ici ça va vu qu'il n'y a qu'une colonne référencée 5 lignes plus haut).

Je complète avec un commentaire pour dire "row[1] will have the resolution time as computed by postgresql", ça évite en effet de devoir remonter lire la requête.

En tout cas un vrai gain de lisibilité pour l'ensemble de wcs ça serait de voir pour utiliser https://www.psycopg.org/docs/extras.html#namedtuple-cursor partout.

Ça me semble envisageable, surtout que ça ne demande pas à être fait de manière intégrale.

> Marrant, la récupération de row[1].total_seconds() dans une variable res_time c'était aussi affaire de lisibilité (car souvent row[index] on ne sait pas ce que c'est, ici ça va vu qu'il n'y a qu'une colonne référencée 5 lignes plus haut). Je complète avec un commentaire pour dire "row[1] will have the resolution time as computed by postgresql", ça évite en effet de devoir remonter lire la requête. > En tout cas un vrai gain de lisibilité pour l'ensemble de wcs ça serait de voir pour utiliser https://www.psycopg.org/docs/extras.html#namedtuple-cursor partout. Ça me semble envisageable, surtout que ça ne demande pas à être fait de manière intégrale.
fpeters force-pushed wip/86006-resolution-time-code from 8d187ca5bf to 9ed5f6240a 2024-01-23 14:42:01 +01:00 Compare
fpeters merged commit bf180b0398 into main 2024-01-26 08:44:50 +01:00
fpeters deleted branch wip/86006-resolution-time-code 2024-01-26 08:44:51 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1060
No description provided.
Delete Branch "wip/86006-resolution-time-code"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?