Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
first attempt to support awkward arrays #647
first attempt to support awkward arrays #647
Changes from 16 commits
5604eac
7dbe908
c0bbf5a
0281324
624a529
05c6c75
3d359de
9bf0cb9
974040c
13c4d59
74ae9e3
1d0e629
aeba549
88a5c83
15b3d1a
7e6beaa
77d5b6c
4aa3d26
3670d8b
72977c8
06032b2
1704aa7
ccc28c2
e524389
bd2f28d
a928198
fee56ee
0775e53
4b89a9b
9d56157
d14de3e
4d62e7e
c4c1b3f
339bce8
012de5e
7884598
e16ae35
0bced2f
588b6af
e687e19
41b1423
fa8a386
733937a
ed532a2
e5706c3
fceab1b
ef0637a
06608e9
3c46363
285e3b3
1dc93a6
7fa65dd
32c44cf
5e1c1da
08154f7
71f0471
2e66409
6cdcaa0
741af1c
c027669
7f7ebb6
771b2ab
4922603
c5c5335
8e7a725
c3ccf2f
a8e1648
c9a6417
d836999
ed95d8f
f7edc67
5a1d056
83effad
21a4b5f
4ff5851
2c59b19
988579e
504cae1
4dc0826
3ab5646
371f66e
d2eaf66
7b57167
b3678b6
d523c89
222998d
3fc9817
6a6657b
7ac4a0c
c016725
0340151
02365a6
c20cc31
8421ee6
2d024f1
ddefdcf
50a8dc3
fe27b74
2aed5b6
746ba7d
a589820
c26db5b
cd1a451
9b2ff61
7637fe3
52a804a
75e7526
d3d1d26
77e3953
536f729
cfe200e
d6d35bd
cf4ad03
46d553f
e8eeb54
5ab0708
5b39691
45a9958
94aa4ef
99853d5
cd2abdd
96bfe31
4a6d119
4243ccc
5ad915a
07246cc
fb137af
6e32637
3883bb0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
@ivirshup, I have no clue what's going on here:
as_view_awkarray
is correctly called when accessing e.g.adata.obsm["awk"]
when adata is a view, butak.Array
, and not an instance ofAwkwardArrayView
__setitem__
inherited from_ViewMixin
, and doesn't copy on modifcation.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.
Yeah, this is weird. I think it has to do with the
behavior
attribute. This is also discussed here:It looks like they don't do subclassing of arrays, but have a different system which is described here: https://awkward-array.readthedocs.io/en/latest/ak.behavior.html
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 tried that, but I'm unsure what the best way is to pass the
view_args
.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.
Asked: scikit-hep/awkward#1177 (reply in thread)
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.
@ivirshup, I implemented the approach with weakref, as discussed in scikit-hep/awkward#1177. The
test_view
(intest_awkward.py
) passes now. Please LMK what you think.