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
make Constrain Hits first class citizen #25333
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25333/7358 |
A new Pull Request was created by @VinInn (Vincenzo Innocente) for master. It involves the following packages: DQM/SiPixelPhase1Track @perrotta, @andrius-k, @kmaeshima, @schneiml, @civanch, @mdhildreth, @cmsbuild, @jfernan2, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@smuzaffar |
@VinInn, for code-checks we only need PR changes and it is quick to run to give fast feedback to L2s. For header checks we need to rebuild all packages which are affected by the change and takes much more time to run. Also frequency of getting internal compiler (ICE) errors is very high for header checks that is why we do not run them at code-checks time and also currently cms-bot does not mark PR tests failed based on the results of header checks (due to high frequency of ICE). |
+1 |
+1 |
@kpedro88 |
+upgrade |
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) |
@fabiocos |
This PR makes Transient Constrain Hits "first class citizen" so that they can be used in standard fits and pattern recognition.
Possible applications of the 5D Constrain can be to represent a "tracklet" (a seed for instance) to be used "asis" during CKF (think of L1 trackless or pixelTrack from FPGA or GPU) .
2D Constrains can be used as "Frozen Hits" w/o any further reference to clusters.
As the required mods touch very basic packages, here is a purely technical PR w/o ay further infrastructure to try/test any of the use case above.
This will allow further development w/o the need of recompiling half of the universe.
purely technical NO regression expected.