-
Notifications
You must be signed in to change notification settings - Fork 414
[Documentation] Missing Documentation on --router_profiler_astar_fac
#2536
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
Comments
@amin1377 Can you handle this? |
Hi Alex, thanks for filing the issue. Sure. I'll add the documentation. |
I made the PR for that. However, we need to think about a way to automatically update the documents based on the parameters listed under |
@amin1377 Thank you so much for handling this! I agree, perhaps some Github bot can be used which will detect when read_options.cpp has changed on master and automatically update the documentation if the version in master does not match. Something that can be investigated in the future! |
@AlexandreSinger Thanks, Alex. It is fixed now. |
@amin1377 Thank you! |
While looking into how the CI runs the Titan benchmarks, specifically for
titan_quick_qor
:https://github.com/verilog-to-routing/vtr-verilog-to-routing/blob/3e8d605f292eea2c0a47964c21c56e995e76cad2/vtr_flow/tasks/regression_tests/vtr_reg_nightly_test2/titan_quick_qor/config/config.txt#L70C147-L70C174
I found that it sets an option called
--router_profiler_astar_fac
to 1.5. I noticed that no documentation on this command line argument exists on the VPR Command-Line Usage Documentation page:https://docs.verilogtorouting.org/en/latest/vpr/command_line_usage/
Based on my quick investigation this appears to be used mainly in the placer, not necessarily in the router but I could be incorrect. Some documentation on what this option does, what its default value is, and how it is different from the astar_fac command-line option would be nice.
The text was updated successfully, but these errors were encountered: