-
Notifications
You must be signed in to change notification settings - Fork 4
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
Tests fail on windows #33
Comments
All the tests are unrelated to pomegrante. Strange... these tests are pretty simple. Nothing that should depend on the operating system |
@MalteOlle Which Python version are you on? |
Seems to be reproducible: https://github.com/mad-lab-fau/gaitmap/actions/runs/5177786883/jobs/9328332843 |
Aah but luckily an easy fix. It has something todo that the default dtype for ints under windows is different. I will try to set the dtype explicit in thee cases. So you can ignore the failures for now. |
Fixed! Took some more work than expected, but should work now. I also created a follow-up issue #35, as during fixing that, I realised that we had at least one bug related to "unsafe" type casting from float to int and we should be more explicit about that by not allowing |
Steps to reproduce:
poetry install
pip install pomegranate==0.14.2
pip install statsmodels
poetry run poe test
Result:
The text was updated successfully, but these errors were encountered: