Changeset 3219

Timestamp:

Nov 5, 2010 6:01:23 PM (10 years ago)

Author:

jwatson

Message:

Supressing a validation test with NumericConstant?. If the user specifies a value, we are (now) assuming it is actually a numeric value - otherwise, the domain check significantly inflates the run-time associated with expression tree creation. This needs to be revisited in the Coopr 2.5 re-write.

File:

• coopr.pyomo/trunk/coopr/pyomo/tests/unit/test_expr.py (modified) (1 diff)

coopr.pyomo/trunk/coopr/pyomo/tests/unit/test_expr.py

@@ -212 +212 @@
 
     def test_vals(self):
-        try:
-            NumericConstant(value='a')
-            self.fail("Cannot initialize a constant with a non-numeric value")
-        except ValueError:
-            pass
+        # the following aspect of this test is being removed due to the check seminatics
+        # of a numeric constant requiring far too much run-time, especially when involved
+        # in expression tree construction. if the user specifies a constant, we're assuming
+        # it is correct.
+#        try:
+#            NumericConstant(value='a')
+#            self.fail("Cannot initialize a constant with a non-numeric value")
+#        except ValueError:
+#            pass
         a = NumericConstant(value=1.1)
         b = float(a)