Skip to content

Conversation

alexander-held
Copy link
Member

Custom fix_pars arguments provided to fit.scan were accidentally modified when the POI gets set to constant. Copy the list to prevent changing this external data.

resolves #545

* avoid side effect where fit.scan accidentally modifies list provided via fix_pars
* updated pre-commit

Copy link

codecov bot commented Sep 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e017f4d) to head (47a1516).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #546   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines         2148      2150    +2     
  Branches       299       300    +1     
=========================================
+ Hits          2148      2150    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alexander-held
Copy link
Member Author

Test failures are currently due to scikit-hep/pyhf#2554.

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.

fit.scan modifies input parameter fix_pars
1 participant