search

scikit-hep / pyhf

pure-Python HistFactory implementation with tensors and autodiff
https://pyhf.readthedocs.io/
Apache License 2.0
274 stars 81 forks source link

feat(backport): Use Python 3.12+ tarfile data extraction filter #2500

Closed meeseeksmachine closed 1 month ago

meeseeksmachine commented 1 month ago

Backport PR #2455: feat: Use Python 3.12+ tarfile data extraction filter

* Backport PR https://github.com/scikit-hep/pyhf/pull/2455
* In Python 3.12 extraction filters are added and will become default in
  Python 3.14. To start using them for when Python 3.12 support is added, and
  to guard against a Python 3.14 DeprecationWarning, use the data extraction
  filter for extracting tarfiles in pyhf.contrib.utils.download.
   - c.f. https://docs.python.org/3.12/library/tarfile.html#extraction-filters
codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (release/v0.7.x@71b87eb). Learn more about missing BASE report.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/v0.7.x #2500 +/- ## ================================================= Coverage ? 98.19% ================================================= Files ? 69 Lines ? 4547 Branches ? 804 ================================================= Hits ? 4465 Misses ? 49 Partials ? 33 ``` | [Flag](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | Coverage Δ | | |---|---|---| | [contrib](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `97.73% <33.33%> (?)` | | | [doctest](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `98.02% <33.33%> (?)` | | | [unittests-3.10](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `96.19% <33.33%> (?)` | | | [unittests-3.11](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `96.19% <33.33%> (?)` | | | [unittests-3.12](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `96.19% <33.33%> (?)` | | | [unittests-3.7](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `96.17% <33.33%> (?)` | | | [unittests-3.8](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `96.21% <33.33%> (?)` | | | [unittests-3.9](https://app.codecov.io/gh/scikit-hep/pyhf/pull/2500/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scikit-hep) | `96.23% <33.33%> (?)` | | 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.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.