myokit 1.37.2__py3-none-any.whl → 1.37.3__py3-none-any.whl

myokit/_myokit_version.py

@@ -14,7 +14,7 @@ __release__ = True
 #    incompatibility
 #  - Changes to revision indicate bugfixes, tiny new features
 #  - There is no significance to odd/even numbers
-__version_tuple__ = 1, 37, 2
+__version_tuple__ = 1, 37, 3
 
 # String version of the version number
 __version__ = '.'.join([str(x) for x in __version_tuple__])

myokit/_unit.py

@@ -487,56 +487,55 @@ class Unit:
         return self.__rtruediv__(other)
 
     @staticmethod
-    def register(name, unit, quantifiable=False, output=False):
+    def register(name, unit, quantifiable=False,
+                 preferred_representation=False):
         """
-        Registers a unit name with the Unit class. Registered units will be
-        recognised by the parse() method.
+        Registers a unit name with the :class:`Unit` class so that it will be
+        recognised when parsing.
 
         Arguments:
 
         ``name``
-            The unit name. A variable will be created using this name.
+            The unit name. This must be a valid myokit name, i.e.
+            ``myokit.check_name(name)`` should not raise any errors.
         ``unit``
             A valid unit object
         ``quantifiable``
             ``True`` if this unit should be registered with the unit class as a
-            quantifiable unit. Typically this should only be done for the
-            unquantified symbol notation of SI or SI derived units. For example
-            m, g, Hz, N but not meter, kg, hertz or forthnight.
-        ``output``
-            ``True`` if this units name should be used to display this unit in.
-            This should be set for all common units (m, kg, nm, Hz) but not for
-            more obscure units (furlong, parsec). Having ``output`` set to
-            ``False`` will cause one-way behavior: Myokit will recognise the
-            unit name but never use it in output.
-            Setting this to ``True`` will also register the given name as a
-            preferred representation format.
-
-        """
-        if not isinstance(name, str):
-            raise TypeError('Given name must be a string.')
+            _quantifiable_ unit, i.e. a unit that can be preceded by an SI
+            prefix such as "m" or "G". Typically this should only be done
+            for the unquantified symbol notation of SI or SI derived units. For
+            example "m", "g", or "Hz" but not "km", "meter", or "bushel".
+        ``preferred_representation``
+            Set to ``True`` to also register ``name`` as the preferred
+            representation for ``unit``.
+
+        """
+        name = myokit.check_name(name)
         if not isinstance(unit, Unit):
             raise TypeError('Given unit must be myokit.Unit')
         Unit._units[name] = unit
         if quantifiable:
             # Overwrite existing entries without warning
             Unit._quantifiable.add(name)
-        if output:
+        if preferred_representation:
             # Overwrite existing entries without warning
             Unit._preferred_representations[unit] = name
 
     @staticmethod
     def register_preferred_representation(rep, unit):
         """
-        Registers a preferred representation for the given unit without
-        registering it as a new type. This method can be used to register
-        common representations such as "umol/L" and "km/h".
+        Sets the preferred representation for the a unit, in terms of already
+        defined units.
+
+        This method can be used to register common representations such as
+        "umol/L" and "km/h". The given representation should be parseable by
+        Myokit, with the result equalling ``unit``.
 
         Arguments:
 
         ``rep``
-            A string, containing the preferred name for this unit. This should
-            be something that Myokit can parse.
+            A string containing the preferred representation for this unit.
         ``unit``
             The unit to register a notation for.
 
