-
Notifications
You must be signed in to change notification settings - Fork 283
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
Fix concatenation of cubes with aux factories #5340
Fix concatenation of cubes with aux factories #5340
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## v3.6.x #5340 +/- ##
=======================================
Coverage 89.32% 89.32%
=======================================
Files 89 89
Lines 22393 22398 +5
Branches 5375 5375
=======================================
+ Hits 20003 20008 +5
Misses 1640 1640
Partials 750 750
☔ View full report in Codecov by Sentry. |
3bedcd5
to
28f4792
Compare
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.
Thanks @schlunma, I ran the script in #5339 and I can confirm that this fixes the error.
Two things:
- The comment below about checking dimension spanning
- Once Fix #5347 #5349 has been merged (which will include the special patch What's New section): please could you include a What's New entry
28f4792
to
04a1e18
Compare
04a1e18
to
6da8979
Compare
🚀 Pull Request
Description
This PR makes sure to properly update all aux factory dependencies when concatenating cubes.
Closes #5339
Consult Iris pull request check list