Skip to content

Commit

Permalink
removing _collapse_cleaning_events so half_norm_clean results are not…
Browse files Browse the repository at this point in the history
… affected
  • Loading branch information
nmoyer committed Aug 21, 2024
1 parent 5ef6c81 commit e66c295
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions rdtools/soiling.py
Original file line number Diff line number Diff line change
Expand Up @@ -438,7 +438,7 @@ def _calc_result_df(self, trim=False, max_relative_slope_error=500.0, max_negati
results.loc[filt, "run_slope"] = 0
results.loc[filt, "run_slope_low"] = 0
results.loc[filt, "run_slope_high"] = 0
results.loc[filt, "valid"] = False
# results.loc[filt, "valid"] = False

# Calculate the next inferred start loss from next valid interval
results["next_inferred_start_loss"] = np.clip(
Expand All @@ -465,10 +465,10 @@ def _calc_result_df(self, trim=False, max_relative_slope_error=500.0, max_negati
results.loc[results.clean_event, "inferred_begin_shift"] = np.clip(
results.inferred_begin_shift, 0, 1)
#######################################################################
'''

if neg_shift is False:
results.loc[filt, "valid"] = False
'''

if len(results[results.valid]) == 0:
raise NoValidIntervalError("No valid soiling intervals were found")
new_start = results.start.iloc[0]
Expand Down

0 comments on commit e66c295

Please sign in to comment.