-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add SimBeamSpotObjects
to MC Global Tags
#86
Comments
In order to help with this step
and hence make progress with cms-sw/cmssw#42664 I started to compile the coupling Global Tag <-> Follows list of GT ⇆ tag couplings
|
While compiling this list, one issue with this approach that occurred to me, is that the current Below the list of GaussVtxSigmaZ4cmSmearingParameters : cms.PSet(
MeanX = cms.double(0.0),
MeanY = cms.double(0.0),
MeanZ = cms.double(0.0),
SigmaX = cms.double(0.0015),
SigmaY = cms.double(0.0015),
SigmaZ = cms.double(4.0),
TimeOffset = cms.double(0.0)
)
GaussVtxSmearingParameters : cms.PSet(
MeanX = cms.double(0.0),
MeanY = cms.double(0.0),
MeanZ = cms.double(0.0),
SigmaX = cms.double(0.0015),
SigmaY = cms.double(0.0015),
SigmaZ = cms.double(5.3),
TimeOffset = cms.double(0.0)
)
HLLHCCrabKissingVtxSmearingParameters : cms.PSet(
BeamProfile = cms.string('Flat'),
BetaStarCrossingPlaneInm = cms.double(0.3),
BetaStarParallelPlaneInm = cms.double(0.075),
CrabAngleCrossingPlaneInurad = cms.double(200.0),
CrabAngleParallelPlaneInurad = cms.double(100.0),
CrabFrequencyCrossingPlaneInMHz = cms.double(400.0),
CrabFrequencyParallelPlaneInMHz = cms.double(400.0),
EprotonInGeV = cms.double(6500.0),
HalfCrossingAngleInurad = cms.double(200.0),
MeanXIncm = cms.double(0.0),
MeanYIncm = cms.double(0.0),
MeanZIncm = cms.double(0.0),
NormalizedEmittanceCrossingPlaneInum = cms.double(2.5),
NormalizedEmittanceParallelPlaneInum = cms.double(2.5),
TimeOffsetInns = cms.double(0.0),
ZsizeInm = cms.double(0.15)
)
HLLHCVtxSmearingParameters : cms.PSet(
BetaCrossingPlaneInm = cms.double(0.2),
BetaSeparationPlaneInm = cms.double(0.2),
BunchLengthInm = cms.double(0.09),
CrabFrequencyInMHz = cms.double(400.0),
CrabbingAngleCrossingInurad = cms.double(380.0),
CrabbingAngleSeparationInurad = cms.double(0.0),
CrossingAngleInurad = cms.double(510.0),
EprotonInGeV = cms.double(6500.0),
HorizontalEmittance = cms.double(2.5e-06),
MeanXIncm = cms.double(0.0),
MeanYIncm = cms.double(0.0),
MeanZIncm = cms.double(0.0),
RF800 = cms.bool(False),
TimeOffsetInns = cms.double(0.0),
VerticalEmittance = cms.double(2.05e-06)
)
Run3FlatOpticsGaussVtxSigmaZ4p2cmSmearingParameters : cms.PSet(
MeanX = cms.double(0.0107682),
MeanY = cms.double(0.041722),
MeanZ = cms.double(0.035748),
SigmaX = cms.double(0.00118),
SigmaY = cms.double(0.00055),
SigmaZ = cms.double(4.2),
TimeOffset = cms.double(0.0)
)
Run3FlatOpticsGaussVtxSigmaZ5p3cmSmearingParameters : cms.PSet(
MeanX = cms.double(0.0107682),
MeanY = cms.double(0.041722),
MeanZ = cms.double(0.035748),
SigmaX = cms.double(0.0015),
SigmaY = cms.double(0.0013),
SigmaZ = cms.double(5.3),
TimeOffset = cms.double(0.0)
) |
@mmusich , would be correct if we will try to resolve the issue in the next release cycle? The dead-line for 13_3 is too close. |
Personally, I don't mind. I am waiting the alca people to comment on the points above. @saumyaphor4252 @francescobrivio |
may I ask where we stand with this issue . @saumyaphor4252 @francescobrivio |
I plan to take a look this afternoon, sorry for the delay! |
Thanks @mmusich for compiling the list and pointing out the missing ingredients!
I compiled a list of all the members that would be needed:
So, given that the original But there are a few parameters defined in
That are not used in any Alternatively one could think about updating the current simBS object with the parameters needed by the GaussEvtVtxGenerator now, and postpone this migration for the Phase2 GT to later, by making an entirely new record/object for the HLLHC case. Would this be technically doable (i.e. to decaouple the Phase2 GT update)? What do you think? I'm open to suggestions! |
I think it's probably better to settle the question now that the record is "virgin" (never used in production). About the parameters never referenced in the source code we should follow up with the developers that created the configuration in the first place. |
Dear @lgray, We are trying to migrate the vertex smearing parameters to the GT (see issue description and the following discussion for details), and we noticed that in cms-sw/cmssw#16942 you removed some of the parameters used by Was that done on purpose? Or simply forgotten? Thanks a lot! Cheers, |
First part of this issue, regarding Run-1/2/3 GTs, is almost done, see: For the Phase-2 GT: the work to define a new CondFormat is ongoing in cms-sw/cmssw#43186 |
Run-1/2/3 MC GTs are being updated in cms-sw/cmssw#43197 |
my take is that this issue can be closed. Is there anything outstanding @francescobrivio ? |
The only remaining open point is the HL-LHC case (GT, CondFormat. etc...). |
Oh the reasoning from back in the day was that those parameterizations did not need those parameters. |
Thanks for the feedback @lgray! No problem I can take care of the cleanup! If you don't mind, could you have a look at cms-sw/cmssw#43186 and provide any feedback in case you have it? |
Closing in favor of #95 which will be used to track the remaining open points for the HL-LHC case. |
Multi-step solution for cms-sw/cmssw#41894:
SimBeamSpotObjects
codeSimBeamSpotObjects
handling utilities cms-sw/cmssw#41967Which GTs should actually be updated, only Run3 or also Run2 and Run1?done hereThe correct correspondence between GT <-> tagdone hereThis is probably already implemented, but I need to find where exactly in the codedone in Read VertexSmearing from GT in Run-1/2/3 MC cms-sw/cmssw#43242The text was updated successfully, but these errors were encountered: