-
Notifications
You must be signed in to change notification settings - Fork 85
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
Slicing ak.Array in cuda backend breaks #3133
Comments
That's an interesting error. How did you install |
This is expected to break because it would use a not-yet-existing kernel, If you've ever installed Awkward with the developer instructions, remove it pip uninstall awkward awkward-cpp get a new clone of Awkward's git repo, and reinstall, starting from the After that (I just did it), you should see
because the |
|
Yes @agoose77, I pip-installed it a couple of weeks ago from the commit @jpivarski: when is 2.6.5 released? Will the PR from @ManasviGoyal fixing #3130 be included in that release? |
So in principle, doing git checkout ManasviGoyal/improve-variable-length-loop-kernels before nox -s prepare
python -m pip install -v ./awkward-cpp
python -m pip install -e . (in a freshly cloned directory, so that all of the headers made with
Awkward 2.6.5 was released yesterday, and it includes #3115. When @ManasviGoyal is done with #3130, I'll review it and merge it if there are no issues, and if it's useful to put that in a release, I'll do it. #3130 will need a new awkward-cpp, which takes more time and PyPI quota than a regular release, so they tend to be spaced out more, but if you need it, I'll make a release. |
@jpivarski: many thanks for the instructions 👍 Being able to slice large data in CUDA (following numpy syntax) is a very important operation, in my opinion. It would be cool if you release it as soon as it is ready! |
Version of Awkward Array
2.6.4
Description and code to reproduce
Hey guys,
While playing with simple slices of ak.Arrays, I noticed the following:
So, in the
cpu
backend, this works as expected. Nowcuda
's turnAm I missing something here?
The text was updated successfully, but these errors were encountered: