Skip to content

add parking vbf and parking hh to 2024 data#114

Open
nprouvost wants to merge 3 commits intomasterfrom
add_parking_vbf_parking_hh_2024
Open

add parking vbf and parking hh to 2024 data#114
nprouvost wants to merge 3 commits intomasterfrom
add_parking_vbf_parking_hh_2024

Conversation

@nprouvost
Copy link
Contributor

No description provided.

@nprouvost nprouvost requested a review from riga February 26, 2026 15:14
Copy link
Member

@riga riga left a comment

Choose a reason for hiding this comment

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

Hey, this looks good 👍

Just three small questions out of interest and scrutiny...

n_files=168 + 197 + 180 + 115 + 150 + 173 + 198 + 237,
n_events=45_446_278 + 45_471_089 + 45_475_567 + 45_489_416 + 45_438_075 + 45_468_127 + 45_487_079 + 45_469_363,
aux={
"prompt": True,
Copy link
Member

Choose a reason for hiding this comment

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

Did you check if all these datasets are really prompt?
(Ok, I guess by definition of parking streams, they should be, but oh well...)

Comment on lines +765 to +780
"/ParkingVBF0/Run2024I-MINIv6NANOv15-v2/NANOAOD",
"/ParkingVBF1/Run2024I-MINIv6NANOv15-v3/NANOAOD",
"/ParkingVBF2/Run2024I-MINIv6NANOv15-v3/NANOAOD",
"/ParkingVBF3/Run2024I-MINIv6NANOv15-v3/NANOAOD",
"/ParkingVBF4/Run2024I-MINIv6NANOv15-v3/NANOAOD",
"/ParkingVBF5/Run2024I-MINIv6NANOv15-v2/NANOAOD",
"/ParkingVBF6/Run2024I-MINIv6NANOv15-v2/NANOAOD",
"/ParkingVBF7/Run2024I-MINIv6NANOv15-v3/NANOAOD",
"/ParkingVBF0/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
"/ParkingVBF1/Run2024I-MINIv6NANOv15_v2-v3/NANOAOD",
"/ParkingVBF2/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
"/ParkingVBF3/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
"/ParkingVBF4/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
"/ParkingVBF5/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
"/ParkingVBF6/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
"/ParkingVBF7/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
Copy link
Member

Choose a reason for hiding this comment

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

Are you sure that both versions per stream (e.g. v2 and v2-v2) are to be used simultaneously?

Comment on lines +894 to +895
"/ParkingHH/Run2024I-MINIv6NANOv15-v3/NANOAOD",
"/ParkingHH/Run2024I-MINIv6NANOv15_v2-v2/NANOAOD",
Copy link
Member

Choose a reason for hiding this comment

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

Same as above: really both of them?

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