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

cms-2016-collision-datasets: adds full provenance #222

Merged
merged 5 commits into from
Mar 8, 2024

Conversation

katilp
Copy link
Member

@katilp katilp commented Feb 25, 2024

(closes #176 )

Adds full provenance for the 2016 collision datasets.

Output includes:

  • config files to be added to the corresponding records
  • json files of the records for 2016 datasets and their configs (I can do a separate PR to the portal repo)

image

@katilp
Copy link
Member Author

katilp commented Mar 4, 2024

Commit e7b0196 adds dataset semantics files.
Similarly to #214 and cernopendata/opendata.cern.ch#3588
the files from outputs/docs/NanoAOD are to be moved to /eos/opendata/cms/dataset-semantics/NanoAOD respecting the subdirectory structure <recid>

@tiborsimko tiborsimko force-pushed the cms-2016-update-collision-datasets branch from e7b0196 to c5b826f Compare March 7, 2024 13:55
@tiborsimko tiborsimko force-pushed the cms-2016-update-collision-datasets branch 2 times, most recently from 6895d19 to 0cf20eb Compare March 7, 2024 17:47
tiborsimko and others added 2 commits March 8, 2024 02:04
Fixes generation of record IDs for RECO configuration files that were
jumping into the space reserved for the collision data already. And we
have more RECO files due to adding full provenance chain, so they have
to jump the given record ID interval.

Adds NanoAOD data semantics documents from latest run.
@tiborsimko tiborsimko force-pushed the cms-2016-update-collision-datasets branch from a451015 to 118cc31 Compare March 8, 2024 01:04
Copy link
Member

@tiborsimko tiborsimko left a comment

Choose a reason for hiding this comment

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

Thanks, fixed RECO record IDs and the usage links, as discussed IRL.

@tiborsimko tiborsimko merged commit 118cc31 into master Mar 8, 2024
6 checks passed
@tiborsimko tiborsimko deleted the cms-2016-update-collision-datasets branch March 8, 2024 01:33
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.

CMS - create directory for 2016 collision record scripts
2 participants