[Spelling] Fixed Spelling in VPR Test, Draw, Place, and Timing

These folders in VPR had ok spelling. Route, power, and pack have a lot
more spelling mistakes and will come in another PR.

Author: AlexandreSinger

vpr/scripts/profile/parse_and_plot_detailed.py

@@ -12,7 +12,7 @@
 import csv
 from datetime import datetime
 
-# generate images wihtout having a window appear
+# generate images without having a window appear
 import matplotlib
 
 matplotlib.use("Agg")

vpr/src/draw/breakpoint.h

@@ -6,13 +6,13 @@
  * 
  * This file holds the declaration of the breakpoint class, and also some of the breakpoint related functions.
  * 
- * This class holds the definiton of type Breakpoint as well as all related functions. 
+ * This class holds the definition of type Breakpoint as well as all related functions. 
  * Breakpoints can be set through the GUI anytime during placement or routing. Breakpoints can also be activated, 
  * deactivated, and deleted. Each breakpoint has a type (BT_MOVE_NUM, BT_TEMP_NUM, BT_FROM_BLOCK, BT_ROUTER_ITER, BT_ROUTE_NET_ID, BT_EXPRESSION) and holds the 
  * values for corresponding to its type, as well as a boolean variable to activate and deactivate a breakpoint. It should be noted that each breakpoint can only have one type and hold one value corresponding to that type. 
  * More complicated breakpoints are set using an expression. (e.g move_num > 3 && block_id == 11) 
- * Breakpoints can be create using 3 constructors, the default contructor that doesn't identify the type and sets a breakpoint that is never reached, 
- * a constructor that takes in the type and an int value, and lastly a constructor that takes in the type and the sting that holds the expression. 
+ * Breakpoints can be create using 3 constructors, the default constructor that doesn't identify the type and sets a breakpoint that is never reached, 
+ * a constructor that takes in the type and an int value, and lastly a constructor that takes in the type and the string that holds the expression. 
  * (e.g Breakpoint(BT_MOVE_NUM, 4) or Breakpoint(BT_EXPRESSION, "move_num += 3")) The == operator has also been provided which returns true when two 
  * breakpoints have the same type, and the same value corresponding to the type.
  */
@@ -69,7 +69,7 @@ class Breakpoint {
             bt_route_net_id = breakpoint_value;
     }
 
-    //sets the breakpoint type to BT_EXPRESSION and sets the string member bt_expression to the expresssion the user inputted. e.g Breakpoint(BT_EXPRESSION, "move_num==3")
+    //sets the breakpoint type to BT_EXPRESSION and sets the string member bt_expression to the expression the user inputted. e.g Breakpoint(BT_EXPRESSION, "move_num==3")
     //the user can set breakpoints, during routing or placement, using an expression with the available variables and operators
     Breakpoint(bp_type ty, std::string expr) {
         type = ty;
@@ -99,7 +99,7 @@ class Breakpoint {
     }
 };
 
-//breakpoint realted functions to be called from draw.cpp, place.cpp and route_timing.cpp
+//breakpoint related functions to be called from draw.cpp, place.cpp and route_timing.cpp
 
 //checks for all types of breakpoint and returns true if a breakpoint is encountered
 bool check_for_breakpoints(bool in_placer);

vpr/src/draw/draw.cpp

@@ -1334,7 +1334,7 @@ ezgl::color get_block_type_color(t_physical_tile_type_ptr type) {
     return color;
 }
 
