Skip to content

Remove place_cost_exp from documentation #2863

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 2 commits into from
Jan 20, 2025
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
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
8 changes: 0 additions & 8 deletions doc/src/vpr/command_line_usage.rst
Original file line number Diff line number Diff line change
Expand Up @@ -868,14 +868,6 @@ Setting any of the following 5 options selects :ref:`Dusty's annealing schedule

**Default:** ``0.25``

.. option:: --place_cost_exp <float>

Wiring cost is divided by the average channel width over a net's bounding box
taken to this exponent. Only impacts devices with different channel widths in
different directions or regions.

**Default:** ``1``

.. option:: --RL_agent_placement {on | off}

Uses a Reinforcement Learning (RL) agent in choosing the appropiate move type in placement.
Expand Down
5 changes: 0 additions & 5 deletions vpr/src/base/vpr_types.h
Original file line number Diff line number Diff line change
Expand Up @@ -991,11 +991,6 @@ enum class e_move_type;
* @param timing_tradeoff
* When in CRITICALITY_TIMING_PLACE mode, what is the
* tradeoff between timing and wiring costs.
* @param place_cost_exp
* Wiring cost is divided by the average channel width over
* a net's bounding box taken to this exponent.
* Only impacts devices with different channel widths in
* different directions or regions. (Default: 1)
* @param place_chan_width
* The channel width assumed if only one placement is performed.
* @param pad_loc_type
Expand Down
6 changes: 2 additions & 4 deletions vpr/src/place/net_cost_handler.h
Original file line number Diff line number Diff line change
Expand Up @@ -253,17 +253,15 @@ class NetCostHandler {
* @details This is only useful for the cost function that takes the length of the net bounding box in each
* dimension divided by the average number of tracks in that direction. For other cost functions, you don't
* have to bother calling this routine; when using the cost function described above, however, you must always
* call this routine before you do any placement cost determination. The place_cost_exp factor specifies to
* what power the width of the channel should be taken -- larger numbers make narrower channels more expensive.
* call this routine before you do any placement cost determination.
*/
void alloc_and_load_chan_w_factors_for_place_cost_();

/**
* @brief Allocates and loads acc_tile_num_inter_die_conn_ which contains the accumulative number of inter-die
* conntections.
*
* @details This is only useful for multi-die FPGAs. The place_cost_exp factor specifies to
* what power the average number of inter-die connections should be take -- larger numbers make narrower channels more expensive.
* @details This is only useful for multi-die FPGAs.
*/
void alloc_and_load_for_fast_vertical_cost_update_();

Expand Down
Loading