Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: set POI for limit calculations via dedicated pyhf API #364

Merged
merged 5 commits into from
Sep 10, 2022

Conversation

alexander-held
Copy link
Member

@alexander-held alexander-held commented Sep 5, 2022

#348 added the possibility to specify a custom POI via a kwarg in fit.limit. This was done by setting model.config.poi_name and model.config.poi_index directly, which is no longer possible after scikit-hep/pyhf#1972 (see scikit-hep/pyhf#1984). scikit-hep/pyhf#1985 adds support for None POI values in model.config.set_poi, which allows that API to be used instead. This PR switches to this dedicated API to support temporarily changing the POI, which requires an update of the minimum pyhf version to 0.7.0rc4.

* use pyhf model.config.set_poi API to support temporary POI changes in limit calculations

@codecov
Copy link

codecov bot commented Sep 5, 2022

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage 👍

Coverage data is based on head (c19b5f4) compared to base (a8ca551).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #364   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines         2018      2013    -5     
  Branches       330       330           
=========================================
- Hits          2018      2013    -5     
Impacted Files Coverage Δ
src/cabinetry/fit/__init__.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alexander-held alexander-held merged commit 0a92ee0 into master Sep 10, 2022
@alexander-held alexander-held deleted the fix/limit-set-poi branch September 10, 2022 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant