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

New to old data converter #64

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Conversation

comane
Copy link
Member

@comane comane commented Apr 16, 2024

Adds scripts for the conversion of the new format to the old

@comane comane changed the title [WIP] New to old data converter New to old data converter Apr 27, 2024
if sys_name == 'stat':
continue

if uncertainty_definitions[sys_name]['treatment'] == "ADD":
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove if clause, since the systematics in the new format are all stored as additive

@FrancescoMerlotti
Copy link
Collaborator

Added a couple of scripts to make the new to old conversion quicker, it is not completely automatic because of the lack of consistency in the NNPDF file naming in the new format, e.g. data.yaml or data_<observable>.yaml or data_legacy_<obs>.yaml.

commondata_new_to_old must run in this PR.
test_commondata_new_to_old must run in nnpdf up to date environment.

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.

2 participants