Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update timescale for 'chelonia/time' to be milliseconds (same as Date.now()) #2593

Open
taoeffect opened this issue Feb 7, 2025 · 0 comments
Open

Update timescale for 'chelonia/time' to be milliseconds (same as Date.now()) #2593

taoeffect opened this issue Feb 7, 2025 · 0 comments
Assignees
@corrideat
Labels
App:Backend App:Frontend Kind:Core Anything that changes or affects the fundamental core data structures & design of the application. Kind:Enhancement Improvements, new features, performance upgrades, etc. Note:Discussion
Milestone
Final breaking ch...

Comments

@taoeffect
Copy link
Member

Problem

It seems weird to have to first divide by 1e3 and then multiply by 1e3 all over the place.

Solution

Why not just stick to Date.now()'s time scale for consistency?
@taoeffect taoeffect added App:Backend App:Frontend Kind:Core Anything that changes or affects the fundamental core data structures & design of the application. Kind:Enhancement Improvements, new features, performance upgrades, etc. labels Feb 7, 2025
@taoeffect taoeffect added this to the Final breaking changes milestone Feb 7, 2025
@taoeffect taoeffect mentioned this issue Feb 7, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@corrideat corrideat

Labels
App:Backend App:Frontend Kind:Core Anything that changes or affects the fundamental core data structures & design of the application. Kind:Enhancement Improvements, new features, performance upgrades, etc. Note:Discussion
Projects
None yet
Milestone
Final breaking changes
Development

No branches or pull requests
2 participants
@taoeffect @corrideat