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

Depend on IntervalSets.jl rather than Intervals.jl? #38

Closed
johnbcoughlin opened this issue Dec 29, 2021 · 1 comment
Closed

Depend on IntervalSets.jl rather than Intervals.jl? #38

johnbcoughlin opened this issue Dec 29, 2021 · 1 comment

Comments

@johnbcoughlin
Copy link

johnbcoughlin commented Dec 29, 2021

(This is an exact copy of a similar issue I just filed with Polynomials.jl)

There are two packages in the ecosystem for representing operations on non-iterable intervals, IntervalsSets.jl and Intervals.jl.

The latter currently depends on TimeZones.jl directly, which is a heavy transitive dependency for pure-math projects which want to use SpecialPolynomials. (I've opened an issue to address this). It is not as light-weight and includes functionality for plotting and printing intervals, as well as anchored intervals. Finally, it is not as widely used in the math ecosystem. See below for the list of each package's dependees.

For all these reasons, it would be nice to switch to using IntervalSets. Unfortunately, this will be a breaking change. What are the thoughts of the core maintainers on this, and would it be something that you'd consider addressing in a future breaking release?

julia> direct_dependents("Intervals")
10-element Vector{String}:
 "ConstraintDomains"
 "DateSelectors"
 "EconoSim"
 "Empirikos"
 "IndependentHypothesisWeighting"
 "LibPQ"
 "PatternFolds"
 "Polynomials"
 "SoapySDR"
 "SpecialPolynomials"

julia> direct_dependents("IntervalSets")
53-element Vector{String}:
 "AbstractPlotting"
 "ApproxFun"
 "ApproxFunBase"
 "ApproxFunFourier"
 "ApproxFunOrthogonalPolynomials"
 "ApproxFunSingularities"
 "AxisArrays"
 "AxisIndices"
 "AxisKeys"
 "BAT"
 "BasicBSpline"
 "CalculusWithJulia"
 "CamiXon"
 "ClassicalOrthogonalPolynomials"
 "ClimaCore"
 "CompactBases"
 "ComplexPhasePortrait"
 "ContinuumArrays"
 "CoulombIntegrals"
 "DirectionalStatistics"
 "DomainIntegrals"
 "DomainSets"
 "DungAnalyse"
 "DungBase"
 ⋮
 "InventoryManagement"
 "Khepri"
 "Makie"
 "MartaCT"
 "MeasureTheory"
 "OrthogonalPolynomialsQuasi"
 "PICDataStructures"
 "Poltergeist"
 "ProfileView"
 "RadiationDetectorSignals"
 "RadiationSpectra"
 "RayTracing"
 "ReinforcementLearningEnvironments"
 "ReinforcementLearningExperiments"
 "ReinforcementLearningZoo"
 "SampledSignals"
 "SolidStateDetectors"
 "StaticRanges"
 "StatsDiscretizations"
 "StatsMakie"
 "TimeAxes"
 "ValueShapes"
 "YAXArrays"
@jverzani
Copy link
Owner

#41 puts this in place. I just need to merge in JuliaMath/Polynomials.jl#381.

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