@@ -545,9 +544,14 @@ class Unit:
         # Overwrite existing entries without warning
         if not isinstance(unit, myokit.Unit):
             raise ValueError(
-                'Second argument to register_preferred_representation must be'
-                ' a myokit.Unit')
-        Unit._preferred_representations[unit] = str(rep)
+                'The second argument to register_preferred_representation must'
+                ' be a myokit.Unit')
+        rep = str(rep)
+        if not myokit.parse_unit(rep) == unit:
+            raise ValueError(
+                f'The representation [{rep}] does not equal {unit._str(True)}'
+                ' when parsed.')
+        Unit._preferred_representations[unit] = rep
 
     def __repr__(self):
         """
@@ -616,7 +620,8 @@ class Unit:
 
     def __str__(self):
 
-        # Strategy 1: Try simple look-up (using float.eq comparison)
+        # Strategy 1: Try simple look-up (using hash based on _str() followed
+        # by check of exponents in __eq__ for comparison
         try:
             return '[' + Unit._preferred_representations[self] + ']'
         except KeyError:
@@ -629,8 +634,8 @@ class Unit:
                 rep = '[' + test + ']'
                 break
 
-        # Strategy 3: Try finding a representation for the exponent and adding
-        # a multiplier to that.
+        # Strategy 3: Try finding a representation for the exponent (without a
+        # multiplier) and adding a multiplier to that.
         if rep is None:
 
             # Because kilos are defined with a multiplier of 1000, the
@@ -640,17 +645,20 @@ class Unit:
             u = Unit(list(self._x), m)
             rep = Unit._preferred_representations.get(u, None)
 
-            # Add multiplier part
+            # Add multiplier part:
             if rep is not None:
-                m = myokit.float.cround(self._m - m)
-                m = 10**m
-                if m >= 1:
-                    m = myokit.float.cround(m)
-                if m < 1e6:
-                    m = str(m)
+                if '(' in rep:
+                    rep = None
                 else:
-                    m = '{:<1.0e}'.format(m)
-                rep = '[' + rep + ' (' + m + ')]'
+                    m = myokit.float.cround(self._m - m)
+                    m = 10**m
+                    if m >= 1:
+                        m = myokit.float.cround(m)
+                    if m < 1e6:
+                        m = str(m)
+                    else:
+                        m = '{:<1.0e}'.format(m)
+                    rep = '[' + rep + ' (' + m + ')]'
 
         # Strategy 4: Build a new representation
         if rep is None:

myokit/tests/test_quantity.py

@@ -107,10 +107,10 @@ class QuantityTest(unittest.TestCase):
         # Test has does not change in quantity's lifetime
 
         try:
-            u = myokit.units.m**8
+            u = myokit.units.m / myokit.units.mol
             q1 = myokit.Quantity(1, u)
             h1 = hash(q1)
-            myokit.Unit.register_preferred_representation('abc', u)
+            myokit.Unit.register_preferred_representation('km/kmol', u)
             q2 = myokit.Quantity(1, u)
             h2 = hash(q2)
             self.assertEqual(h1, h2)

myokit/tests/test_unit.py

@@ -296,24 +296,35 @@ class MyokitUnitTest(unittest.TestCase):
         self.assertEqual(x.exponents(), [0, 2, 1.5, 0, 0, 0, -1.23])
 
     def test_register_errors(self):
-        # Test errors for Unit.register (rest is already used a lot).
-        self.assertRaises(TypeError, myokit.Unit.register, 4, myokit.Unit())
+        # Test errors for Unit.register (rest of the method is already used
+        # throughout other tests).
+        self.assertRaises(myokit.InvalidNameError, myokit.Unit.register, 4,
+                          myokit.Unit())
         self.assertRaises(TypeError, myokit.Unit.register, 'hi', 4)
 
     def test_register_preferred_representation(self):
         # Test new representations can be registered
 
-        u = myokit.units.m**8
-        self.assertEqual(str(u), '[m^8]')
+        u = myokit.units.K / myokit.units.A
         try:
-            myokit.Unit.register_preferred_representation(
-                'abc', myokit.units.m**8)
-            self.assertEqual(str(u), '[abc]')
+            # Register valid unit
+            self.assertEqual(str(u), '[K/A]')  # also auto registers!
+            myokit.Unit.register_preferred_representation('mK/mA', u)
+            self.assertEqual(str(u), '[mK/mA]')
 
+            # Attempt registering something that isn't a unit
             self.assertRaisesRegex(
                 ValueError, 'must be a myokit.Unit',
                 myokit.Unit.register_preferred_representation, 'x', 123)
 
+            # Attempt registering a representation that doesn't match the unit
+            self.assertRaisesRegex(
+                ValueError, r'does not equal \[K/A] when parsed',
+                myokit.Unit.register_preferred_representation, 'm', u)
+            self.assertRaisesRegex(
+                ValueError, r'does not equal \[K/A \(2\)] when parsed',
+                myokit.Unit.register_preferred_representation, 'K/A', u * 2)
+
         finally:
             # Bypassing the public API, this is bad test design!
             if u in myokit.Unit._preferred_representations:
@@ -365,6 +376,16 @@ class MyokitUnitTest(unittest.TestCase):
         self.assertEqual(str(c), '[V]')
         self.assertEqual(str(d), '[V]')
 
+        # 1115: Test str() doesn't create representations with more than one
+        # multiplier.
+        unit1 = myokit.units.mol / (1e3 * myokit.units.g)
+        unit2 = 1e-12 * myokit.units.mol / (1e3 * myokit.units.g)
+        unit3 = 1e-6 * unit2
+        self.assertEqual(str(unit1), '[mol/g (0.001)]')
+        self.assertEqual(str(unit2), '[mol/g (1e-15)]')
+        # Before fixing 1115, this returned [mol/g (0.001) (1e-12)]
+        self.assertEqual(str(unit3), '[mol/g (1e-21)]')
+
     def test_repr(self):
         # Test :meth:`Unit.repr()`.
 

{myokit-1.37.2.dist-info → myokit-1.37.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: myokit
-Version: 1.37.2
+Version: 1.37.3
 Summary: A modeling and simulation tool for cardiac cellular electrophysiology
 Home-page: http://myokit.org
 Author: Michael Clerx

{myokit-1.37.2.dist-info → myokit-1.37.3.dist-info}/RECORD

@@ -8,12 +8,12 @@ myokit/_err.py,sha256=n8Ggy2VKnyx_h_ySuNe77tIFXzh1yO7sWBvc7YPC2cM,11974
 myokit/_expressions.py,sha256=VpxJAax0OHAWj0AnIH5xkGVWLrmVxBP9tykaccV86BU,109246
 myokit/_io.py,sha256=2ll5Yb-sbP4tvc3pPmzhNuv_PaRMPJlNRWb1TzXfELg,8948
 myokit/_model_api.py,sha256=LRHT-RcqyB2UTNrvg7iMXN0D_lpParrks5YKcVZPJJY,194650
-myokit/_myokit_version.py,sha256=UpACRw-23yvpgf0yY2FrJL7h9kjhnS9ubxpUYQRLwnE,726
+myokit/_myokit_version.py,sha256=DpsDatFLxBQC5tRXhL6D6VFWDyhKYOxR0wnzl8CvAT4,726
 myokit/_parsing.py,sha256=GTVUwJvqjkQ3rc6BiwMFnzanLivbSUzW796gHCTJrNg,74065
 myokit/_progress.py,sha256=Mi2QU6Vqj4qudhXc76amXM31iID2Mo8Gljw5OH2COZ4,4410
 myokit/_protocol.py,sha256=_N16LGHppIv4yCSPjy0q2f-Oed0yG1wMk0ipTtXnMck,30756
 myokit/_system.py,sha256=IX3Q7_7f9X9aYsxSL2CXJc9bTcFJsJbfq_e2w-OkhOI,4602
-myokit/_unit.py,sha256=wbBT2UnFuxh1_d7rqvVLiL4xeYnjJ7ZORtqJcWS6dk4,29849
+myokit/_unit.py,sha256=prEupTNDeoqf2suN_i9ffBDGfVkid49gUDBTWTVzTww,30102
 myokit/float.py,sha256=7-BqdABN5Vm4HIgRMHSgcZ4O4Vd085-5hMB6z8zJZtg,3066
 myokit/pacing.py,sha256=_My0GwIPJh4DuUiK_kYDNBcacUaOKsffjbK5LvgK6z8,4552
 myokit/pype.py,sha256=JTKuoiwxGui6iqBXK4gxNlCdtH_IGY0FiZLtmaJ580c,8230
@@ -268,7 +268,7 @@ myokit/tests/test_protocol.py,sha256=DThz7WImRqUngx91GzVX5niJVhAUy7Z3eOKXNBChLlY
 myokit/tests/test_protocol_floating_point.py,sha256=R7It9eIHn1l3T-xHuJNaJw-hnIFOl1TXpmh_xZ5VhCI,12565
 myokit/tests/test_protocol_time_series.py,sha256=ktBkmKNtonorKa--g3nSQZ3nga88IEn_-BHp8n9IflY,2318
 myokit/tests/test_pype.py,sha256=vf3mympasLZwgTUQTSoe8e98Hp31cXo71APjlZEANKE,2402
-myokit/tests/test_quantity.py,sha256=bubdRZPCcQwogvVerADK8a9sqQyAWA6uAlgedT91Eiw,7353
+myokit/tests/test_quantity.py,sha256=etmbqlqBNt2OZFku_P-s5n8RlA-gxVyRAWjOPq9xD7s,7373
 myokit/tests/test_rhs_benchmarker.py,sha256=Gb3LaAiu8SW__BBM3l0TbMzxML1PpEqm3g8TCteyCZg,3452
 myokit/tests/test_sbml_api.py,sha256=9SIwsAT5lyGYnVZvPjW-EOd6mdIbrGMFpB5e981vAxM,74820
 myokit/tests/test_sbml_export.py,sha256=qdhSZ_zht_yOLJ3OfY2yt1-5uwj48nWn2Xtn5nSH8mo,13351
@@ -284,7 +284,7 @@ myokit/tests/test_simulation_opencl_vs_cvode.py,sha256=5b-dlUCwh7LieRoAb44OgxvwQ
 myokit/tests/test_simulation_opencl_vs_sim1d.py,sha256=QYXZZay_YiHZ1qPO0naGP4U1YtSS0yR1QtEv76GBah0,5216
 myokit/tests/test_system_info.py,sha256=H4vcHTc0HCfVd0mUfVgQDUgwNIwMa9g43rBapGR6NAs,562
 myokit/tests/test_tools.py,sha256=1-Gn7MGrXcINZa1hcgtVevcPYkaksx_bgzgSwwYRNfE,10358
-myokit/tests/test_unit.py,sha256=ZvDCQsI-xHx73DB0MdCv_XY8aLPNJjVOC1DtR0H_arc,13635
+myokit/tests/test_unit.py,sha256=JcfEHnz0fTLm0Bq7RamkxpvjynAE_s4c1Sasfabdesw,14752
 myokit/tests/test_user_functions.py,sha256=r7Xc3eJv-SIELMEGxx77Od2c_eBii5gjTvqarQvgteU,1721
 myokit/tests/test_variable.py,sha256=siV8OKkPpg0QPXeM_FaeQ_aa2dmLGppFCD-PUDacUEY,33331
 myokit/tests/data/beeler-1977-model-compare-a.mmt,sha256=Erkz4rr7irVeOPJRGvDJ-9c0iD1125xq9lLeQ6CwmK8,3022
@@ -401,9 +401,9 @@ myokit/tests/data/multi/beeler-no-name.mmt,sha256=tBeWcTVag1gAQAvxWlUqH6GQhF1D4D
 myokit/tests/data/multi/lr-1991.mmt,sha256=9jzHRAy1nLiBOPioiEpXmsRvIqwEezRY_ve2eHMDbTQ,6013
 myokit/tests/data/multi/not-a-model.csv,sha256=7ek3pQXr2fpjNjFTs-B-T_kEehx8IQ-evdcg24dtMEs,109
 myokit/tests/data/multi/subdir/beeler-no-name.mmt,sha256=tBeWcTVag1gAQAvxWlUqH6GQhF1D4DMnlmuePQn1vBY,3008
-myokit-1.37.2.dist-info/LICENSE.txt,sha256=0WNshB_he7HnJ4ROvVAaX95yz9760DE9ps8PbyvrGvw,1833
-myokit-1.37.2.dist-info/METADATA,sha256=UIVm2lR9c260M_8zaTgAK-jtfq3CRmNWkITLYUrf_ls,6273
-myokit-1.37.2.dist-info/WHEEL,sha256=GJ7t_kWBFywbagK5eo9IoUwLW6oyOeTKmQ-9iHFVNxQ,92
-myokit-1.37.2.dist-info/entry_points.txt,sha256=yP9wy3w0YAAYUD5PYGliYKhnKvEp5l6p4S_XvXsqreM,48
-myokit-1.37.2.dist-info/top_level.txt,sha256=vopnhGEticqud7tKy6L6dvi_n_AMXoiYBEiboRTnsWY,7
-myokit-1.37.2.dist-info/RECORD,,
+myokit-1.37.3.dist-info/LICENSE.txt,sha256=0WNshB_he7HnJ4ROvVAaX95yz9760DE9ps8PbyvrGvw,1833
+myokit-1.37.3.dist-info/METADATA,sha256=NocrKBZA4t1VCvp0KCNuoSQfAoRzngbbE4XEvtPlkXw,6273
+myokit-1.37.3.dist-info/WHEEL,sha256=GJ7t_kWBFywbagK5eo9IoUwLW6oyOeTKmQ-9iHFVNxQ,92
+myokit-1.37.3.dist-info/entry_points.txt,sha256=yP9wy3w0YAAYUD5PYGliYKhnKvEp5l6p4S_XvXsqreM,48
+myokit-1.37.3.dist-info/top_level.txt,sha256=vopnhGEticqud7tKy6L6dvi_n_AMXoiYBEiboRTnsWY,7
+myokit-1.37.3.dist-info/RECORD,,