Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add BRAM numbers to resource estimate executive summary #2360

Merged
merged 1 commit into from
Jan 20, 2025
Merged

Conversation

polybeandip
Copy link
Collaborator

@polybeandip polybeandip commented Nov 20, 2024

Closes #2288

I've checked that this grabs the correct values. Say our main_utilization_placed.rpt file had the following BRAM table

nine
(not a real utilization file, I changed the highlighted entry from 0 to 9 to make sure I'm grabbing the correct value)

Running fud e --from synth-files --to resource-estimate report yields the correct BRAM count of 9:

{
  "lut": 169,
  "dsp": 0,
  "brams": 9,
  "registers": 60,
  "muxes": 30,
  "clb_registers": 210,
  "carry8": 10,
  "f7_muxes": 0,
  "f8_muxes": 0,
  "f9_muxes": 0,
  "clb": 389,
  "meet_timing": 1,
  "worst_slack": 4.758,
  "period": 7.0,
  "frequency": 142.857,
  "uram": -1,
  "cell_lut1": 5,
  "cell_lut2": 110,
  "cell_lut3": 38,
  "cell_lut4": 17,
  "cell_lut5": 18,
  "cell_lut6": 49,
  "cell_fdre": 210
}

@rachitnigam
Copy link
Contributor

Let's merge #2356 first which is a more substantial rework of the extraction logic.

Copy link
Contributor

This pull request has not seen activity in 14 days and is being marked as stale. If you're continuing work on this, please reply and state how to get this PR in a mergeable state or what issues it is blocked on. If the PR is not ready for review, please mark it as a draft. The PR will be closed in 7 days if there is no further activity.

@rachitnigam
Copy link
Contributor

Let's merge this!

@rachitnigam rachitnigam merged commit 211fa8a into main Jan 20, 2025
18 checks passed
@rachitnigam rachitnigam deleted the brams branch January 20, 2025 16:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resource estimate executive summary lacks BRAM numbers
2 participants