search

dolphin-acoustics-vip / artwarp

MATLAB program for automated categorisation of tonal animal sounds
https://github.com/dolphin-acoustics-vip/artwarp/wiki
GNU Lesser General Public License v3.0
6 stars 9 forks source link

Add changes by Michael Stanway (April 2023) #51

Open olexandr-konovalov opened 9 months ago

olexandr-konovalov commented 9 months ago

Attempt to do PR #50 properly. Submitted this PR using an archive provided to us by @vmjanik via MS Teams for the Dolphin Acoustics VIP.

olexandr-konovalov commented 9 months ago

This looks manageable - 6 files changed and not 55 like it was in #50!

However, we have failed test - needs to be investigated:

  Error occurred in network_test/network_test and it did not run to completion.
      ---------
      Error ID:
      ---------
      'MATLAB:TooManyInputs'
      --------------
      Error Details:
      --------------
      Error using NetworkFactory.new_network
      Too many input arguments.

      Error in network_test (line 6)
      network = NetworkFactory.new_network(contours);
olexandr-konovalov commented 8 months ago

Test failed because of

  ================================================================================
  Error occurred in network_test/network_test and it did not run to completion.
      ---------
      Error ID:
      ---------
      'MATLAB:TooManyInputs'
      --------------
      Error Details:
      --------------
      Error using NetworkFactory.new_network
      Too many input arguments.

      Error in network_test (line 6)
      network = NetworkFactory.new_network(contours);
  ================================================================================

Seems that some function changed the number of arguments, but the test was not adjusted.

olexandr-konovalov commented 8 months ago

I have rebased this, hoping that we can also see code coverage for this PR - however, the test terminates because of the error prior to uploading results to Codecov.

olexandr-konovalov commented 7 months ago

Now part of this was added in PR #64 by @tatedunbar which has been merged. After that Iv'e rebased this PR #51, so we can now see what's left. @tatedunbar please check if there is anything else what we need to salvage, and then make another PR if you want to backport more of Michael's changes. Thanks!