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

deprecate this package in favor of Intervals.jl #2

Open
jrevels opened this issue Dec 30, 2020 · 2 comments
Open

deprecate this package in favor of Intervals.jl #2

jrevels opened this issue Dec 30, 2020 · 2 comments

Comments

@jrevels
Copy link
Member

jrevels commented Dec 30, 2020

...as it's strictly more generic. ref https://invenia.github.io/Intervals.jl

Requires a pass to consolidate whatever extra functionality that may be here into there (esp. the notion of a minimal duck-typed interface), then a subsequent pass to update downstream TimeSpans dependents to use Intervals instead.

@haberdashPI
Copy link
Member

One note about this: I've been working on some functions to support set operations over TimeSpans (#11). These are much easier to implement if you assume an interval of the form [start, stop), rather than any arbitrary closed/open end points.

@haberdashPI
Copy link
Member

If/when this occurs it would be cool to be able to use Unitful values, mostly for the syntax benefits of writing 10s rather than Second(10), but also because I think it is generally easier to work with Unitful values compared Period values.

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