Development #86006
documenter ou réécrire plus lisiblement un bout de code sur les stats de temps de résolution
0%
Description
return [res_time for row in results if (res_time := row[1].total_seconds()) > 0]
Révisions associées
sql: consider instant transitions when computing resolution times (#86006)
Historique
Mis à jour par Frédéric Péters il y a 3 mois
ça serait
return [row[1].total_seconds() for row in results if row[1].total_seconds() > 0]
mais je me pose la question du sens du > 0, (qu'en pratique on ne doit pas toucher, mais qu'on touche lors des tests), qui influencerait le calcul de la médiane (vu que ça va diminuer le nombre de points).
Bref, ne pourrait-on pas juste mettre
return [row[1].total_seconds() for row in results]
et en fait non, ce que ça cherche à éviter c'est les temps négatifs (vu via "# specify start status that is after end status").
Mais donc, quand même, je serais pour :
return [row[1].total_seconds() for row in results if row[1].total_seconds() >= 0]
Mis à jour par Robot Gitea il y a 3 mois
- Statut changé de Nouveau à En cours
- Assigné à changé de Valentin Deniaud à Frédéric Péters
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1060
- Titre : WIP: lisibilité code stats de temps de résolution (#86006)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1060/files
Mis à jour par Robot Gitea il y a 3 mois
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud (vdeniaud) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 3 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1060
- Titre : lisibilité code stats de temps de résolution (#86006)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1060/files
Mis à jour par Transition automatique il y a 3 mois
- Statut changé de Résolu (à déployer) à Solution déployée
misc: rewrite resolution time list comprehension for readability (#86006)