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

Consider using nanoseconds for internal time measurements #33

Open
geoffxy opened this issue Jan 19, 2020 · 0 comments
Open

Consider using nanoseconds for internal time measurements #33

geoffxy opened this issue Jan 19, 2020 · 0 comments
Labels
plugin This is primarily related to the Atom plugin server This is primarily related to the server tech debt This should be improved

Comments

@geoffxy
Copy link
Member

geoffxy commented Jan 19, 2020

We currently use milliseconds for time measurements and store the measurements in floating point values. However, when we perform arithmetic on these values, they are susceptible to numerical error.

Our time measurements are precise to the nanosecond. So we should consider using nanoseconds instead, and storing the measurements in integers.
@geoffxy geoffxy added tech debt This should be improved plugin This is primarily related to the Atom plugin server This is primarily related to the server labels Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
plugin This is primarily related to the Atom plugin server This is primarily related to the server tech debt This should be improved
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@geoffxy