Skip to content

Conversation

haydar-c
Copy link
Contributor

@haydar-c haydar-c commented Sep 9, 2025

This PR is for small changes in the FlatRecon to align the results for reproducibility. Currently there is only 1 minor change but I will be updating below list if required as I am collecting all parallel results.

Current Changes:

  • Remove same tile type condition from neighbor pass: At least for 1 circuit (denoise) with LAB utilization of 1.00 on titan_small, this check was eliminating some valid neighbors and making its packing unsuccessful by creating 4 more clusters than supplied type.

No other change is needed.

@github-actions github-actions bot added VPR VPR FPGA Placement & Routing Tool lang-cpp C/C++ code labels Sep 9, 2025
Copy link
Contributor

@vaughnbetz vaughnbetz left a comment

Choose a reason for hiding this comment

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

LGTM

@haydar-c haydar-c changed the title [WIP] FlatRecon Result Collection FlatRecon Result Collection Sep 17, 2025
@haydar-c haydar-c merged commit ee7f952 into master Sep 17, 2025
30 checks passed
@haydar-c haydar-c deleted the flatRecon_result_collection branch September 17, 2025 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-cpp C/C++ code VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants