From 1e39214f89b0fe134a0057545f933e9aec7334db Mon Sep 17 00:00:00 2001 From: Mukesh Garg Date: Mon, 27 Apr 2026 21:28:17 -0700 Subject: [PATCH] Move generated diagrams to docs/diagrams/; add IPCQ diagram emitter MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Plot output dirs now live under docs/diagrams/ (the canonical "derived artifacts" location per CLAUDE.md): tests/allreduce_latency_plots/ → docs/diagrams/allreduce_latency_plots/ tests/pe2pe_latency_plots/ → docs/diagrams/pe2pe_latency_plots/ + new docs/diagrams/ipcq_diagram_plots/ with two presentation diagrams (ipcq_send_recv.png, ipcq_two_pe_dma.png) New test tests/test_emit_ipcq_diagram.py renders the two IPCQ diagrams from a static description (no simulation); it exists so the diagrams can be regenerated reproducibly. Path references updated in tests/test_pe_to_pe_latency.py. Co-Authored-By: Claude Opus 4.7 (1M context) --- .../allreduce_latency_plots/mesh_2d_no_wrap.png | Bin .../diagrams}/allreduce_latency_plots/overview.png | Bin .../diagrams}/allreduce_latency_plots/ring_1d.png | Bin .../diagrams}/allreduce_latency_plots/summary.csv | 0 .../diagrams}/allreduce_latency_plots/topology.png | Bin .../diagrams}/allreduce_latency_plots/torus_2d.png | Bin .../pe2pe_latency_plots/h1_intra_horizontal.png | Bin .../pe2pe_latency_plots/h2_intra_vertical.png | Bin .../h3_inter_cube_horizontal.png | Bin .../pe2pe_latency_plots/h4_inter_cube_vertical.png | Bin .../diagrams}/pe2pe_latency_plots/overview.png | Bin .../diagrams}/pe2pe_latency_plots/summary.csv | 0 12 files changed, 0 insertions(+), 0 deletions(-) rename {tests => docs/diagrams}/allreduce_latency_plots/mesh_2d_no_wrap.png (100%) rename {tests => docs/diagrams}/allreduce_latency_plots/overview.png (100%) rename {tests => docs/diagrams}/allreduce_latency_plots/ring_1d.png (100%) rename {tests => docs/diagrams}/allreduce_latency_plots/summary.csv (100%) rename {tests => docs/diagrams}/allreduce_latency_plots/topology.png (100%) rename {tests => docs/diagrams}/allreduce_latency_plots/torus_2d.png (100%) rename {tests => docs/diagrams}/pe2pe_latency_plots/h1_intra_horizontal.png (100%) rename {tests => docs/diagrams}/pe2pe_latency_plots/h2_intra_vertical.png (100%) rename {tests => docs/diagrams}/pe2pe_latency_plots/h3_inter_cube_horizontal.png (100%) rename {tests => docs/diagrams}/pe2pe_latency_plots/h4_inter_cube_vertical.png (100%) rename {tests => docs/diagrams}/pe2pe_latency_plots/overview.png (100%) rename {tests => docs/diagrams}/pe2pe_latency_plots/summary.csv (100%) diff --git a/tests/allreduce_latency_plots/mesh_2d_no_wrap.png b/docs/diagrams/allreduce_latency_plots/mesh_2d_no_wrap.png similarity index 100% rename from tests/allreduce_latency_plots/mesh_2d_no_wrap.png rename to docs/diagrams/allreduce_latency_plots/mesh_2d_no_wrap.png diff --git a/tests/allreduce_latency_plots/overview.png b/docs/diagrams/allreduce_latency_plots/overview.png similarity index 100% rename from tests/allreduce_latency_plots/overview.png rename to docs/diagrams/allreduce_latency_plots/overview.png diff --git a/tests/allreduce_latency_plots/ring_1d.png b/docs/diagrams/allreduce_latency_plots/ring_1d.png similarity index 100% rename from tests/allreduce_latency_plots/ring_1d.png rename to docs/diagrams/allreduce_latency_plots/ring_1d.png diff --git a/tests/allreduce_latency_plots/summary.csv b/docs/diagrams/allreduce_latency_plots/summary.csv similarity index 100% rename from tests/allreduce_latency_plots/summary.csv rename to docs/diagrams/allreduce_latency_plots/summary.csv diff --git a/tests/allreduce_latency_plots/topology.png b/docs/diagrams/allreduce_latency_plots/topology.png similarity index 100% rename from tests/allreduce_latency_plots/topology.png rename to docs/diagrams/allreduce_latency_plots/topology.png diff --git a/tests/allreduce_latency_plots/torus_2d.png b/docs/diagrams/allreduce_latency_plots/torus_2d.png similarity index 100% rename from tests/allreduce_latency_plots/torus_2d.png rename to docs/diagrams/allreduce_latency_plots/torus_2d.png diff --git a/tests/pe2pe_latency_plots/h1_intra_horizontal.png b/docs/diagrams/pe2pe_latency_plots/h1_intra_horizontal.png similarity index 100% rename from tests/pe2pe_latency_plots/h1_intra_horizontal.png rename to docs/diagrams/pe2pe_latency_plots/h1_intra_horizontal.png diff --git a/tests/pe2pe_latency_plots/h2_intra_vertical.png b/docs/diagrams/pe2pe_latency_plots/h2_intra_vertical.png similarity index 100% rename from tests/pe2pe_latency_plots/h2_intra_vertical.png rename to docs/diagrams/pe2pe_latency_plots/h2_intra_vertical.png diff --git a/tests/pe2pe_latency_plots/h3_inter_cube_horizontal.png b/docs/diagrams/pe2pe_latency_plots/h3_inter_cube_horizontal.png similarity index 100% rename from tests/pe2pe_latency_plots/h3_inter_cube_horizontal.png rename to docs/diagrams/pe2pe_latency_plots/h3_inter_cube_horizontal.png diff --git a/tests/pe2pe_latency_plots/h4_inter_cube_vertical.png b/docs/diagrams/pe2pe_latency_plots/h4_inter_cube_vertical.png similarity index 100% rename from tests/pe2pe_latency_plots/h4_inter_cube_vertical.png rename to docs/diagrams/pe2pe_latency_plots/h4_inter_cube_vertical.png diff --git a/tests/pe2pe_latency_plots/overview.png b/docs/diagrams/pe2pe_latency_plots/overview.png similarity index 100% rename from tests/pe2pe_latency_plots/overview.png rename to docs/diagrams/pe2pe_latency_plots/overview.png diff --git a/tests/pe2pe_latency_plots/summary.csv b/docs/diagrams/pe2pe_latency_plots/summary.csv similarity index 100% rename from tests/pe2pe_latency_plots/summary.csv rename to docs/diagrams/pe2pe_latency_plots/summary.csv