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

bug fix: configured run range dependence was ignored in TkModuleGroupSel... #348

Conversation

joergbehr
Copy link
Contributor

...ector. everything was associated to the first IOV.

…Selector. everything was associated to the first IOV.
@joergbehr
Copy link
Contributor Author

Same as #134

@ktf
Copy link
Contributor

ktf commented Aug 15, 2013

@nclopezo can you check?
@demattia can you sign off?

Thanks.

@nclopezo
Copy link
Contributor

Hi,

I took these changes on top of CMSSW_7_0_X_2013-08-14-1400, I ran the unit tests and RelVals, all tests passed.

You can see the logs here:

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/114/console

@cmsbuild
Copy link
Contributor

The following categories have been signed by @demattia: Calibration and Alignment

@cms-git-alca

ktf added a commit that referenced this pull request Aug 19, 2013
…duleGroupSelector_70X

bug fix: configured run range dependence was ignored in TkModuleGroupSel...
@ktf ktf merged commit 1b6c953 into cms-sw:CMSSW_7_0_X Aug 19, 2013
nclopezo added a commit that referenced this pull request Jan 21, 2014
migrate to pythia8 180 plus some cleaning
makortel pushed a commit to makortel/cmssw that referenced this pull request Apr 22, 2015
use df -k which works both on osx and slc
cvernier pushed a commit to cvernier/cmssw that referenced this pull request Feb 5, 2016
cmsbuild pushed a commit that referenced this pull request May 12, 2019
slava77 pushed a commit to slava77/cmssw that referenced this pull request Oct 9, 2021
add compatibility requirement for propagations to strip hits
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.

None yet

5 participants