Mark (with TODO comments) part of functions to be reworked to use enums or booleans.

Author: PMeira

dss/dss_capi_gr/ICNData.py

@@ -60,7 +60,7 @@ def GMRac(self, Value):
 
     @property
     def GMRUnits(self):
-        return self._lib.CNData_Get_GMRUnits()
+        return self._lib.CNData_Get_GMRUnits() #TODO: use enum
 
     @GMRUnits.setter
     def GMRUnits(self, Value):
@@ -78,7 +78,7 @@ def Radius(self, Value):
 
     @property
     def RadiusUnits(self):
-        return self._lib.CNData_Get_RadiusUnits()
+        return self._lib.CNData_Get_RadiusUnits() #TODO: use enum
 
     @RadiusUnits.setter
     def RadiusUnits(self, Value):
@@ -87,7 +87,7 @@ def RadiusUnits(self, Value):
 
     @property
     def ResistanceUnits(self):
-        return self._lib.CNData_Get_ResistanceUnits()
+        return self._lib.CNData_Get_ResistanceUnits() #TODO: use enum
 
     @ResistanceUnits.setter
     def ResistanceUnits(self, Value):

dss/dss_capi_gr/ICktElement.py

@@ -234,7 +234,7 @@ def OCPDevIndex(self):
     @property
     def OCPDevType(self):
         '''(read-only) 0=None; 1=Fuse; 2=Recloser; 3=Relay;  Type of OCP controller device'''
-        return self._lib.CktElement_Get_OCPDevType()
+        return self._lib.CktElement_Get_OCPDevType() #TODO: use enum
 
     @property
     def PhaseLosses(self):

dss/dss_capi_gr/IGenerators.py

@@ -22,7 +22,7 @@ def ForcedON(self, Value):
     @property
     def Model(self):
         '''Generator Model'''
-        return self._lib.Generators_Get_Model()
+        return self._lib.Generators_Get_Model() #TODO: use enum
 
     @Model.setter
     def Model(self, Value):

dss/dss_capi_gr/ILineCodes.py

@@ -110,7 +110,7 @@ def Rmatrix(self, Value):
 
     @property
     def Units(self):
-        return self._lib.LineCodes_Get_Units()
+        return self._lib.LineCodes_Get_Units() # TODO: use enum
 
     @Units.setter
     def Units(self, Value):

dss/dss_capi_gr/ILineGeometries.py

@@ -87,7 +87,7 @@ def Cmatrix(self, Frequency, Length, Units):
     @property
     def Units(self):
         self._lib.LineGeometries_Get_Units_GR()
-        return self._get_int32_gr_array()
+        return self._get_int32_gr_array() #TODO: use enum
 
     @Units.setter
     def Units(self, Value):

dss/dss_capi_gr/ILineSpacings.py

@@ -32,7 +32,7 @@ def Nconds(self, Value):
 
     @property
     def Units(self):
-        return self._lib.LineSpacings_Get_Units()
+        return self._lib.LineSpacings_Get_Units() #TODO: use enum
 
     @Units.setter
     def Units(self, Value):

dss/dss_capi_gr/ILines.py

@@ -220,7 +220,7 @@ def TotalCust(self):
 
     @property
     def Units(self):
-        return self._lib.Lines_Get_Units()
+        return self._lib.Lines_Get_Units() #TODO: use enum
 
     @Units.setter
     def Units(self, Value):

dss/dss_capi_gr/ILoads.py

@@ -179,7 +179,7 @@ def Spectrum(self, Value):
     @property
     def Status(self):
         '''Response to load multipliers: Fixed (growth only), Exempt (no LD curve), Variable (all).'''
-        return self._lib.Loads_Get_Status()
+        return self._lib.Loads_Get_Status() #TODO: use enum
 
     @Status.setter
     def Status(self, Value):

dss/dss_capi_gr/IParallel.py

@@ -33,7 +33,7 @@ def ActiveParallel(self):
         (read) Sets ON/OFF (1/0) Parallel features of the Engine
         (write) Delivers if the Parallel features of the Engine are Active
         '''
-        return self._lib.Parallel_Get_ActiveParallel()
+        return self._lib.Parallel_Get_ActiveParallel()  #TODO: use boolean for consistency
 
     @ActiveParallel.setter
     def ActiveParallel(self, Value):
@@ -68,7 +68,7 @@ def ConcatenateReports(self):
         (read) Reads the values of the ConcatenateReports option (1=enabled, 0=disabled)
         (write) Enable/Disable (1/0) the ConcatenateReports option for extracting monitors data
         '''
-        return self._lib.Parallel_Get_ConcatenateReports()
+        return self._lib.Parallel_Get_ConcatenateReports() #TODO: use boolean for consistency
 
     @ConcatenateReports.setter
     def ConcatenateReports(self, Value):

dss/dss_capi_gr/IReactors.py

@@ -17,7 +17,7 @@ def SpecType(self):
         How the reactor data was provided: 1=kvar, 2=R+jX, 3=R and X matrices, 4=sym components.
         Depending on this value, only some properties are filled or make sense in the context.
         '''
-        return self._lib.Reactors_Get_SpecType()
+        return self._lib.Reactors_Get_SpecType() #TODO: use enum
 
     @property
     def IsDelta(self):