Conversation
aa1f11c to
ae90bb9
Compare
|
Great, thank you! |
tomeichlersmith
left a comment
There was a problem hiding this comment.
While I'm fine with including this, I do want to point out that the python ecosystem has much better environment handling than building frozen packages into container images.
I would suggest using uproot/awkward instead of PyROOT. The annoyance of switching to a different package will be outweighted by the time saved (uproot/awkward is faster) and the ease of use. Without using PyROOT, you can just
uv init
uv add scikit-hep
and you are good to go with the latest packages which you can mirror across machines by carrying around the uv.lock file that uv creates. https://docs.astral.sh/uv/
|
Yes it's long overdue to move the UCSB scripts to |
|
Now that's its merged, it will be built into This will not propagate into |
|
OK so I'd like it to be in the |
I am adding a new package to the container, here are the details.
Resolves #160
What new packages does this PR add to the development image?
shapCheck List
So it works