Skip to content

[AP][GP] Tuning Parameters #3202

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.

Sign up for GitHub

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 1 commit into from
Jul 24, 2025
+4 −4
Merged

[AP][GP] Tuning Parameters #3202

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions vpr/src/analytical_place/analytical_solver.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -436,14 +436,14 @@ class B2BSolver : public AnalyticalSolver {
/// solved solution HPWL).
/// Decreasing this number toward zero would cause the B2B solver to run
/// more iterations to try and reduce the HPWL further.
static constexpr double b2b_convergence_gap_fac_ = 0.001;
static constexpr double b2b_convergence_gap_fac_ = 0.0001;

/// @brief The number of times the B2B loop should "converge" before stopping
/// the loop. Due to numerical inaccuracies, it is possible for the
/// HPWL to bounce up and down as it converges. Increasing this number
/// will allow more bounces which may get better quality; however
/// more iterations will need to be run.
static constexpr unsigned target_num_b2b_convergences_ = 2;
static constexpr unsigned target_num_b2b_convergences_ = 1;

/// @brief Max number of bound update / solve iterations. Increasing this
/// number will yield better quality at the expense of runtime.
Expand All @@ -460,7 +460,7 @@ class B2BSolver : public AnalyticalSolver {
/// to prevent this behaviour and get good runtime.
// TODO: Need to investigate this more to find a good number for this.
// TODO: Should this be a proportion of the design size?
static constexpr unsigned max_cg_iterations_ = 150;
static constexpr unsigned max_cg_iterations_ = 100;

// The following constants are used to configure the anchor weighting.
// The weights of anchors grow exponentially each iteration by the following
Expand Down
2 changes: 1 addition & 1 deletion vpr/src/analytical_place/global_placer.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ class SimPLGlobalPlacer : public GlobalPlacer {
/// between the two bounds, normalized to the upper-bound, is smaller
/// than this number.
/// This number was empircally found to work well.
static constexpr double target_hpwl_relative_gap_ = 0.05;
static constexpr double target_hpwl_relative_gap_ = 0.01;

/// @brief The solver which generates the lower-bound placement.
std::unique_ptr<AnalyticalSolver> solver_;
Expand Down