Skip to content

Update cluster gains once per cluster #1886

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

Merged
merged 16 commits into from
Nov 7, 2021
Merged

Conversation

sfkhalid
Copy link
Contributor

Description

Currently during clustering the gain values are updated once for each level in a t_pb - from the root node down to the child node. However, the gain information only needs to be stored for the root pb, - the gain stored for the child pbs is never used. This pull request fixes the excess updates done during clustering.

@github-actions github-actions bot added the VPR VPR FPGA Placement & Routing Tool label Oct 19, 2021
@sfkhalid
Copy link
Contributor Author

QoR comparison before and after gain update changes:
gain_updates_qor_comparison.xlsx

No significant changes in pack time.

@vaughnbetz
Copy link
Contributor

@sfkhalid : there are warnings in the code that need to be fixed to pass CI.

@sfkhalid
Copy link
Contributor Author

@sfkhalid : there are warnings in the code that need to be fixed to pass CI.

Yep, I'm looking into fixing those now.

@sfkhalid
Copy link
Contributor Author

QoR Results Comparisons:
gain_updates_qor_comparison_2.xlsx
titan_qor_comparison.xlsx
No significant changes in pack time.

@sfkhalid sfkhalid merged commit bae983f into master Nov 7, 2021
@sfkhalid sfkhalid deleted the improve_clustering_gain_updates branch November 7, 2021 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants