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

Create a nnpdf data cli and package #2037

Open
scarlehoff opened this issue Apr 6, 2024 · 2 comments
Open

Create a nnpdf data cli and package #2037

scarlehoff opened this issue Apr 6, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@scarlehoff
Copy link
Member

scarlehoff commented Apr 6, 2024

This is just a collection of scripts / ideas that I think they might be useful. I've been writing some scripts already while preparing the talk for DIS so might as well make it into a cli for the data package.

  • Given a dataset name, return the hepdata / arxiv entry
  • Given an arxiv/hepdata entry, return matching datasets.
  • Given a runcard / yaml file with dataset_inputs, return a list of all matching hep data / arxiv entry. Perhaps even prepare a nice latex format table and, if inspire/arxiv info is available, through the API, the bib file with the corresponding references.
  • Some filtering (e.g., all datasets that match CMS_1JET_13TEV and have a theory)
  • Provide a way to read and/or search in all theories
@scarlehoff scarlehoff added the enhancement New feature or request label Apr 6, 2024
@scarlehoff scarlehoff changed the title Create a nnpdf data cli Create a nnpdf data cli and package Oct 22, 2024
@scarlehoff
Copy link
Member Author

In order to separate the data package from the rest of the code (while keeping the reading features), the only necessary step is to move this function (and its dependencies) to the data:

def parse_new_metadata(metadata_file, observable_name, variant=None):

The only piece that keeps a dependence on validphys are the old plotting options (plus perhaps a few constants that are defined in other files). Once no more old commondata files need to be read, those will be easily removed.

In the meantime, if someone wanta to move things around anyway they can do so, but some kind of "fake plotting options"* will be needed in the middle.

*plotting options are, obviously, more than plotting options and have an effect also in e.g. the kinematics

@felixhekhorn
Copy link
Contributor

see #2225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants