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
Improve HGCAL geometry handling in fireworks #35558
Conversation
Trying to open the object panel in Fireworks while displaying Tracksters, an error appears complaining about a missing symbol. That was due to a declaration without any implementation of a trackster's method. This removes that.
Make use of the topology information, in the form of 9 floats for each DetId, to pass HGCAL specific information to Fireworks. This will avoid the weak and error prone practice of unpacking the DetId bits while rendering the objects via the dedicated proxies. Three floats are still available. This will increase the coupling between the release, the geometry dumping and the explored file. On the other hand, at present, the unpacking is wrong for new geometries.
Rename private variables with a final '_', in accordance to CMSSW coding rules. Also, make use of the topology information now stored inside the geometry description of Fireworks. This makes the code easier to read and maintain. Finally, make Trackster-as-layer fully heatmap compatible.
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35558/25791
|
A new Pull Request was created by @rovere (Marco Rovere) for master. It involves the following packages:
@Dr15Jones, @alja, @makortel, @cmsbuild, @AdrianoDee, @srimanob, @slava77, @jpata can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9b6982/19435/summary.html Comparison SummarySummary:
|
+1 |
ping upgrade |
+Upgrade For the upgrade related code, only technical change is implemented. One unused inline function is removed. |
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Trying to open the object panel in Fireworks while displaying
Tracksters, an error appears complaining about a missing symbol. That
was due to a declaration without any implementation of a trackster's
method. This removes that.
Make use of the topology information, in the form of 9 floats for each
DetId, to pass HGCAL specific information to Fireworks. This will avoid
the weak and error-prone practice of unpacking the DetId bits while
rendering the objects via the dedicated proxies. Three floats are still
available. This will increase the coupling between the release, the
geometry dumping and the explored file. On the other hand, at present,
the unpacking is wrong for new geometries.
Rename private variables with a final '_', in accordance to CMSSW coding
rules. Also, make use of the topology information now stored inside the
geometry description of Fireworks. This makes the code easier to read
and maintain. Finally, make Trackster-as-layer fully heatmap compatible.
PR validation:
usual
runTheMatrix.py -l limited -i all
if this PR is a backport please specify the original PR and why you need to backport that PR: