search

cms-L1TK / cmssw

Fork of CMSSW where improvements to L1 tracking code are developed.
http://cms-sw.github.io/
Apache License 2.0
4 stars 5 forks source link

Seed stub IDs #202

Closed aehart closed 1 year ago

aehart commented 1 year ago

PR description:

This PR adds the seed stub IDs to the TrackBuilder output, for use by the TrackMerger.

PR validation:

I ran the code with writeMem_ = true, and saw the seed stub IDs correctly output in the resulting test vectors.

tomalin commented 1 year ago

If you're modifying the TrackBuilder output, please update https://twiki.cern.ch/twiki/bin/view/CMS/HybridDataFormat#TrackWord_StubWord_written_by_Tr to reflect the change.

tschuh commented 1 year ago

It would be superb if we could get in f/w the whole stubs including bend since downstream consumer requiring that information. Apart from that the PR looks good.

tomalin commented 1 year ago

If you're modifying the TrackBuilder output, please update https://twiki.cern.ch/twiki/bin/view/CMS/HybridDataFormat#TrackWord_StubWord_written_by_Tr to reflect the change.

@aehart I'm only waiting for this comment to be addressed before merging your PR.

aehart commented 1 year ago

If you're modifying the TrackBuilder output, please update https://twiki.cern.ch/twiki/bin/view/CMS/HybridDataFormat#TrackWord_StubWord_written_by_Tr to reflect the change.

@aehart I'm only waiting for this comment to be addressed before merging your PR.

Done.