Updates for condor on CMS Connect in singularity #3845
Conversation
| @@ -581,6 +585,10 @@ make_gridpack () { | |||
| echo "cleaning temporary gridpack" | |||
| rm $WORKDIR/pilotrun_gridpack.tar.gz | |||
|
|
|||
| # awightma start: Force the re-weight step to only use 1 core | |||
| echo "nb_core = 1" >> $WORKDIR/process/madevent/Cards/me5_configuration.txt | |||
There was a problem hiding this comment.
this does not affect phase space integration?
There was a problem hiding this comment.
No, this is the step after when EFT reweighting is done. I can remove this too since it's not for general use.
There was a problem hiding this comment.
This is due to a bug in madgraph that doesn't work when nb_core is not 1? If so, it might be worth keeping it
| @@ -694,6 +703,8 @@ else | |||
| cmssw_version=CMSSW_10_2_24_patch1 | |||
| elif [[ $SYSTEM_RELEASE == *"release 7"* ]]; then | |||
| cmssw_version=CMSSW_10_6_19 | |||
| elif [[ $SYSTEM_RELEASE == *"release 9"* ]]; then | |||
| cmssw_version=CMSSW_13_2_9 | |||
There was a problem hiding this comment.
el9 gridpacks won't work for official production yet (that's why we just didn't care about el9 so much in the script), or is the container-in-container prepared? @DickyChant
There was a problem hiding this comment.
I think we should officially disable el9 instead of making it work like this (el9 gridpacks are not going to be usable for official production) , talked to sitian
| if [ -e "$input_files" ]; then rm "$input_files"; fi | ||
| tar -zchf "$input_files" "$card_dir" "$patches_directory" "$utilities_dir" "${plugin_directory}" | ||
| tar -zchf "$input_files" "$card_dir" "$patches_directory" "$utilities_dir" "${plugin_directory}" "${addons_dir}" |
There was a problem hiding this comment.
this is not safe since not all people are going to be using a model that lives under addons directory and tar command will eventually crash if it does not exist. why not just steer it with extramodels.dat ?
this will be wgetted from link
There was a problem hiding this comment.
Thanks for explaining, I've removed this. Our current model is not in https://cms-project-generators.web.cern.ch/cms-project-generators/. Can we add it or can only people in the gen group make changes?
There was a problem hiding this comment.
Should ask GEN conveners
This PR fixes condor on CMS Connect. The
cmssw-cc7-condor-python27singularity container is required to run, and must be activated from the topgenproductionsdirectory.