clean up comments

Author: dbnyu

qunit_test_all_math.html

@@ -61,6 +61,7 @@
 	    bb.endrings.calc_er_mutual_inductance(bb.legs);
 
       var TEST_TOL = 5e-5; // allowing 0.5 rounding error for 4-digit app outputs
+
       assert.close(1e9 * bb.legs.leg_self_inductance,       56.0517, TEST_TOL, 'Leg Self Inductance (rc = 12cm (LSI stays same))');
       assert.close(1e9 * bb.endrings.er_self_inductance,    64.7763, TEST_TOL, 'EndRing Self Inductance (rc = 12cm)');
       assert.close(1e9 * bb.legs.leg_mutual_inductance,     59.9066, TEST_TOL, 'Leg Mutual Inductance (rc = 12cm)'); // TODO unhardcode index?
@@ -75,7 +76,7 @@
 		  bb.endrings.set_endrings_rect(0.01);
 	    bb.endrings.calc_er_mutual_inductance(bb.legs);
 
-      var TEST_TOL = 5e-5; // allowing 0.5 rounding error for 4-digit app outputs
+
       assert.close(1e9 * bb.legs.leg_self_inductance,       56.0517, TEST_TOL, 'Leg Self Inductance (rc = 123cm (LSI stays same))');
       assert.close(1e9 * bb.endrings.er_self_inductance,    1113.6057, 10*TEST_TOL, 'EndRing Self Inductance (rc = 123cm) ADJUSTED TOLERANCE');
       assert.close(1e9 * bb.legs.leg_mutual_inductance,     56.5239, TEST_TOL, 'Leg Mutual Inductance (rc = 123cm)'); // TODO unhardcode index?
@@ -123,37 +124,20 @@
 
 
     // TODO repeat -  go through all combinations/defaults in app!
-
     // TODO some have 0 or 1 digits (after decimal)
-    //var PAPER_TEST_TOL = 5e-3; // paper results only have 2 digits after decimal
-    var PAPER_TEST_TOL = 5; // picoFarads
-    // TODO change Paper tests to assert.close.percent ? might make more sense
 
 
-//    QUnit.test('BirdcageBuilder 2002-A', function(assert) {
-//      /* Test against Table 1 results from 2002 Birdcage Builder paper 
-//       *
-//       * Chin, Collins, et al. MRM Vol 15(2) 156-163. 2002.
-//       */
-//
-//	    //           constructor(n,        f,     rc,     rs,   config) 
-//	    bb = new BirdcageBuilder(12, 21.25e6, 3.8e-2, 100e-2, 'lowpass');
-//      assert.step('NOTE - Custom endring arclength');
-//		  bb.legs.set_legs_rect(10e-2, 0.6e-2); // TODO assuming rectangular? 'a' superscript?
-//      bb.endrings.er_arclen = 1.98e-2;
-//		  bb.endrings.set_endrings_rect(0.6e-2);  // TODO - paper specifies endring SEGMENT LENGTH??
-//	    bb.endrings.calc_er_mutual_inductance(bb.legs);
-//
-//      // TODO - NOTE: the units are manually adjusted here, to reflect the app output values:
-//      assert.close(1e12 * bb.calc_capacitor(), 269.54, PAPER_TEST_TOL, 'Paper Capacitor A');
-//
-//      assert.verifySteps(['NOTE - Custom endring arclength']);
-//    });
 
 
-    //function test_2002_paper_cap(type, N, f, rc, rs, 
+    /*  Test against 2002 paper values */
+      
 
 
+    //var PAPER_TEST_TOL = 5e-3; // paper results only have 2 digits after decimal
+    var PAPER_TEST_TOL = 5; // picoFarads
+
+    // TODO change Paper tests to assert.close.percent ? might make more sense
+
     QUnit.test('BirdcageBuilder 2002 Paper Cap Tests', function(assert) {
       /* Test against Table 1 results from original 2002 Birdcage Builder paper 
        *
@@ -252,7 +236,6 @@
       assert.close(1e12 * bb.calc_capacitor(), 15.04, PAPER_TEST_TOL, 'Paper Capacitor H (Bandpass ER Caps)');
 
 
-
       // Coil I:
 	    //           constructor(n,      f,     rc,     rs,   config) 
 	    bb = new BirdcageBuilder(12, 170e6, 9.5e-2, 0.0, 'bandpass_leg');
@@ -290,6 +273,7 @@
           "NOTE - Custom endring arclength",
           "NOTE - Custom endring arclength"
           ], 'verify steps (debug notes)');
+
     });