-
-
Notifications
You must be signed in to change notification settings - Fork 398
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
Bump compat of DimensionalData #3707
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3707 +/- ##
=======================================
Coverage 98.35% 98.35%
=======================================
Files 43 43
Lines 5709 5709
=======================================
Hits 5615 5615
Misses 94 94 ☔ View full report in Codecov by Sentry. |
I forgot to up the lower bound. Its fixed and the other version will be yanked. |
I don't really understand why this would help? It's failing on nightly, not on an older version like 1.6
There is no need to yank patch versions |
Your |
Thats nightly failing, for multiple reasons |
Sure, but I don't want to bump the compat because then JuMP will fail tests. It looks like you added the problematic rafaqz/DimensionalData.jl#585 only yesterday? We shouldn't touch |
To clarify the yank is for 1.8 and 1.9... they will also break for unrelated reasons (like effects system macros). I just forgot about them. I misunderstood your comment as nightly will fail anyway for other reasons even if I fix that particular bug. |
DD unavoidably hits a few internals where Base doesnt let us pass through dimension types. But that new one was probably a mistake, its just very tricky to get dispatch working. Im working around it now. (Also this version bump is huge, like 40 closed issues. So might be messy for a few days as dust settles.) |
It's okay if your tests fail because of printing, etc. But ideally, they shouldn't cause the basic tests in JuMP to fail: |
For sure. I will fix that asap. Hopefuly the other ones out if my control dont affect your tests. |
We can fix the printing. But rafaqz/DimensionalData.jl#664 is a blocker.