fix AeroDist ctor arg bug in README codes. Closes #393 (#394)

Author: slayoo

README.md

@@ -143,9 +143,7 @@ aero_dist = ppmc.AeroDist(
             "num_conc": 3200 / si.cm**3,
             "geom_mean_diam": 8.64 * si.nm,
             "log10_geom_std_dev": 0.28,
-        }
-    },
-    {
+        },
         "diesel": {
             "mass_frac": [{"OC": [0.3]}, {"BC": [0.7]}],
             "diam_type": "geometric",
@@ -187,9 +185,7 @@ aero_dist = ppmc.AeroDist(aero_data, (
       "num_conc" => 3200 / si.cm^3,
       "geom_mean_diam" => 8.64 * si.nm,
       "log10_geom_std_dev" => .28,
-    )
-  ),
-  Dict( 
+    ),
     "diesel" => Dict(
       "mass_frac" => (Dict("OC" => (.3,)), Dict("BC" => (.7,))),
       "diam_type" => "geometric",
@@ -198,7 +194,7 @@ aero_dist = ppmc.AeroDist(aero_data, (
       "geom_mean_diam" => 50 * si.nm,
       "log10_geom_std_dev" => .24,
     )
-  )
+  ),
 ))
 
 n_part = 100
@@ -232,9 +228,7 @@ aero_dist = ppmc.AeroDist(aero_data, py.tuple({ ...
       "num_conc", 3200 / si.cm^3, ...
       "geom_mean_diam", 8.64 * si.nm, ...
       "log10_geom_std_dev", .28 ...
-    )) ...
-  )), ...
-  py.dict(pyargs( ... 
+    )), ...
     "diesel", py.dict(pyargs( ...
       "mass_frac", py.tuple({ ...
         py.dict(pyargs("OC", py.tuple({.3}))), ...

src/aero_dist.hpp

@@ -49,8 +49,6 @@ struct AeroDist {
         ptr(f_aero_dist_ctor, f_aero_dist_dtor),
         aero_data(aero_data)
     {
-        if (!InputJSONResource::unique_keys(json))
-            throw std::runtime_error("Mode names must be unique");
         for (const auto &mode : json)
             AeroMode::check_mode_json(mode.begin().value());
 

tests/test_aero_dist.py

@@ -188,19 +188,6 @@ def test_get_source_names():
         for i, key in enumerate(keys):
             assert sources[i] == key  # pylint: disable=unsubscriptable-object
 
-    @staticmethod
-    @pytest.mark.skipif(platform.machine() == "arm64", reason="TODO #348")
-    def test_ctor_multimode_error_on_repeated_mode_names():
-        # arrange
-        aero_data = ppmc.AeroData(AERO_DATA_CTOR_ARG_MINIMAL)
-
-        # act
-        with pytest.raises(Exception) as exc_info:
-            ppmc.AeroDist(aero_data, AERO_DIST_CTOR_ARG_MINIMAL * 2)
-
-        # assert
-        assert str(exc_info.value) == "Mode names must be unique"
-
     @staticmethod
     @pytest.mark.skipif(platform.machine() == "arm64", reason="TODO #348")
     def test_ctor_error_on_repeated_massfrac_keys():