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

Add DELPHI datasets #3698

Merged
merged 1 commit into from
Dec 12, 2024
Merged

Add DELPHI datasets #3698

merged 1 commit into from
Dec 12, 2024

Conversation

dietrichliko
Copy link
Contributor

Added the DELPHI-datasets ...

@tiborsimko tiborsimko changed the base branch from qa to master November 11, 2024 12:36
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.

There is a "DontKnow" category for the following 5 records:

  • DELPHI simulation data xs_wandy3mvud_e188.6_m80.4_c98_1l_e1
  • DELPHI simulation data xs_wandy4mvud_e188.6_m80.4_c98_1l_e1
  • DELPHI simulation data xs_wandy3mvud24_e188.6_m80.4_c98_1l_e1
  • DELPHI simulation data xs_wandy3scud_e188.6_m80.4_c98_1l_e1
  • DELPHI simulation data xs_wandy2mvud_e188.6_m80.4_c98_1l_e1

Will you have an update for those records?

@tiborsimko tiborsimko self-assigned this Dec 3, 2024
@tiborsimko tiborsimko force-pushed the delphi-datasets branch 2 times, most recently from 50e721d to 2edf3fd Compare December 9, 2024 16:28
@tiborsimko
Copy link
Member

FYI I had to amend record IDs for two datasets:

$ sed -i 's|"84000"|"80998"|g' data/records/delphi-datasets-1999.json
$ sed -i 's|"84001"|"80999"|g' data/records/delphi-datasets-1999.json

because they were clashing with released-in-the-meantime TOTEM records.

@tiborsimko tiborsimko merged commit e8249be into master Dec 12, 2024
12 checks passed
@tiborsimko tiborsimko deleted the delphi-datasets branch December 12, 2024 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants