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

Don't register default values of untracked parameter sets #248

Merged

Conversation

wmtan
Copy link
Contributor

@wmtan wmtan commented Aug 7, 2013

The function ParameterSet::getUntrackedParameterSet registers the passed in default value argument. This is not thread safe, and is totally unnecesary, This pull request removes the registration. This is a step toward the multithreaded framework.

@nclopezo
Copy link
Contributor

nclopezo commented Aug 7, 2013

Hi,

I took CMSSW_7_0_X_2013-08-07-0200, pulled these changes, and ran the RelVals and unit tests, all tests passed.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2013

The following categories have been signed by chrjones (a.k.a. @Dr15Jones on GitHub): Core

@cms-git-visualization, @cms-git-core, @cms-git-geometry

ktf added a commit that referenced this pull request Aug 7, 2013
…eters

Don't register default values of untracked parameter sets
@ktf ktf merged commit ce1cb90 into cms-sw:CMSSW_7_0_X Aug 7, 2013
@wmtan wmtan deleted the doNotRegisterDefaultUntrackedParameters branch August 8, 2013 04:07
jshlee referenced this pull request in gem-sw/cmssw Aug 7, 2014
parbol pushed a commit to parbol/cmssw that referenced this pull request Mar 5, 2015
makortel pushed a commit to makortel/cmssw that referenced this pull request Apr 22, 2015
IB Pages: show when the IB is a full build or a patch IB
cmsbuild pushed a commit that referenced this pull request Dec 11, 2023
* Update BuildFile.xml to add conifer 

* Delete L1Trigger/TrackTrigger/interface/conifer.h
zhenbinwu pushed a commit to zhenbinwu/cmssw that referenced this pull request Feb 14, 2024
* Update BuildFile.xml to add conifer 

* Delete L1Trigger/TrackTrigger/interface/conifer.h
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.

4 participants