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

misc: add years and months as units to display durations (#27036) #611

Merged
fpeters merged 1 commits from wip/27036-bigger-human-time into main 2023-08-17 10:32:30 +02:00
Conversation 2 Commits 1 Files Changed 2 +27
fpeters commented 2023-08-17 08:49:41 +02:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-08-17 08:49:42 +02:00
fpeters changed title from WIP: misc: add years and months as units to display durations (#27036) to misc: add years and months as units to display durations (#27036) 2023-08-17 09:10:56 +02:00
pmarillonnet approved these changes 2023-08-17 10:09:02 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok.

Rien à voir, et complètement du détail, mais dans le code existant, pas compris pourquoi on cherche la précision avec

_year = int(_day * 365.25)

mais on n’a pas l’équivalent

_month = int(_day * 30.44)

au lieu d’un arrondi à l’unité (qui n’est pas le bon d’ailleurs).
Ticket ?

Ok. Rien à voir, et complètement du détail, mais dans le code existant, pas compris pourquoi on cherche la précision avec ```python _year = int(_day * 365.25) ``` mais on n’a pas l’équivalent ```python _month = int(_day * 30.44) ``` au lieu d’un arrondi à l’unité (qui n’est pas le bon d’ailleurs). Ticket ?
fpeters commented 2023-08-17 10:12:50 +02:00
Author
Owner
Copy Link

Je dirais qu'il n'y a pas de raison particulière, mais que maintenant que c'est là ça serait moche de le changer (ça ferait que des affichages avec "une année et 6 heures" plutôt que "une année").

Je dirais qu'il n'y a pas de raison particulière, mais que maintenant que c'est là ça serait moche de le changer (ça ferait que des affichages avec "une année et 6 heures" plutôt que "une année").
pmarillonnet commented 2023-08-17 10:13:50 +02:00
Owner
Copy Link

Je dirais qu'il n'y a pas de raison particulière, mais que maintenant que c'est là ça serait moche de le changer (ça ferait que des affichages avec "une année et 6 heures" plutôt que "une année").

Ok, fair enough.

> Je dirais qu'il n'y a pas de raison particulière, mais que maintenant que c'est là ça serait moche de le changer (ça ferait que des affichages avec "une année et 6 heures" plutôt que "une année"). Ok, fair enough.
fpeters merged commit f99dc37685 into main 2023-08-17 10:32:30 +02:00
fpeters deleted branch wip/27036-bigger-human-time 2023-08-17 10:32:30 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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#611
No description provided.
Delete Branch "wip/27036-bigger-human-time"

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?