Skip to content
Snippets Groups Projects
  1. Jul 25, 2023
  2. Jul 11, 2023
    • Arin Wongprommoon's avatar
      fix(postproc): detrend process only removes columns with all NaNs · 71150227
      Arin Wongprommoon authored
      WHY IS THIS CHANGE NEEDED?:
      - detrend process overzealously removes columns if input dataframes
        have NaNs near the beginning or end.
      - this is because by default, df.dropna() uses how="any", i.e. it drops
        columns with at least one NaN
      
      HOW DOES THE CHANGE SOLVE THE PROBLEM?:
      - drops columns with at least one NaN is not the intention -- the
        intention is to remove column filled with NaNs created by rolling
        window operations.
      - so, specified the how argument
      
      WHAT SIDE EFFECTS DOES THIS CHANGE HAVE?:
      - i honestly don't know why i didn't catch it in the 1.5 years this
        process existed -- this seems so obvious.
      - this is a deprecated signal processing post-process (i use the
        butterworth filter instead), so i don't expect it to break things much
      
      EVIDENCE THAT COMMIT WORKS:
      71150227
  3. Jun 28, 2023
  4. Jun 27, 2023
  5. May 16, 2023
  6. Apr 04, 2023
  7. Apr 03, 2023
  8. Mar 23, 2023
  9. Mar 21, 2023
  10. Mar 20, 2023
  11. Mar 17, 2023
Loading