-//Lightens a color's luminance [0, 1] by an aboslute 'amount'
+//Lightens a color's luminance [0, 1] by an absolute 'amount'
 ezgl::color lighten_color(ezgl::color color, float amount) {
     constexpr double MAX_LUMINANCE = 0.95; //Clip luminance so it doesn't go full white
     auto hsl = color2hsl(color);

vpr/src/draw/draw.h

@@ -2,7 +2,7 @@
 /**
  * @file draw.h
  *
- * The main drawing file. Contains the setup for ezgl application, ui setup, and graphis functions
+ * The main drawing file. Contains the setup for ezgl application, ui setup, and graphics functions
  *
  * This is VPR's main graphics application program. The program interacts with ezgl/graphics.hpp,
  * which provides an API for displaying graphics on both X11 and Win32. The most important

vpr/src/draw/draw_basic.cpp

@@ -520,13 +520,13 @@ void draw_routing_bb(ezgl::renderer* g) {
     //
     //In the graphics we represent this by drawing the BB so that legal RR node start/end points
     //are contained within the drawn box. Since VPR associates each x/y channel location to
-    //the right/top of the tile with the same x/y cordinates, this means we draw the box so that:
+    //the right/top of the tile with the same x/y coordinates, this means we draw the box so that:
     //  * The left edge is to the left of the channel at bb xmin (including the channel at xmin)
     //  * The bottom edge is to the below of the channel at bb ymin (including the channel at ymin)
     //  * The right edge is to the right of the channel at bb xmax (including the channel at xmax)
     //  * The top edge is to the right of the channel at bb ymax (including the channel at ymax)
     //Since tile_x/tile_y correspond to the drawing coordinates the block at grid x/y's bottom-left corner
-    //this means we need to shift the top/right drawn co-ordinate one tile + channel width right/up so
+    //this means we need to shift the top/right drawn coordinate one tile + channel width right/up so
     //the drawn box contains the top/right channels
     double draw_xlow = draw_coords->tile_x[bb->xmin];
     double draw_ylow = draw_coords->tile_y[bb->ymin];
@@ -617,7 +617,7 @@ void draw_routed_net(ParentNetId net_id, ezgl::renderer* g) {
             draw_state->draw_rr_node[inode].color = DEFAULT_RR_NODE_COLOR;
         }
 
-        // When drawing a new branch, add the parent node to the vector to ensure that the conenction is drawn.
+        // When drawing a new branch, add the parent node to the vector to ensure that the connection is drawn.
         if (rr_nodes_to_draw.empty() && rt_node.parent().has_value()) {
             rr_nodes_to_draw.push_back(rt_node.parent().value().inode);
         }

vpr/src/draw/draw_basic.h

@@ -127,7 +127,7 @@ void draw_routed_timing_edge_connection(tatum::NodeId src_tnode,
                                         ezgl::renderer* g);
 
 /* Draws any placement macros (e.g. carry chains, which require specific relative placements
- * between some blocks) if the Placement Macros (in the GUI) is seelected.
+ * between some blocks) if the Placement Macros (in the GUI) is selected.
  */
 void draw_placement_macros(ezgl::renderer* g);
 

vpr/src/draw/draw_color.h

@@ -165,7 +165,7 @@ static constexpr ezgl::color blk_ALICEBLUE(0xF0, 0xF8, 0xFF);
 //The colours used to draw block types
 const std::vector<ezgl::color> block_colors{
     //This first set of colours is somewhat curated to yield
-    //a nice colour pallette
+    //a nice colour palette
     blk_BISQUE, //EMPTY type is usually the type with index 0, so this colour
                 //usually unused
     blk_LIGHTGREY,

vpr/src/draw/draw_debug.cpp

@@ -4,7 +4,7 @@
 #include "draw_global.h"
 #include "vtr_expr_eval.h"
 
-//keeps track of open windows to avoid reopenning windows that are alerady open
+//keeps track of open windows to avoid reopenning windows that are already open
 struct open_windows {
     bool debug_window = false;
     bool advanced_window = false;
@@ -31,7 +31,7 @@ class DrawDebuggerGlobals {
     }
 };
 
-//the global variable that holds all global variables realted to breakpoint graphics
+//the global variable that holds all global variables related to breakpoint graphics
 DrawDebuggerGlobals draw_debug_glob_vars;
 
 //draws main debugger window
@@ -360,7 +360,7 @@ void refresh_bpList() {
         gtk_widget_set_halign(checkbox, GTK_ALIGN_END);
         gtk_widget_set_valign(checkbox, GTK_ALIGN_CENTER);
 
-        //button seetings
+        //button settings
         GtkWidget* deleteButton = gtk_button_new();
         GtkWidget* image = gtk_image_new_from_file("src/draw/trash.png");
         gtk_button_set_image((GtkButton*)deleteButton, image);

vpr/src/draw/draw_noc.cpp

@@ -36,7 +36,7 @@ void draw_noc(ezgl::renderer* g) {
         return;
     }
 
-    // check that the NoC tile has a capacity greater than 0 (can we assume it always will?) and if not then we cant draw anything as the NoC tile won't be drawn
+    // check that the NoC tile has a capacity greater than 0 (can we assume it always will?) and if not then we can't draw anything as the NoC tile won't be drawn
     /* since the vector of routers all have a reference positions on the grid to the corresponding physical tile, just use the first router in the vector and get its position, then use this to get the capacity of a noc router tile
      */
     const auto& type = device_ctx.grid.get_physical_type({router_list.begin()->get_router_grid_position_x(),
@@ -54,7 +54,7 @@ void draw_noc(ezgl::renderer* g) {
     // Now construct the coordinates for the markers that represent the connections between links (relative to the noc router tile position)
     ezgl::rectangle noc_connection_marker_bbox = get_noc_connection_marker_bbox(noc_router_logical_type);
 
-    // only draw the noc useage if the user selected the option
+    // only draw the noc usage if the user selected the option
     if (draw_state->draw_noc == DRAW_NOC_LINK_USAGE) {
         draw_noc_usage(noc_link_colors);
 

vpr/src/draw/draw_searchbar.h

@@ -58,7 +58,7 @@ void draw_highlight_fan_in_fan_out(const std::set<RRNodeId>& nodes);
 std::set<RRNodeId> draw_expand_non_configurable_rr_nodes(RRNodeId hit_node);
 
 /* Sets the color of all clbs, nets and rr_nodes to the default.
- * as well as clearing the highlighed sub-block */
+ * as well as clearing the highlighted sub-block */
 void deselect_all();
 
 #endif /* NO_GRAPHICS */