Lukas Koch

Results 32 comments of Lukas Koch

I tried the method you suggested, but I get an error: ``` --------------------------------------------------------------------------- DeserializationError Traceback (most recent call last) ~/.local/lib/python3.7/site-packages/uproot/reading.py in get(self) 2484 try: -> 2485 out = cls.read(chunk, cursor,...

Would adding some tests for TMatrixT, TMatrixTSparse, and TMatrixTSym be enough for that? Maybe both with ROOT 5 and ROOT 6 files? Or is the concern more about the principle...

I just tried to load the matrix with the PR branch of uproot 4, and it seems to work as expected. Though now I have the problem that the resulting...

I guess nothing has happened so the issue is still relevant?

Hi! Thanks a lot for this PR. Could you add a test of this feature to `.github/workflows/pythontests.yml`? To make sure it keeps working in the future.

If you look at the output of the tests (e.g. by clicking on the green checkmark next to the latest commit). you will see that it did not actually run...

Could you also remove the newlines from the option help text, so it behaves like the others, and also make the docstring of your test function follow the convention of...

As you can see, the tests fail now: https://github.com/scikit-hep/histoprint/actions/runs/7023198492/job/19109160857?pr=124 Aside from fixing that, you also need to add the added option to the CHANGELOG.md. Just put it as a bullet...

Pandas is only needed optionally for looking at CSV files with the CLI. We could add it as an optional extra requirement, but those are not used very often anyway....

Hi, let me just make sure I understand the request. You would like to do to something like this: ``` $ histoprint /path/to/rootfile:path/to/tree/and/branch:path/to/tree/and/second_branch ``` Which would be equivalent to: ```...