-
Notifications
You must be signed in to change notification settings - Fork 19
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: support logical concatenation at axis=0
#425
Conversation
3552a4d
to
312cc2b
Compare
for i, k in enumerate( | ||
k for collection in arrays for k in collection.__dask_keys__() | ||
) | ||
} |
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.
I did not know this level of list/dict comprehension existed 🤯
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.
I'm still absorbing the new "enforcement" stuff in upstream awkward, but the way the graphs are getting constructed makes sense to me so I think this is good to go
312cc2b
to
969427e
Compare
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #425 +/- ##
==========================================
- Coverage 93.22% 92.96% -0.27%
==========================================
Files 23 23
Lines 3187 3197 +10
==========================================
+ Hits 2971 2972 +1
- Misses 216 225 +9 ☔ View full report in Codecov by Sentry. |
Warning
Depends upon scikit-hep/awkward#2862, scikit-hep/awkward#2860
Fixes #305.