search

scikit-hep / pylhe

Lightweight Python interface to read Les Houches Event (LHE) files
https://pypi.org/project/pylhe/
Apache License 2.0
41 stars 23 forks source link

docs: add examples using parquet-awkward interaction #208

Closed tomeichlersmith closed 1 year ago

tomeichlersmith commented 1 year ago

This resolves #203 (I guess?) by adding a few more examples highlighting how awkward's interaction with parquet allows users to cache data originating from LHEs in an intermediate form either to speed up future reads or to merge many LHE files into one sample.

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: -0.46 :warning:

Comparison is base (caf0d3a) 88.18% compared to head (7769cda) 87.72%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #208 +/- ## ========================================== - Coverage 88.18% 87.72% -0.46% ========================================== Files 2 2 Lines 220 220 Branches 48 48 ========================================== - Hits 194 193 -1 Misses 23 23 - Partials 3 4 +1 ``` | Flag | Coverage Δ | | |---|---|---| | unittests-3.10 | `87.72% <ø> (ø)` | | | unittests-3.8 | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep#carryforward-flags-in-the-pull-request-comment) to find out more. [see 1 file with indirect coverage changes](https://app.codecov.io/gh/scikit-hep/pylhe/pull/208/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

eduardo-rodrigues commented 1 year ago

Hello. I think @matthewfeickert is travelling and may take a bit more time to respond. But this is not forgotten :-).

eduardo-rodrigues commented 1 year ago

Hello @matthewfeickert, I'm going to merge since I approved already this nice addition. We can always get a follow-up from you, I hope. Let's not delay this any longer.