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

Adding correct GTs for all early Run 3 data #61

Merged
merged 1 commit into from
Nov 1, 2024
Merged

Conversation

borzari
Copy link
Contributor

@borzari borzari commented Oct 30, 2024

This PR adds some new variables to manually set the correct global tags for all distinct cases of Run 3 data processing. The GTs are extracted from here for 2022 and here for 2023.

dataYear: valid for both, collected data and MC;
dataEra: is the regular era for data, e.g., E, F or G, and the campaign name for MC, e.g., preEE or postEE;
runWithData: switch to use collected data GTs or MC GTs.

The default values are set to run with the 2022F data GT.

Lines https://github.com/OSU-CMS/DisappTrks/blob/update_CMSSW_13/StandardAnalysis/python/protoConfig_cfg.py#L114-L121 were commented out, because even when adding an or runWithData in the data case, things inside OSUT3Analysis.DBTools.osusub_cfg would not be defined and the code doesn't run.

Checked the different cases and the script is picking up the correct GTs.

Please, while we don't find an automatic solution for GT setting with CRAB, remember to manually set the correct values for the new variables, even when running with osusub. The GT picking is manual for now.

Also commented out the secondary input files when running with CMSSW 12_4_X or 13_0_X.

@borzari borzari requested a review from carriganm95 October 30, 2024 17:20
@borzari borzari merged commit 9479f51 into update_CMSSW_13 Nov 1, 2024
@borzari borzari deleted the addGTs branch November 1, 2024 14:44
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