Skip to content
Snippets Groups Projects

Interpolations cleanup

Merged David Schäfer requested to merge interpol-cleanup into develop

Addresses #399 (closed)

Merge request reports

Pipeline #160513 passed

Pipeline passed for 6755d666 on interpol-cleanup

Test coverage 76.00% (-1.00%) from 1 job
Approval is optional

Merged by David SchäferDavid Schäfer 2 years ago (Mar 30, 2023 6:57pm UTC)

Merge details

Pipeline #160514 passed

Pipeline passed for 8668f59e on develop

Test coverage 76.00% (-1.00%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Bert Palm
  • Bert Palm
  • Bert Palm
  • Bert Palm
    • Resolved by David Schäfer

      This is just a suggestion to further simplifiy the functionality-inventory and is not meant to block the MR. But if it is agreed upon it would make sence to include it right away. I thought, actually we could also get rid of the resampling function, by adding a func keyword to align and trigger the resample/alignment, by method=resample, func=sum for example. This makes sence imo, since the mechanics are structurally quite akin.

      Edited by Peter Lünenschloß
  • David Schäfer added 1 commit

    added 1 commit

    • e24e2966 - Apply 2 suggestion(s) to 1 file(s)

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading