Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Latest release does not match latest code version #107

Open
ythevalil-dd opened this issue Mar 29, 2023 · 0 comments
Open

Latest release does not match latest code version #107

ythevalil-dd opened this issue Mar 29, 2023 · 0 comments

Comments

@ythevalil-dd
Copy link

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):

  • OS: Ubuntu on Github Actions
  • Version Using python 3.8

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant