From cd922c4c12d75010625d1c84bf62dcb0ed3b2ac1 Mon Sep 17 00:00:00 2001
From: David Holdeman <dave@holdemanenterprises.com>
Date: Tue, 21 Mar 2023 20:58:04 -0500
Subject: [PATCH] comments

---
 bin/export-vrml.py |  1 +
 bin/export.sh      | 15 ++++++++++++---
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/bin/export-vrml.py b/bin/export-vrml.py
index 5f7993f..d5233ae 100644
--- a/bin/export-vrml.py
+++ b/bin/export-vrml.py
@@ -4,6 +4,7 @@ from pcbnewTransition.pcbnew import LoadBoard, EXPORTER_VRML
 b = LoadBoard(sys.argv[1])
 e = EXPORTER_VRML(b)
 
+# X and Y of Drill/Place origin are passed as parameters
 x = float(sys.argv[2])
 y = float(sys.argv[3])
 
diff --git a/bin/export.sh b/bin/export.sh
index 5d85637..0a1e5a0 100644
--- a/bin/export.sh
+++ b/bin/export.sh
@@ -5,25 +5,34 @@ if [ -z "$1" ]; then
   exit 1
 fi
 
+# Get path of script so we can call python scripts
 DIR=$(dirname $0)
 
 IN="$1"
 
+# Copy to backup so we can modify before exporting
 cp "$IN.kicad_pcb" "$IN.kicad_pcb.bak"
 
+# Export PDF from schematic
 kicad-cli sch export pdf "$IN.kicad_sch" --no-background-color -o "gerber/$IN.pdf"
+
+# Export netlist from schematic, then run BOM plugin script on it.
 kicad-cli sch export netlist "$IN.kicad_sch" --format kicadxml -o "gerber/$IN.net"
 python "$DIR/../kicad/hellen-one-kicad-bom-plugin.py" "gerber/$IN.net" "gerber/$IN.csv"
 
-X=$(grep "aux_axis_origin" "$IN.kicad_pcb" | tr -s ' ' | cut -d ' ' -f 3)
-Y=$(grep "aux_axis_origin" "$IN.kicad_pcb" | tr -s ' ' | cut -d ' ' -f 4 | tr -d ')')
-
+# Fill zones
 python "$DIR/fill-zones.py" "$IN.kicad_pcb"
 
+# Export Gerbers, drill file, and positions file
 kicad-cli pcb export gerbers --disable-aperture-macros -l "F.Cu,B.Cu,F.Paste,B.Paste,F.SilkS,B.SilkS,F.Mask,B.Mask,Edge.Cuts,In2.Cu,In1.Cu" --no-x2 --use-drill-file-origin "$IN.kicad_pcb" -o gerber/
 kicad-cli pcb export drill --map-format ps --drill-origin plot --excellon-zeros-format suppressleading "$IN.kicad_pcb" -o gerber/
 kicad-cli pcb export pos --format csv --units mm --use-drill-file-origin --bottom-negate-x "$IN.kicad_pcb" -o "gerber/$IN-all-pos.csv"
 
+# Get Drill/Place origin from PCB
+X=$(grep "aux_axis_origin" "$IN.kicad_pcb" | tr -s ' ' | cut -d ' ' -f 3)
+Y=$(grep "aux_axis_origin" "$IN.kicad_pcb" | tr -s ' ' | cut -d ' ' -f 4 | tr -d ')')
+# Export VRML
 python "$DIR/export-vrml.py" "$IN.kicad_pcb" "$X" "$Y" "gerber/$IN.wrl"
 
+# Restore backup
 cp "$IN.kicad_pcb.bak" "$IN.kicad_pcb"