fix tests and docstrings

greco-project · May 26, 2021 · be58676 · be58676
1 parent 5641840
commit be58676
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/pvcompare/pv_feedin.py b/pvcompare/pv_feedin.py
@@ -316,9 +316,9 @@ def set_up_system(technology, surface_azimuth, surface_tilt, add_sam_si_module=N
         surface azimuth of the module
     surface_tilt: float
         surface tilt of the module
-    add_sam_si_module: dict
-        with library (’CECMod’  or "SandiaMod") as key and module name as value.
-        E.g. {"cecmod":'Canadian_Solar_Inc__CS5P_220M'}
+        Dictionary with library (’CECMod’  or "SandiaMod") as key and module name as value.
+        E.g. {"cecmod":'Canadian_Solar_Inc__CS5P_220M'}.
+        Note that the SI module is only considered if there is the technology "SI" in
 
     Returns
     -------
@@ -413,8 +413,9 @@ def create_si_time_series(
         If True: Time series is normalized. Otherwise absolute time series is
         returned.
     add_sam_si_module: dict
-        with library (’CECMod’  or "SandiaMod") as key and module name as value.
-        E.g. {"cecmod":'Canadian_Solar_Inc__CS5P_220M'}
+        Dictionary with library (’CECMod’  or "SandiaMod") as key and module name as value.
+        E.g. {"cecmod":'Canadian_Solar_Inc__CS5P_220M'}.
+        Note that the SI module is only considered if there is the technology "SI" in
 
     Returns
     -------
@@ -613,7 +614,7 @@ def create_psi_time_series(
 
 
 def nominal_values_pv(
-    technology, area, surface_azimuth, surface_tilt, psi_type, add_sam_si_module
+    technology, area, surface_azimuth, surface_tilt, psi_type, add_sam_si_module=None
 ):
     """
     calculates the maximum installed capacity for each pv module.
@@ -633,6 +634,10 @@ def nominal_values_pv(
         surface azimuth of the modules
     surface_tilt: float
         surface tilt of the modules
+    add_sam_si_module: dict or None
+        Dictionary with library (’CECMod’  or "SandiaMod") as key and module name as value.
+        E.g. {"cecmod":'Canadian_Solar_Inc__CS5P_220M'}.
+        Note that the SI module is only considered if there is the technology "SI" in
 
     Returns
     -------

diff --git a/tests/test_main.py b/tests/test_main.py
@@ -39,7 +39,9 @@ def setup_class(self):
             os.path.dirname(__file__), "data_test_main/user_inputs/pvcompare_inputs/"
         )
         self.static_inputs_directory = constants.DEFAULT_STATIC_INPUTS_DIRECTORY
-        self.test_static_inputs_directory = "data/static_inputs/"
+        self.test_static_inputs_directory = os.path.join(
+            os.path.dirname(__file__), "data/static_inputs/"
+        )
         self.user_inputs_mvs_directory = os.path.join(
             os.path.dirname(__file__), "data_test_main/user_inputs/mvs_inputs/"
         )