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

TrackBuilder updates #312

Merged
merged 5 commits into from
Jan 23, 2024
Merged

TrackBuilder updates #312

merged 5 commits into from
Jan 23, 2024

Conversation

aehart
Copy link
Contributor

@aehart aehart commented Dec 21, 2023

This PR makes three updates to the TrackBuilder:

  • The TrackBuilder now outputs something on every iteration. Previously, if the track that was built had too few matches, nothing would be written in the C++. Now, an invalid track is written, thus simulating the gaps in valid output that are present in the FW.
  • A "done" signal has been added, which goes high for one clock cycle when no more full matches are available for processing. Hopefully, this addresses DR wishes an extra done signal from TB #307.
  • The latency pragma has been added to the main processing loop. This is so that the latency of each TrackBuilder is the same, which I believe simplifies integration into chains with multiple TrackBuilders.

The latency and pipelining of the L1L2 TrackBuilder is unchanged, while the minimum clock period reported by HLS improves slightly from 3.486 ns to 3.420 ns. The LUT usage decreases slightly, while the FF usage increases:

L1L2 before L1L2 after
BRAM_18K 0 0
DSP48E 0 0
FF 7841 10908
LUT 35359 34445
URAM 0 0

TrackletAlgorithm/TrackBuilder.h Show resolved Hide resolved
emData/project_generation_scripts Outdated Show resolved Hide resolved
@jasonfan393 jasonfan393 merged commit 152dd59 into master Jan 23, 2024
1 check passed
@aehart aehart deleted the tb_updates branch April 16, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants