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.

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

Use negative timespans for future dates / update to 1.1's datetime methods #157

Closed
NathanKell opened this issue Apr 5, 2016 · 3 comments

Comments

@NathanKell
Copy link

You may already do this, in which case disregard, but if not I would ask you to use a negative UT timespan whenever you are getting the time difference between the current time and a future date. This is because the time formatter in RSS will take leap years into account, and this means it must distinguish whether a timespan is going forwards or backwards from the current UT.

Also, the existing date/time handling should IMO be removed, instead letting any mod changing the date-time-printer interface in KSP handle timespan printing. But let's be clear, it's served well for a long time, for which the whole RSS/RO community is very grateful. :)

Instead, I would ask that KAC support changing between delta display, raw seconds display, and absolute date display (PrintDate() ).

@NathanKell
Copy link
Author

Bumping this. Give me a holler when you're free, would love to touch base on this (and possibly do some/most of the work for you, since that seems only fair).

@TriggerAu
Copy link
Owner

@NathanKell ready to give this a crack if you have some time over the weekend... just let me know

@TriggerAu
Copy link
Owner

Closing this one with the Kronometer changes in #211

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants