Closed matthewfeickert closed 2 years ago
I can fix this later (unless Alex beats me to it).
Thanks! #366 is needed first for CI to pass, I'll get that resolved and then we can get this in too.
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:
Coverage data is based on head (
4ec9480
) compared to base (278d088
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
There is also an unrelated new issue in the 3.7 CI when building the documentation (which should work in principle):
Theme error:
An error happened in rendering the page advanced.
Reason: ValueError('too many values to unpack (expected 3)')
I'll have a look at that.
edit: This is https://github.com/readthedocs/sphinx_rtd_theme/issues/1343, I'll fix that too in #366.
For completeness, the related API change for .par_names
is https://github.com/scikit-hep/pyhf/pull/2027.
Following the release of
pyhf
v0.7.0
, update to the latest API and update the lower bounds.