-
Notifications
You must be signed in to change notification settings - Fork 416
Vpr noc output #2333
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
Vpr noc output #2333
Conversation
Modified print_placement_move_types_stats() function to create a table to show the statistics in a more organized and intuitive manner.
Removed repetitive entries in the "block type" column to improve visual appeal.
Please add an example output so we can look at. |
Ran make format to pass all CI tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code change looks fine. Thanks!
Looks good. A few suggested tweaks: Round off the "Percentage(%)" column values to 2 digits after the decimal. |
Made small adjustments to the names of the headers, and fixed spacing.
Ready to be merged. @vaughnbetz |
Updating NoC usage output message format
Description
Updated the NoC usage output messages to organize the data into a table for better visual appeal and intuitiveness
Related Issue
Motivation and Context
Makes it easier for users to view the information when it is organized
How Has This Been Tested?
Types of changes
Checklist: