matrix-profile-foundation / matrixprofile

A Python 3 library making time series data mining tasks, utilizing matrix profile algorithms, accessible to everyone.
https://matrixprofile.org
Apache License 2.0
363 stars 62 forks source link

Latest release does not match latest code version #107

Open ythevalil-dd opened 1 year ago

ythevalil-dd commented 1 year ago

Describe the bug The latest release of the library does not include the latest changes which fix syntax errors. As an example, the issue fixed at this line is not fixed in the latest release. This can cause issues in certain cases such as running unit tests with Github Actions. Pytest fails to run because it flags the syntax error on import and does not event run the tests. My issue happened while using tsfresh which uses matrixprofile as an optional dependency for certain functions to run.

To Reproduce Steps to reproduce the behavior:

  1. Run unit tests on a project using tsfresh and matrixprofile
  2. Error happens on import

Expected behavior No syntax error should be raised on import in any case.

Screenshots matrixprofile_error

Desktop (please complete the following information):

Additional context The problem does not always seem to occur when running tests locally but always happens in Github Actions. The fix seems pretty easy and quick since the code is already merged in main branch.