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

Update Run2 reco geometry, new SLHC reco geometry #4

Merged
4 commits merged into from Aug 29, 2014
Merged

Conversation

alja
Copy link
Contributor

@alja alja commented Aug 9, 2014

I would like the three files (cmsGeom10.root, cmsGeomRun2.root, and cmsRecoGeom2019.root) to be integrated in the following releases:
7_1_X
7_2_X
6_2_SLHC_X

I know it is not trivial to integrate this in CMSSW 7x and SLHC releases. Let me know if I can make this any easier.

Thanks,
Alja

@alja
Copy link
Contributor Author

alja commented Aug 9, 2014

+1

@alja
Copy link
Contributor Author

alja commented Aug 10, 2014

+1 put 2019 extended geometry as default geometry for SLHC releases.

@ianna @osschar please follow that

@alja
Copy link
Contributor Author

alja commented Aug 11, 2014

@ktf @Degano @smuzaffar

Would really like the files in the next SLHC and the next 7_1_X release. It is the first time I do this and will probably take several iterations to get it right.

@alja
Copy link
Contributor Author

alja commented Aug 18, 2014

+1

@ktf
Copy link

ktf commented Aug 29, 2014

@Degano can you give an update on this?

ghost pushed a commit that referenced this pull request Aug 29, 2014
Update Run2 reco geometry, new SLHC reco geometry
@ghost ghost merged commit f64fc9c into cms-data:master Aug 29, 2014
@ghost
Copy link

ghost commented Aug 29, 2014

@alja The update should also be integrated in 72X or just 71X and SLHC?

On Fri, Aug 29, 2014 at 3:58 PM, Alessandro Degano a.degano@gmail.com
wrote:

Dure I'll do it Asap.
On Aug 29, 2014 3:42 PM, "Giulio Eulisse" notifications@github.com
wrote:

@Degano https://github.com/degano can you update this?


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

@alja
Copy link
Contributor Author

alja commented Aug 29, 2014

@Degano

Yes, the update is also for 72X.

Thanks a lot,
Alja

This pull request was closed.
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

2 participants