feat: ATLAS Physlite file for uproot#171
Conversation
Co-authored-by: Tatiana Ovsiannikova <TatianaOvsiannikova@users.noreply.github.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #171 +/- ##
=======================================
Coverage 72.38% 72.38%
=======================================
Files 3 3
Lines 134 134
=======================================
Hits 97 97
Misses 37 37 ☔ View full report in Codecov by Sentry. |
|
@ArturU043 thanks for this PR! Just a couple of things:
|
|
Hi @ariostas The current version is indeed V1.0.0.0. Some developers are working on fixing some issues with metadata collection with the root Python API, but the current version is practically production-ready. If major changes arise, we can add a new file. Concerning the first point, this Physlite format is stable for the current Atlas analysis Software releases. The format might be improved in the future with RNtuple compatibility in mind... I changed the file name as you requested. |
|
@all-contributors please add @ArturU043 for data |
|
I've put up a pull request to add @ArturU043! 🎉 |
Adding an ATLAS derivation file
PHYSLITEfor uproot CI tests.The file contains 100 events in the RNTuple format (EventData).
File built with
ROOT 6.34.04and validated for testing withuproot 5.5.1