-
Notifications
You must be signed in to change notification settings - Fork 39
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
CMS example with ServiceX 3 client #225
Conversation
I've pushed the fixes for the func_adl version of the CMS notebook, with fixes courtesy of @gordonwatts (this needs func_adl >= 3.3.3). I would like to update the ATLAS example as well, but it seems that at the moment there are problems with the UNL storage which prevent me from accessing those files. |
I have updated the ATLAS examples. |
* link additional AGC-related projects * update website * exclude notebooks from ruff CI * formatting
The issue with the CI complaining about unrelated files should be fixable by rebasing to incorporate #226. |
In the CMS ttbar notebook after the ServiceX delivery cell I see
What is this about? |
Other than the small things about imports above, could we please keep ServiceX off by default with One more very picky request: could we please move |
This is a particular issue related to how |
…d-challenge into ponyisi-servicex-3
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.
This looks all good to me, thank you very much! I believe @oshadura wanted to ensure we can run this on the Open Data coffea-casa before merging but I'll let her comment.
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.
It works on https://coffea-opendata.casa with new ServiceX! thanks for the patience! :)
This works with uproot-raw right now with
servicex==3.0.0b1
. The func_adl version does not yet work (@gordonwatts is working on this), and in the end we would want a version with the final 3.0.0 release.