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
Validation/Geometry: Tracker Air Categorization #25231
Conversation
* Air is not part of the Categorization Map
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25231/7253 |
please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @vargasa (Andrés Vargas) for master. It involves the following packages: Validation/Geometry @andrius-k, @Dr15Jones, @kmaeshima, @cvuosalo, @schneiml, @ianna, @mdhildreth, @cmsbuild, @jfernan2, @civanch can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
+geometry |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
@vargasa this PR removes several LogInfo dumping information about the material. Are they not considered any more interesting? |
@fabiocos none of the removed code was being executed as stated in the commit description |
@vargasa ok, thanks for the clarification. I am not sure why the matTable was empty (could you please clarify ?), but of course if this is the case it makes sense. |
+1 |
@fabiocos The map for the categorization process that is being used is created in a later stage, this |
Logs were showing that Air was being sent to 'Other' Category:
MaterialBudgetData: Material forced to 'Other': Air in volume Tracker
Air itself is not part of the tracker maps (x0, x1), and it may be related with a previous issue #23734 in which Air category histograms were rendering empty.
This PR addresses this issue as well as make independent Tracker/HGCal building maps process and some cleaning. @apsallid confirmed none of these changes affect HGCal work