Update CI Action to run on PRs (#151)

* Update CI Action to run on PRs

* Add CI Badge to README.md

* Fix failing hybrid test

.github/workflows/ci.yml

@@ -1,6 +1,6 @@
 name: Testing
 
-on: [ push ]
+on: [ push, pull_request ]
 
 jobs:
   build:

README.md

@@ -1,5 +1,7 @@
 # Hybrid Optimization and Performance Platform
 
+![CI Tests](https://github.com/NREL/HOPP/actions/workflows/ci.yml/badge.svg)
+
 As part of NREL's [Hybrid Energy Systems Research](https://www.nrel.gov/wind/hybrid-energy-systems-research.html), this
 software assesses optimal designs for the deployment of utility-scale hybrid energy plants, particularly considering wind,
 solar and storage.

hybrid/layout/pv_design_utils.py

@@ -108,7 +108,7 @@ def size_electrical_parameters(
         inverter_power=inverter_power,
         )
 
-    if n_inputs_combiner:
+    if n_inputs_combiner is not None and n_inputs_inverter is not None:
         n_combiners = math.ceil(n_strings / n_inputs_combiner)
         # Ensure there are enough inverters for the number of combiner boxes
         n_inverters = max(n_inverters, math.ceil(n_combiners / n_inputs_inverter))

tests/hybrid/test_hybrid.py

@@ -266,7 +266,9 @@ def test_hybrid_detailed_pv_only(site):
             solar_only['pv']['tech_config']['cec_i_mp_ref'] \
             * solar_only['pv']['tech_config']['cec_v_mp_ref'] \
             * 1e-3,
-        inverter_power=solar_only['pv']['tech_config']['inv_snl_paco'] * 1e-3
+        inverter_power=solar_only['pv']['tech_config']['inv_snl_paco'] * 1e-3,
+        n_inputs_inverter=50,
+        n_inputs_combiner=32
     )
     assert n_strings == 13435
     assert n_combiners == 420