-
Notifications
You must be signed in to change notification settings - Fork 10
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
[WIP] Rewrite the whole thing based on a high-accuracy epoch type #43
Conversation
Codecov Report
@@ Coverage Diff @@
## master #43 +/- ##
=========================================
+ Coverage 87.38% 92.3% +4.92%
=========================================
Files 7 7
Lines 539 169 -370
=========================================
- Hits 471 156 -315
+ Misses 68 13 -55
Continue to review full report at Codecov.
|
test/epochs.jl
Outdated
ep1 = Epoch2{TDB}(ep, -Inf) | ||
@test ep1.epoch == typemin(Int64) | ||
@test ep1.offset == -Inf | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New line at the end of the file 🙂
src/Epochs2.jl
Outdated
|
||
Epoch2{S}(ep::Epoch2{S}, Δt) where {S} = Epoch2{S}(ep.epoch, ep.offset, Δt) | ||
|
||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing newline at the end of the file (or extra empty incomplete line) 🙂
@giordano Do you happen to know how one overloads the colon operator? |
It used to be lowered to the |
If timestamps are time-scale-agnostic than a conversion is just a relabeling, duh!
...and we have ranges 🎉 |
I got started with a new accuracy-preserving type. The design is based on the
AbsoluteDate
class from Orekit: https://github.com/CS-SI/Orekit/blob/develop/src/main/java/org/orekit/time/AbsoluteDate.javaEDIT: This is basically a straight port of Orekit's time package. Imitation is the sincerest form of flattery 😉
CC: @bgodard, @giordano, @prakharcode
ToDo