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

Phase-2 geometry clean up in 14_0 #43251

Open
srimanob opened this issue Nov 12, 2023 · 16 comments
Open

Phase-2 geometry clean up in 14_0 #43251

srimanob opened this issue Nov 12, 2023 · 16 comments

Comments

@srimanob
Copy link
Contributor

srimanob commented Nov 12, 2023

I've opened this ticket to follow up on the next round of Phase-2 geometry clean up. The proposal is to remove all geometry with I14 and I15. In short, all geometries from D86 - D94, and D97 will be removed.

  • D86 = T24+C17+M10+I14+O8+F8 (to be deleted)
  • D88 = T24+C17+M10+I15+O9+F8 (to be deleted)
  • D91 = T30+C17+M10+I15+O9+F8 (to be deleted)
  • D92 = T24+C18+M10+I15+O9+F8 (to be deleted)
  • D93 = T24+C19+M10+I15+O9+F8 (to be deleted)
  • D94 = T24+C20+M10+I15+O9+F8 (to be deleted)
  • D95 = T31+C17+M10+I16+O9+F8
  • D96 = T31+C18+M10+I16+O9+F8
  • D97 = T25+C17+M10+I15+O9+F8 (to be deleted)
  • D98 = T32+C17+M10+I16+O9+F8 (Current Phase-2 baseline from CMSSW_13_2_0_pre2)
  • D99 = T32+C18+M10+I16+O9+F8
  • D100 = T34+C17+M11+I16+O9+F8
  • D101 = T34+C18+M11+I16+O9+F8
  • D102 = T35+C17+M11+I16+O9+F8
  • D103 = T34+C21+M11+I16+O9+F8

Please share your concerns, or if new combinations are needed.

@cmsbuild
Copy link
Contributor

A new Issue was created by @srimanob Phat Srimanobhas.

@sextonkennedy, @smuzaffar, @antoniovilela, @Dr15Jones, @makortel, @rappoccio can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@srimanob
Copy link
Contributor Author

FYI @cms-sw/mtd-dpg-l2 @cms-sw/simulation-l2 @cms-sw/geometry-l2

@srimanob
Copy link
Contributor Author

assign upgrade

@cmsbuild
Copy link
Contributor

New categories assigned: upgrade

@AdrianoDee,@srimanob you have been requested to review this Pull request/Issue and eventually sign? Thanks

@epalencia
Copy link
Contributor

We (L1T) would like to keep D88 (the reason is that this was the geometry used in the last L1T MC campaign in 12_5) until, approx., mid 2024 (we would like to have our next set of Phase 2 samples produced in 14_1, hopefully available by March/April).

@mmusich
Copy link
Contributor

mmusich commented Dec 5, 2023

adding @cms-sw/trk-dpg-l2 @emiglior as well.

@emiglior
Copy link
Contributor

emiglior commented Dec 5, 2023

ph-2 tracker has some interest in keeping D91 as it is the only geometry including T30 aka "w/ realistic TFPX description".
Depending on the timescale for this clean up, we can deliver a Tracker geometry with the same TFPX description as in T30, but with a description of the rest of Inner Tracker closer to the final project.

@mmusich
Copy link
Contributor

mmusich commented Dec 5, 2023

Depending on the timescale for this clean up,

if this is still intended for 14_0_0 that will be closed at the end of Jan / beginning of Feb 2024.

@emiglior
Copy link
Contributor

emiglior commented Dec 5, 2023

adding @bartonaz @Raffaella07

@srimanob
Copy link
Contributor Author

@cms-sw/l1-l2 Following on this, should we target clean up in 14_2?

@cms-sw/trk-dpg-l2, I understand from your plan that you will combine T30 with T35. For now, can you do T30 on top of D110 (T35+C18+M11+I17+O9+F8), so that we don't need to keep I15 anymore. It is a primary goal of this clean up.

Thx. FYI @cms-sw/mtd-dpg-l2

@cmsbuild
Copy link
Contributor

cms-bot internal usage

@bsunanda
Copy link
Contributor

Also we need to keep a scenario with C20. If D94 is to be deleted we have to create one with C20 as D111

@fabiocos
Copy link
Contributor

@cms-sw/upgrade-l2 when is this clean up supposed to finally happen?

@fabiocos
Copy link
Contributor

fabiocos commented Jul 1, 2024

@cms-sw/upgrade-l2 we have now almost ready the (final) update of our BTL numbering scheme, prepared by @cquarant and discussed with @cms-sw/mtd-dpg-l2 , that is compatible with BTL v2 and v3 (i.e. MTD I16 or I17), but not with previous versions. We would like to know when alla scenarios up to D94 included, and D97, can be finally dropped from the development releases, as discussed back during last Fall/Winter.

@srimanob
Copy link
Contributor Author

srimanob commented Jul 1, 2024

@cms-sw/upgrade-l2 we have now almost ready the (final) update of our BTL numbering scheme, prepared by @cquarant and discussed with @cms-sw/mtd-dpg-l2 , that is compatible with BTL v2 and v3 (i.e. MTD I16 or I17), but not with previous versions. We would like to know when alla scenarios up to D94 included, and D97, can be finally dropped from the development releases, as discussed back during last Fall/Winter.

@cms-sw/l1-l2 Do you agree to drop as original proposal?
@bsunanda Could you please help to move geometry with HFNose?
Thx.

@epalencia
Copy link
Contributor

Since the D95 and D110 geometries (used for the 13_1 and 14_0 MC productions) would remain available, L1T is ok with the current proposal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants