address review comments

- Added better description on the user options
- Added test with random_shuffle nodes reordering

Signed-off-by: Alessandro Comodi <[email protected]>

Author: acomodi

vpr/src/base/read_options.cpp

@@ -2099,7 +2099,12 @@ argparse::ArgumentParser create_arg_parser(std::string prog_name, t_options& arg
             "            architectures)\n"
             " * map: An advanced lookahead which accounts for diverse wire type\n"
             " * extended_map: A more advanced and extended lookahead which accounts for a more\n"
-            "                 exhaustive node sampling method")
+            "                 exhaustive node sampling method\n"
+            "\n"
+            " The extended map differs from the map lookahead in the lookahead computation.\n"
+            " It is better suited for architectures that have specialized routing for specific\n"
+            " kinds of connections, but note that the time and memory necessary to compute the\n"
+            " extended lookahead map are greater than the basic lookahead map.\n")
         .default_value("map")
         .show_in(argparse::ShowIn::HELP_ONLY);
 

vtr_flow/tasks/regression_tests/vtr_reg_strong/strong_router_lookahead/config/config.txt

@@ -28,3 +28,4 @@ script_params_common = -starting_stage vpr --route_chan_width 60
 script_params_list_add = --router_lookahead classic
 script_params_list_add = --router_lookahead map
 script_params_list_add = --router_lookahead extended_map
+script_params_list_add = --router_lookahead extended_map --reorder_rr_graph_nodes_algorithm random_shuffle