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

hlt-tracking parabolic mf build fit #2244

Closed

Conversation

tropiano
Copy link
Contributor

added on top of Giuseppe's commit the possibility to switch to magnetic field also in the fitting stage.

@tropiano
Copy link
Contributor Author

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @tropiano for CMSSW_7_1_X.

hlt-tracking parabolic mf build fit

It involves the following packages:

Configuration/StandardSequences
MagneticField/ParametrizedEngine
RecoTracker/CkfPattern
RecoTracker/IterativeTracking
RecoTracker/TkNavigation
RecoTracker/TkSeedGenerator
RecoTracker/TrackProducer
TrackingTools/GeomPropagators
TrackingTools/KalmanUpdators
TrackingTools/MaterialEffects
TrackingTools/Producers

@nclopezo, @vlimant, @franzoni, @cerminar, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano, @fabiocos can you please review it and eventually sign? Thanks.
@ghellwig, @cerati, @gpetruc, @GiacomoSguazzoni, @rovere this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@vlimant
Copy link
Contributor

vlimant commented Jan 30, 2014

+1

Le 30/01/2014 18:18, cmsbuild a écrit :

A new Pull Request was created by @tropiano
https://github.com/tropiano for CMSSW_7_1_X.

hlt-tracking parabolic mf build fit

It involves the following packages:

Configuration/StandardSequences
MagneticField/ParametrizedEngine
RecoTracker/CkfPattern
RecoTracker/IterativeTracking
RecoTracker/TkNavigation
RecoTracker/TkSeedGenerator
RecoTracker/TrackProducer
TrackingTools/GeomPropagators
TrackingTools/KalmanUpdators
TrackingTools/MaterialEffects
TrackingTools/Producers

@nclopezo https://github.com/nclopezo, @vlimant
https://github.com/vlimant, @franzoni https://github.com/franzoni,
@cerminar https://github.com/cerminar, @cmsbuild
https://github.com/cmsbuild, @anton-a https://github.com/anton-a,
@thspeer https://github.com/thspeer, @slava77
https://github.com/slava77, @Degano https://github.com/degano,
@fabiocos https://github.com/fabiocos can you please review it and
eventually sign? Thanks.
@ghellwig https://github.com/ghellwig, @cerati
https://github.com/cerati, @gpetruc https://github.com/gpetruc,
@GiacomoSguazzoni https://github.com/GiacomoSguazzoni, @rovere
https://github.com/rovere this is something you requested to watch as
well.
You can sign-off by replying to this message having '+1' in the first
line of your reply.
You can reject by replying to this message having '-1' in the first line
of your reply.
@ktf https://github.com/ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of
your comment.


Reply to this email directly or view it on GitHub
#2244 (comment).

@cmsbuild
Copy link
Contributor

-1
When I ran the RelVals I found an error in the following worklfows:
1306.0 step3

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step3_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log
----- Begin Fatal Exception 31-Jan-2014 09:57:11 CET-----------------------
An exception of category 'NoProxyException' occurred while
   [0] Processing run: 1
   [1] Calling beginRun for unscheduled module ConversionTrackCandidateProducer/'conversionTrackCandidates'
   [2] Using EventSetup component NavigationSchoolESProducer/'navigationSchoolESProducer' to make data NavigationSchool/'SimpleNavigationSchool' in record NavigationSchoolRecord
Exception Message:
No data of type "MagneticField" with label "ParabolicMf" in record "IdealMagneticFieldRecord"
 Please add an ESSource or ESProducer to your job which can deliver this data.
----- End Fatal Exception -------------------------------------------------

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2244/4/summary.html

@mtosi
Copy link
Contributor

mtosi commented Feb 3, 2014

ciao,
I've just realized this isuue
and I'm trying to fix it
I'll be back soon
@Martin-Grunewald @perrotta @fwyzard @cerati @rovere

@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre3, CMSSW_7_1_0_pre2 Feb 5, 2014
@slava77
Copy link
Contributor

slava77 commented Feb 5, 2014

@mtosi @tropiano
Mia and Antonio: is this PR superseded by #2244?
If so, Antonio, please close this PR.

@mtosi
Copy link
Contributor

mtosi commented Feb 5, 2014

ciao,
Antonio quit CMS on Jan 31st
=> I do not know if he will be able to close this pull
the #2291 fixes the problem spotted in
#2244
=> yes, #2244 is superseded

mia

On Wed, Feb 5, 2014 at 1:36 PM, slava77 [email protected] wrote:

@mtosi https://github.com/mtosi @tropiano https://github.com/tropiano
Mia and Antonio: is this PR superseded by #2244#2244
?
If so, Antonio, please close this PR.

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

@ktf ktf closed this Feb 5, 2014
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre4, CMSSW_7_1_0_pre3 Feb 24, 2014
@nclopezo nclopezo modified the milestones: CMSSW_7_1_0_pre5, CMSSW_7_1_0_pre4 Mar 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants