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

StepRangeLen broken for AnchoredIntervals on Julia 1.8 #199

Open
BaileyBox opened this issue Sep 9, 2022 · 0 comments
Open

StepRangeLen broken for AnchoredIntervals on Julia 1.8 #199

BaileyBox opened this issue Sep 9, 2022 · 0 comments

Comments

@BaileyBox
Copy link

StepRange has added the ability to also use Length as a parameter in Julia1.8 (along with making stop and length as only constructors possible). This change somehow broke StepRangeLen with AnchoredIntervals.

An example showing how this is broken:

Julia1.6:

using Intervals

s = AnchoredInterval{-1}(1):1:AnchoredInterval{-1}(5)
r = s[1:5]

s isa StepRangeLen # false
s isa StepRange # true
r isa StepRangeLen # true
r isa StepRange # false

Julia1.8:

using Intervals

s = AnchoredInterval{-1}(1):1:AnchoredInterval{-1}(5)
r = s[1:5]

s isa StepRangeLen # false
s isa StepRange # true
r isa StepRangeLen # false
r isa StepRange # true

This failure can also be observed in the CI and Nightly pipelines. Current test failing is at anchoredinterval.jl:586.

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

1 participant