Seeding layer cache #186
Merged
Seeding layer cache #186
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…dditions to HitExtractorSTRP
|
Hi, I took CMSSW_7_0_X_2013-07-25-1400, pulled these changes and ran the unit tests and RelVals, all the tests passed. |
|
The following categories have been signed by speer (a.k.a. @thspeer on GitHub): Reconstruction @cms-git-reconstruction |
|
No Changes seen in tests |
| @@ -221,7 +225,18 @@ void HitExtractorSTRP::cleanedOfClusters( const edm::Event& ev, HitExtractor::Hi | |||
| edm::Handle<SiStripMatchedRecHit2DCollection> matchedHits; | |||
| ev.getByLabel( theMatchedHits, matchedHits); | |||
| if (skipClusters) cleanFrom=result.size(); | |||
| range2SeedingHits( *matchedHits, result, accessor.stripTOBLayer(theIdLayer), sl, es); | |||
| if (minAbsZ>0.) { | |||
There was a problem hiding this comment.
uhm, I probably forgot to re-indent after un-commenting some parts. Thanks for spotting this, I will try to fix it when pushing a related development.
tahuang1991
added a commit
to tahuang1991/cmssw
that referenced
this pull request
Sep 15, 2014
…CH16 use correct geometry in calcuating gemdphi in ME11
tahuang1991
pushed a commit
to tahuang1991/cmssw
that referenced
this pull request
Sep 15, 2014
…-forSLHC14 Merge pull request cms-sw#186 from tahuang1991/CSC_Upgrade_Trigger_SLCH16
nclopezo
pushed a commit
to nclopezo/cmssw
that referenced
this pull request
Nov 21, 2014
fix package name so that script can be run from command-line
parbol
pushed a commit
to parbol/cmssw
that referenced
this pull request
Mar 5, 2015
Upstream updates from cms-sw#185
mariadalfonso
added a commit
to mariadalfonso/cmssw
that referenced
this pull request
Dec 18, 2017
Trained parametric model for 2L
cmsbuild
pushed a commit
that referenced
this pull request
Aug 9, 2018
Update tau MVA classifier decisions and raw values to be stored for 2016 94X NanoAOD
fwyzard
added a commit
to fwyzard/cmssw
that referenced
this pull request
Oct 6, 2018
bi-ran
added a commit
to bi-ran/cmssw
that referenced
this pull request
Mar 12, 2019
…-sw#186) * add various electron track variables trk pt error, track momentum extrapolations * add variables for electron energy regression turned off by default in foresting configs seed crystal eta/phi/ieta/iphi in the endcaps are filled instead of being zero-ed (used for regression) also moved effective area, conversion veto variables outside of electron id block
slava77
pushed a commit
to slava77/cmssw
that referenced
this pull request
Oct 9, 2021
Modified cmssw-ranking
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
a few changes related to seeding layers:
tracking results expected to be unchanged (functionalities to be used in the future by strip triplet seeding)