BUG: fixed time evaluation

Moved up the time evaluation for the forecast F10.7 data in the combine function. Also removed a duplicate definition of a date range variable.
pysat · Oct 2, 2024 · 13b0f9f · 13b0f9f
1 parent 2f164d6
commit 13b0f9f
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/pysatSpaceWeather/instruments/methods/f107.py b/pysatSpaceWeather/instruments/methods/f107.py
@@ -245,11 +245,10 @@ def combine_f107(standard_inst, forecast_inst, start=None, stop=None):
                     good_vals = np.array([
                         not is_fill_val(val, fill_val) for val
                         in forecast_inst['f107'][good_times]])
+                    new_times = list(forecast_inst.index[good_times][good_vals])
 
                 # Save desired data and cycle time
-                if len(good_vals) > 0:
-                    new_times = list(
-                        forecast_inst.index[good_times][good_vals])
+                if len(new_times) > 0:
                     f107_times.extend(new_times)
                     new_vals = list(
                         forecast_inst['f107'][good_times][good_vals])
@@ -274,8 +273,6 @@ def combine_f107(standard_inst, forecast_inst, start=None, stop=None):
     if len(f107_times) == 0:
         f107_times = date_range
 
-    date_range = pds.date_range(start=start, end=end_date, freq=freq)
-
     if date_range[0] < f107_times[0]:
         # Extend the time and value arrays from their beginning with fill
         # values