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

New mip treatment 2 #113

Closed
wants to merge 9 commits into from
Closed

Conversation

giamman
Copy link
Contributor

@giamman giamman commented Jul 15, 2013

Trivial fix: the previous pull request (#80) was not compiling because I had forgotten to commit one header.

@giamman giamman mentioned this pull request Jul 15, 2013
@cmsbuild
Copy link
Contributor

The following categories have been signed by agiamman (a.k.a. @giamman on GitHub): Fast Simulation

@cms-git-simulation, @cms-git-fastsim

@nclopezo
Copy link
Contributor

Hi,
I tested these changes and the unit tests and matrix tests passed. But I noticed that these changes should be done in top of CMSSW_7_0_X, not of 6_2_0_pre8. Sine it contains he commit 57ba31a

@giamman
Copy link
Contributor Author

giamman commented Jul 18, 2013

Hi,
the change was done on top of CMSSW_7_0_X, as far as I recall. If something was mixed with 6_2_0_pre8, that was not intentional.
By the way, can you please explain why there is a Tutorial/Test file in my pull request? It must have entered accidentally during the forward porting...

Andrea


Andrea Giammanco
Phone (CERN): +41 22 76 71567
Mobiles: +41 76 2323672 (CH), +32 497 135121 (BE), +39 349 5552471 (IT)


From: David Mendez [[email protected]]
Sent: 18 July 2013 16:05
To: cms-sw/cmssw
Cc: Andrea Giammanco
Subject: Re: [cmssw] New mip treatment 2 (#113)

Hi,
I tested these changes and the unit tests and matrix tests passed. But I noticed that these changes should be done in top of CMSSW_7_0_X, not of 6_2_0_pre8. Sine it contains he commit 57ba31ahttps://github.com/cms-sw/cmssw/commit/57ba31a


Reply to this email directly or view it on GitHubhttps://github.com//pull/113#issuecomment-21185465.

@nclopezo
Copy link
Contributor

Hi Andrea,

It should not have the commit 57ba31a, somehow it mixed on your branch. Anyway, could you create a new branch on top of 7_0_X and cherrypick your commits? I think it is the easiest way to go.

@giamman
Copy link
Contributor Author

giamman commented Jul 18, 2013

Ok I am doing that, and I will make a new pull request in a few minutes, but can you please have a look at this problem: in CMSSW_7_0_X_2013-07-18-0200, that I am using for testing and committing, all my usual workflows fail with this message that is not fastsim-related (and indeed there were absolutely no changes in fastsim in 7_0_X so far):

$ cmsDriver.py SingleMuPt10_cfi.py -s GEN --fast --pileup=NoPileUp --conditions auto:mc --beamspot Realistic8TeVCollision --eventcontent=FEVTDEBUGHLT --datatier GEN-SIM-DIGI-RECO -n 10
GEN,ENDJOB
We have determined that this is simulation (if not, rerun cmsDriver.py with --data)
Step: GEN Spec:
Loading generator fragment from Configuration.Generator.SingleMuPt10_cfi
Step: ENDJOB Spec:
globaltag = PRE_MC62_V8::All
Starting cmsRun SingleMuPt10_cfi_py_GEN.py
globaltag = PRE_MC62_V8::All
271 DQMStore::DQMStore
----- Begin Fatal Exception 18-Jul-2013 17:45:41 CEST-----------------------
An exception of category 'PluginNotFound' occurred while
[0] Constructing the EventProcessor
Exception Message:
Unable to find plugin 'XMLIdealGeometryESProducer' in category 'CMS EDM Framework ParameterSet Description'. Please check spelling of name.
----- End Fatal Exception -------------------------------------------------

(Here I am showing it for -s GEN, just to highlight the fact that it is not limited to fastsim.)

Am I doing something wrong? Are you aware of this?

@giamman giamman mentioned this pull request Jul 18, 2013
@giamman giamman closed this Jul 18, 2013
@ktf
Copy link
Contributor

ktf commented Jul 18, 2013

I confirm this is not FastSim related. Should be fixed by #135.

nclopezo added a commit to nclopezo/cmssw that referenced this pull request Nov 21, 2014
IB Pages: Ignore tags that contain 'DEBUG_TEST'
gpetruc pushed a commit to gpetruc/cmssw that referenced this pull request Nov 23, 2014
…CMSSW_7_0_6

Cmg mini aod lite v6 0 from cmssw 7 0 6
gdimperi pushed a commit to gdimperi/cmssw that referenced this pull request Jun 21, 2015
…DF-when-loading-snashot

Tested on Hgg bernstein cards. Closes cms-sw#30.
cfmcginn pushed a commit to cfmcginn/cmssw that referenced this pull request Mar 19, 2017
reverting the binning of the hiFJRhoProducer from 9 to 7 (remove 1, 1…
mariadalfonso pushed a commit to mariadalfonso/cmssw that referenced this pull request May 6, 2017
fwyzard added a commit to fwyzard/cmssw that referenced this pull request Aug 2, 2018
slava77 pushed a commit to slava77/cmssw that referenced this pull request Oct 9, 2021
Update benchmarking scripts and macros
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