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

add more reports #37

Merged
merged 1 commit into from
May 3, 2021
Merged

add more reports #37

merged 1 commit into from
May 3, 2021

Conversation

timsnyder-siv
Copy link
Contributor

use -hierarchical_percentages in report_utilization
seems to indicate percent utilization while obeying hierarchical pblock constraints

report_control_sets after synthesis, gives nice overview of the clocking
in the design. Learned about this report from
https://forums.xilinx.com/t5/Implementation/Vivado-2014-3-1-place-error-place-30-487/td-p/589987

use -hierarchical_percentages in report_utilization
seems to indicate percent utilization while obeying hierarchical pblock constraints

report_control_sets after synthesis, gives nice overview of the clocking
in the design.
@timsnyder-siv
Copy link
Contributor Author

@davidbiancolin I don't seem to be able to add you as a reviewer to this for some reason. I guess the permissions on this repo are different.

@davidbiancolin davidbiancolin self-requested a review May 2, 2021 20:09
Copy link
Contributor

@davidbiancolin davidbiancolin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Can you add something to the main firesim release notes?

@timsnyder-siv
Copy link
Contributor Author

LGTM. Can you add something to the main firesim release notes?

Done. I also don't see the big green button in this repo. @davidbiancolin , I think you'll have to merge for me. Should I open a PR in firesim to bump aws-fpga-firesim to tip of dev or do you want to wait until closer to the release?

@davidbiancolin
Copy link
Contributor

Do you have permission now?

@timsnyder-siv
Copy link
Contributor Author

@davidbiancolin still no big green button and I'm not able to add reviewers.

@timsnyder-siv
Copy link
Contributor Author

After reading my email and accepting your invite, I see the button now.

@timsnyder-siv timsnyder-siv merged commit 6cd1a8c into firesim:dev May 3, 2021
@timsnyder-siv timsnyder-siv deleted the mo_reports_wip branch May 3, 2021 16:05
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.

3 participants