From b5c4660d5fbf2fd01b1216879f42d17f7c0ef0b0 Mon Sep 17 00:00:00 2001
From: Christophe Geuzaine <cgeuzaine@uliege.be>
Date: Mon, 16 Mar 2020 18:11:42 +0100
Subject: [PATCH] more comments

---
 Magnetodynamics/transfo.pro | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/Magnetodynamics/transfo.pro b/Magnetodynamics/transfo.pro
index f0fb862..abeeed7 100644
--- a/Magnetodynamics/transfo.pro
+++ b/Magnetodynamics/transfo.pro
@@ -16,6 +16,9 @@
 Include "transfo_common.pro";
 
 DefineConstant[
+  // The "Massive" option is non-physical, but it's interesting to highlight the
+  // fact that both massive and stranded conductors can be handled in the same
+  // way as far as circuit-coupling is concerned
   ConductorType = {2, Choices{1 = "Massive", 2 = "Coil"}, Highlight "Blue",
     Name "Parameters/01Conductor type"}
   Freq = {1, Min 0, Max 1e3, Step 1,
@@ -63,8 +66,8 @@ Function {
   SignBranch[Coil_2_M] = -1;
 
   If(ConductorType == 2)
-    // Number of turns (same for PLUS and MINUS portions) (half values because
-    // half coils are defined)
+    // Number of turns in the coils (same for PLUS and MINUS portions) - half
+    // values because half coils are defined geometrically due to symmetry!
     Ns[Coil_1] = 100;
     Ns[Coil_2] = 200;
 
@@ -117,8 +120,8 @@ Group {
 
 Function {
   deg = Pi/180;
-  // Input RMS voltage (half of the voltage because of symmetry; half coils
-  // are defined)
+  // Input RMS voltage (half of the voltage because of symmetry; half coils are
+  // defined!)
   val_E_in = 1.;
   phase_E_in = 90 *deg; // Phase in radian (from phase in degree)
 
-- 
GitLab