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

Reading ShapeFactors in xml workspace #1080

Closed
3 tasks done
alexander-held opened this issue Sep 25, 2020 · 0 comments · Fixed by #1136
Closed
3 tasks done

Reading ShapeFactors in xml workspace #1080

alexander-held opened this issue Sep 25, 2020 · 0 comments · Fixed by #1136
Labels
bug Something isn't working user request Request coming form a pyhf user

Comments

@alexander-held
Copy link
Member

Description

The .xml -> .json workspace conversion currently does not support ShapeFactor modifiers.

Expected Behavior

The ShapeFactor modifiers defined in a .xml should appear as shapefactor modifiers in the .json workspace.

Actual Behavior

A warning is printed, and the modifier skipped.

WARNING:pyhf.readxml:not considering modifier tag <Element 'ShapeFactor' at 0x7fc145eb0f90>

Steps to Reproduce

With pyhf version 0.5.2:

pyhf xml2json minimal_example/RooStats/minimal_example.xml

results in

WARNING:pyhf.readxml:not considering modifier tag <Element 'ShapeFactor' at 0x7fb8c5fff090>

using an example input workspace provided here: /afs/cern.ch/user/a/alheld/public/pyhf_shapefactor.

Checklist

  • Run git fetch to get the most up to date version of master
  • Searched through existing Issues to confirm this is not a duplicate issue
  • Filled out the Description, Expected Behavior, Actual Behavior, and Steps to Reproduce sections above or have edited/removed them in a way that fully describes the issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working user request Request coming form a pyhf user
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants