Add fixmes for remaining items to do

Author: markcampanelli

pvlib/pvsystem.py

@@ -3028,7 +3028,7 @@ def combine_loss_factors(index, *losses, fill_method='ffill'):
 def _negative_total_power(current, *args):
     """
     Compute negative of total power generated by devices in series at
-    specified current.
+    specified current. Designed for use by scipy.optimize.minimize.
     """
     return -np.sum(current * v_from_i(current, *args))
 
@@ -3043,6 +3043,8 @@ def max_power_point_mismatched(
     i_mp_ic=None,
 ):
     """
+    FIXME Replace this with proper docstring.
+
     Compute maximum power info for (possibly) mismatched set of devices in
     series. When using this serially on time-series data, passing i_mp_ic from
     previous step may speed up computation. Algorithm falls back to automated

pvlib/tests/test_pvsystem.py

@@ -2555,6 +2555,7 @@ def test_Array_temperature_missing_parameters(model, keys):
 @pytest.mark.parametrize(
     'inputs',
     [
+        # FIXME Need many more argument combinations in additional test cases.
         {
             "photocurrent": 6.2,
             "saturation_current": 1.0e-8,
@@ -2645,13 +2646,12 @@ def test_max_power_point_mismatched(inputs):
             nNsVth,
         )
 
+    # FIXME Replace this with test assertions.
     print(result)
 
 
 def test_max_power_point_mismatched_unsuccessful_solver(monkeypatch):
-    """
-    Test errored max power point computation where solver is unsuccessful.
-    """
+    """Test mismatched max power point computation where solver is unsuccessful."""
     photocurrent = 6.2
     saturation_current = 1.0e-8
     resistance_series = 0.0001
@@ -2684,4 +2684,5 @@ def minimize_monkeypatched(*_, **__):
             nNsVth,
         )
 
+    # FIXME Replace this with test assertions.
     print(e_info)