From: Arnaud Giersch <arnaud.giersch@univ-fcomte.fr>
Date: Tue, 17 May 2022 19:08:57 +0000 (+0200)
Subject: Cosmetics: declare graph edges with the "right" direction.
X-Git-Tag: v3.32~240
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/90d928f994ad7bc31969effd49b3b2ff501f1009

Cosmetics: declare graph edges with the "right" direction.
---

diff --git a/src/kernel/routing/StarZone.cpp b/src/kernel/routing/StarZone.cpp
index 16b72625a8..aab5ccdb40 100644
--- a/src/kernel/routing/StarZone.cpp
+++ b/src/kernel/routing/StarZone.cpp
@@ -75,10 +75,10 @@ void StarZone::get_graph(const s_xbt_graph_t* graph, std::map<std::string, xbt_n
     previous = star_node;
     for (auto const* link : routes_[src->id()].links_down) {
       xbt_node_t current = new_xbt_graph_node(graph, link->get_cname(), nodes);
-      new_xbt_graph_edge(graph, current, previous, edges);
+      new_xbt_graph_edge(graph, previous, current, edges);
       previous = current;
     }
-    new_xbt_graph_edge(graph, src_node, previous, edges);
+    new_xbt_graph_edge(graph, previous, src_node, edges);
   }
 }
 
diff --git a/tools/graphicator/graphicator.tesh b/tools/graphicator/graphicator.tesh
index a33759b850..928b6abfb7 100644
--- a/tools/graphicator/graphicator.tesh
+++ b/tools/graphicator/graphicator.tesh
@@ -27,25 +27,25 @@ $ cat test.dot
 >   "bob_cluster_link_6_DOWN";
 >   "bob_cluster_link_6_UP";
 >   "bobbob_cluster_router.hamburger.edu";
->   "bob_cluster_link_0_DOWN" -- "bob_cluster_backbone";
+>   "bob_cluster_link_0_DOWN" -- "bob0.hamburger.edu";
+>   "bob_cluster_link_2_DOWN" -- "bob2.hamburger.edu";
+>   "bob_cluster_link_3_DOWN" -- "bob3.hamburger.edu";
+>   "bob_cluster_link_4_DOWN" -- "bob4.hamburger.edu";
+>   "bob_cluster_link_6_DOWN" -- "bob6.hamburger.edu";
 >   "bob_cluster_link_0_UP" -- "bob_cluster_backbone";
->   "bob_cluster_link_2_DOWN" -- "bob_cluster_backbone";
 >   "bob_cluster_link_2_UP" -- "bob_cluster_backbone";
->   "bob_cluster_link_3_DOWN" -- "bob_cluster_backbone";
 >   "bob_cluster_link_3_UP" -- "bob_cluster_backbone";
->   "bob_cluster_link_4_DOWN" -- "bob_cluster_backbone";
 >   "bob_cluster_link_4_UP" -- "bob_cluster_backbone";
->   "bob_cluster_link_6_DOWN" -- "bob_cluster_backbone";
 >   "bob_cluster_link_6_UP" -- "bob_cluster_backbone";
->   "bob0.hamburger.edu" -- "bob_cluster_link_0_DOWN";
+>   "bob_cluster_backbone" -- "bob_cluster_link_0_DOWN";
 >   "bob0.hamburger.edu" -- "bob_cluster_link_0_UP";
->   "bob2.hamburger.edu" -- "bob_cluster_link_2_DOWN";
+>   "bob_cluster_backbone" -- "bob_cluster_link_2_DOWN";
 >   "bob2.hamburger.edu" -- "bob_cluster_link_2_UP";
->   "bob3.hamburger.edu" -- "bob_cluster_link_3_DOWN";
+>   "bob_cluster_backbone" -- "bob_cluster_link_3_DOWN";
 >   "bob3.hamburger.edu" -- "bob_cluster_link_3_UP";
->   "bob4.hamburger.edu" -- "bob_cluster_link_4_DOWN";
+>   "bob_cluster_backbone" -- "bob_cluster_link_4_DOWN";
 >   "bob4.hamburger.edu" -- "bob_cluster_link_4_UP";
->   "bob6.hamburger.edu" -- "bob_cluster_link_6_DOWN";
+>   "bob_cluster_backbone" -- "bob_cluster_link_6_DOWN";
 >   "bob6.hamburger.edu" -- "bob_cluster_link_6_UP";
 >   "bob_cluster_backbone" -- "bob_cluster";
 >   "bobbob_cluster_router.hamburger.edu" -- "bob_cluster";
@@ -58,25 +58,25 @@ $ ${bindir:=.}/graphicator ${srcdir:=.}/teshsuite/platforms/one_cluster.xml test
 
 $ cat test.csv
 > src,dst
-> bob_cluster_link_0_DOWN,bob_cluster_backbone
+> bob_cluster_link_0_DOWN,bob0.hamburger.edu
+> bob_cluster_link_2_DOWN,bob2.hamburger.edu
+> bob_cluster_link_3_DOWN,bob3.hamburger.edu
+> bob_cluster_link_4_DOWN,bob4.hamburger.edu
+> bob_cluster_link_6_DOWN,bob6.hamburger.edu
 > bob_cluster_link_0_UP,bob_cluster_backbone
-> bob_cluster_link_2_DOWN,bob_cluster_backbone
 > bob_cluster_link_2_UP,bob_cluster_backbone
-> bob_cluster_link_3_DOWN,bob_cluster_backbone
 > bob_cluster_link_3_UP,bob_cluster_backbone
-> bob_cluster_link_4_DOWN,bob_cluster_backbone
 > bob_cluster_link_4_UP,bob_cluster_backbone
-> bob_cluster_link_6_DOWN,bob_cluster_backbone
 > bob_cluster_link_6_UP,bob_cluster_backbone
-> bob0.hamburger.edu,bob_cluster_link_0_DOWN
+> bob_cluster_backbone,bob_cluster_link_0_DOWN
 > bob0.hamburger.edu,bob_cluster_link_0_UP
-> bob2.hamburger.edu,bob_cluster_link_2_DOWN
+> bob_cluster_backbone,bob_cluster_link_2_DOWN
 > bob2.hamburger.edu,bob_cluster_link_2_UP
-> bob3.hamburger.edu,bob_cluster_link_3_DOWN
+> bob_cluster_backbone,bob_cluster_link_3_DOWN
 > bob3.hamburger.edu,bob_cluster_link_3_UP
-> bob4.hamburger.edu,bob_cluster_link_4_DOWN
+> bob_cluster_backbone,bob_cluster_link_4_DOWN
 > bob4.hamburger.edu,bob_cluster_link_4_UP
-> bob6.hamburger.edu,bob_cluster_link_6_DOWN
+> bob_cluster_backbone,bob_cluster_link_6_DOWN
 > bob6.hamburger.edu,bob_cluster_link_6_UP
 > bob_cluster_backbone,bob_cluster
 > bobbob_cluster_router.hamburger.edu,bob_cluster