Skip to content

feat: populate validated_runs field in records #3754

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

IssaAlBawwab
Copy link
Member

Adds a machine-readable validated_runs field parsed from the
abstract field. Supports consistency and structure across all datasets.

Closes #3746

@IssaAlBawwab
Copy link
Member Author

lint-commitlint error is giving me alot of trouble, i am following the rules yet its still failing

@katilp
Copy link
Member

katilp commented Jul 11, 2025

Is the message in https://github.com/cernopendata/opendata.cern.ch/actions/runs/16144318595/job/45559029553?pr=3754
of any help?

✖   body's lines must not be longer than 72 characters [body-max-line-length]
✖   subject may not be empty [subject-empty]
✖   type may not be empty [type-empty]

✖   found 3 problems, 0 warnings
ⓘ   Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint

@IssaAlBawwab IssaAlBawwab force-pushed the master branch 3 times, most recently from f64e78a to 31e1198 Compare July 11, 2025 09:35
This commit adds the new `validated_runs` field to all relevant data
records.

A data migration script was run to parse the existing `abstract`
field and create the new structured data. It correctly identifies
records that mention either "validated runs" or "validated lumi
sections," ensuring consistency across all datasets.

To maintain backward compatibility, the original `abstract` field and
its links have been preserved. This change only enriches the records
with the new, machine-readable field.

Closes cernopendata#3746
A data migration script was run to parse the existing `abstract` field
and create the new structured data. The script correctly identifies
records that mention either "validated runs" or "validated lumi
sections," ensuring consistency across all datasets.

To maintain backward compatibility, the original `abstract` field and
its links have been preserved. This change only enriches the records
with the new, machine-readable field.

Closes cernopendata#3746
A data migration script was run to parse the existing `abstract` field
and create the new structured data. It correctly identifies records
that mention either "validated runs" or "validated lumi sections,"
ensuring consistency across all datasets.

Closes cernopendata#3746
A data migration script was run to parse the existing `abstract`
field and create the new structured data. The script correctly
identifies records that mention either "validated runs" or
"validated lumi sections," ensuring consistency across all datasets.

To maintain backward compatibility, the original `abstract` field
and its links have been preserved. This change only enriches the
records with the new, machine-readable field.

Closes cernopendata#3746
zlmarshall and others added 4 commits July 17, 2025 14:59
This is 2025 February ATLAS Education and Outreach Open Data Release.
The release contains 25 datasets based on the recent proton-proton
Research Open Data release, but with several simplified formats for
various educational projects and skims that satisfy a variety of
use-cases.

Co-authored-by: Tibor Šimko <tibor.simko@cern.ch>
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.

RFC a new field to store information about validated runs
4 participants