Skip to content

Improve hardcoded thickness values in HGCAL TPG #6

Draft
jbsauvan wants to merge 15 commits into
waredjeb:fix_V19_HLT_RECO_PRfrom
jbsauvan:improve-hardcoded-v19
Draft

Improve hardcoded thickness values in HGCAL TPG #6
jbsauvan wants to merge 15 commits into
waredjeb:fix_V19_HLT_RECO_PRfrom
jbsauvan:improve-hardcoded-v19

Conversation

@jbsauvan

@jbsauvan jbsauvan commented Jul 4, 2025

Copy link
Copy Markdown

Improve hardcoded thickness values in HGCAL TPG:

  • Using V19 procModifier in CTC/STC configs
  • Remove hardcoded thickness constants in C++, possibly retrieving the number of thicknesses from the geometry

@jbsauvan jbsauvan force-pushed the improve-hardcoded-v19 branch from 341fdf9 to 3dc43f9 Compare July 23, 2025 11:59
@waredjeb waredjeb force-pushed the fix_V19_HLT_RECO_PR branch 3 times, most recently from a4b8a8b to 1eae11c Compare July 30, 2025 16:48
waredjeb pushed a commit that referenced this pull request Sep 29, 2025
* first try distributing process Modifiers of Cleaning step

* disable finale state radiation for mu->mu embedding

* update center of mass energy for run3

* implement correct ModifierChain for tau embedding process modifiers

* revert some changes

* set all process modifier for tau embeddign cleaning step

* Introduce special cleaning and simulation process modifier

* add process modifiers for sim gen step

* Set process modifiers for reco sim step

* add process modifier for tau embedding merging step
Those "toReplaceWith" modifiers need to be applied as late as possible e.g. in StandardSequences.Reconstruction_Data_cff.
This is to avoid collisions with other modules that are cloned and modified versions from the original modules or "toModify" modifier calls that try to modify the original modules. They throw an exeption if they want to do this on the replaced merger modules.

* add autoNANO option for the tau embedding nanoaod files

* adjust test cmsDriver commands to work with the new process modifiers

---------

Co-authored-by: Christian Winter <c.winter@kit.edu>
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.

3 participants