-
Notifications
You must be signed in to change notification settings - Fork 9
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 Madgraph for HINPbPbSpring23 central campaign #151
base: master
Are you sure you want to change the base?
Conversation
So HIN L2s are happy (at least for MG5 ones). On the technical side, we need special CMSSW version ( |
@stahlleiton hi thanks for doing all the templates. I believe you copy pasted them from existing ones in this repo? What are the processes you need? @DickyChant for the gridpack production i think we need a. I think we should ask GEN, b. I can take care |
for a. does it mean that the script won't accept those options? |
@DickyChant for now we choose a branch in genproductions so it directly takes the default cmssw from there. We can technically pass down which cmssw we want to use but this needs geovanny to make a clickable option in our web interface. |
I started from Run3Summer23 campaign and changed the beam energy and changed the templates to use nuclear PDFs. |
If possible, I think it would be better if the CMSSW is not taken by default from the genproductions (since creating a new branch just to change the default CMSSW release would immediately make that branch go out of sync with the master branch) |
I feel unfortunately the chance for the development is tiny however I just came up with two ugly solutions
|
Can't we simply add an extra field in the json file "Campaigns/HINPbPbSpring23/HINPbPbSpring23.json" called "CMSSW" and use that instead (e.g. "CMSSW": "CMSSW_13_0_18_HeavyIon")?. So if present, use that otherwise, use the default from genproduction. |
This should require development too, anyhow we can request a feature here: https://github.com/cms-PdmV/GridpackMachine |
I would imagine this is a two line change actually? If we have that in the json fields we append it, otherwise no, for this line https://github.com/cms-PdmV/GridpackMachine/blob/7e34f08a9fd2aa94e190a5b33c7d613b342230ee/src/gridpack.py#L500 |
The process are:
|
Any news on this? |
Thanks for reminder, in two weeks there will be a preMccM meeting and we will discuss there. In my point of view we should have them if common background team doesn’t feel a huge burden 1: normally heavy ion campaigns are always spring which means it will get used at a different time other than pp, so we could benefit from adding HIN requests to keep the cores busy I also could understand if there is going to be blame on HIN not so common… Speaking of being common, if there is a low PU run extended from a pp ref, then the gridpacks here would be a good fit for those runs as well. For which @stahlleiton do we have some preliminary plan on that? Would be a nice point for supporting so. |
hey sorry for disappearing from this, somehow based on my private chats with sitian i misunderstood as if HIN wanted to separately do things themselves. my bad. once we have rereco + 24 campaign set up, we can converge on having central production for HIN as well |
This PR adds the HINPbPbSpring23 templates for MadGraph central production.
@sihyunjeon