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

multi-param case: added limits and fixed status plot #109

Merged
merged 8 commits into from
May 17, 2023

Conversation

sofia-calgaro
Copy link
Collaborator

@sofia-calgaro sofia-calgaro commented May 17, 2023

Limits, for plotting, were missing from the dictionary plot_info in plotting.py. Added to it as a dictionary, ie as "limits": { par1: [A1, B1], par2: [A2, B2]}. They were later included as lines in plots.

@sofia-calgaro
Copy link
Collaborator Author

sofia-calgaro commented May 17, 2023

Tested and found to work for:

  • "variation": trueand limits!=None
  • "variation": true and limits=None both for par1 and par2
  • "variation": true and limits=None for par1 and limits!=None for par2 (and vice versa)

The status plots, one for each parameter, are simply added to the pdf already containing the par1 vs par2 plots.

@sofia-calgaro sofia-calgaro merged commit 1070c2a into legend-exp:dev May 17, 2023
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