Batch/Obj: tweak object setter type priority to allow batch.ObjRef = [object, ...]

Author: PMeira

altdss/Batch.py

@@ -529,7 +529,7 @@ def _set_batch_obj_prop(self, idx: int, other: Optional[Union[DSSObj, List[DSSOb
             self._check_for_error()
             return
 
-        if other is not None or isinstance(other, (bytes, str)) or (isinstance(other, LIST_LIKE) and len(other) and isinstance(other[0], (bytes, str))):
+        if (other is not None and isinstance(other, (bytes, str))) or (isinstance(other, LIST_LIKE) and len(other) and isinstance(other[0], (bytes, str))):
             self._set_batch_string(idx, other, flags)
             return
 

docs/changelog.md

@@ -11,7 +11,9 @@ relevant. See [DSS C-API's repository](https://github.com/dss-extensions/dss_cap
     - Implement `OCPDevice`
 
 - NonUniformBatch: allow `batch[idx]` to get a single element by index.
-- Setters: allow using `None` to clear object references (e.g. `altdss.Load[0].Daily = None`)
+- Setters: 
+    - Allow using `None` to clear object references (e.g. `altdss.Load[0].Daily = None`)
+    - Allow using list of live objects to fill a property that represents a DSS object.
 - ArrayProxy: 
     - Fix `BatchFloat64ArrayProxy.to_list()`. It was returning an array instead a list.
     - Accepts lists for in-place sub, e.g. `arrayProxy -= [1.1, 1.4]` now works as expected.

tests/test_obj.py

@@ -764,6 +764,42 @@ def test_no_new_attr():
         loads.non_existent_attribute_should_fail = True
 
 
+def test_obj_setter():
+    create_ref_ckt13(altdss)
+    altdss.Load.Daily = 'default'
+    default_shape = altdss.LoadShape['default']
+    assert all(l.Daily == default_shape for l in altdss.Load)
+
+    create_ref_ckt13(altdss)
+    altdss.Load.Daily_str = 'default'
+    default_shape = altdss.LoadShape['default']
+    assert all(l.Daily == default_shape for l in altdss.Load)
+
+    create_ref_ckt13(altdss)
+    default_shape = altdss.LoadShape['default']
+    altdss.Load.Daily = default_shape
+    assert all(l.Daily_str == 'default' for l in altdss.Load)
+    assert all(l.Daily == default_shape for l in altdss.Load)
+
+    create_ref_ckt13(altdss)
+    default_shape = altdss.LoadShape['default']
+    altdss.Load.Daily = ['default'] * len(altdss.Load)
+    assert all(l.Daily_str == 'default' for l in altdss.Load)
+    assert all(l.Daily == default_shape for l in altdss.Load)
+
+    create_ref_ckt13(altdss)
+    default_shape = altdss.LoadShape['default']
+    altdss.Load.Daily_str = ['default'] * len(altdss.Load)
+    assert all(l.Daily_str == 'default' for l in altdss.Load)
+    assert all(l.Daily == default_shape for l in altdss.Load)
+
+    create_ref_ckt13(altdss)
+    default_shape = altdss.LoadShape['default']
+    altdss.Load.Daily = [default_shape] * len(altdss.Load)
+    assert all(l.Daily_str == 'default' for l in altdss.Load)
+    assert all(l.Daily == default_shape for l in altdss.Load)
+
+
 if __name__ == '__main__':
     # Adjust for manual running a test-case
     test_register_values()