Skip to content
Snippets Groups Projects

move eq_load test to right place

Merged David Verelst requested to merge eq_load_test into master

Any comments to this test?

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
  • I think it is a fine test.

    It would be even better if m and amplitude was not 1 in the second test. It could be two tests or

    for npeaks, ampl, m in [(10,1,1),(20,3,4)]:

    to avoid duplicate code

    but then you cannot compare the result of the two tests as you do in the end

  • You're right, it would be better to also test with values other than 1. I think I can figure out a way to do just that but without too much code duplication. I'll do that next week.

  • mentioned in commit 8003cc91

Please register or sign in to reply
Loading