Skip to content
Snippets Groups Projects

refactor(core): Simplify detrend process

Merged Arin Wongprommoon requested to merge issue-021 into master

WHY IS THIS CHANGE NEEDED:

  • logical steps of computation unclear, therefore unclear if result of computation is what we want

HOW DOES THE CHANGE SOLVE THE PROBLEM:

  • replace moving_average() and lines 74-76 with pandas.DataFrame methods -- simplifies code; standardised routines
  • rename variables so they are more informative

EVIDENCE THAT COMMIT WORKS:

  • generate random DataFrame, original & new implementation produce same outputs

REFERENCES:

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading