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

percentile_approx issue fix for python3.6 on Linux #386

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

JovanVeljanoski
Copy link
Member

@JovanVeljanoski JovanVeljanoski commented Aug 25, 2019

This PR is for the handling of the issue of percentile_approx method that pops up only on Linux and for python3.6.

Handles issue #385.

@JovanVeljanoski
Copy link
Member Author

I introduced yet another test via pytest for faster iterations.

@JovanVeljanoski
Copy link
Member Author

This is a most bizarre issue. While debugging, I noticed that putting print statements of certain variables alleviates the issue for python3.6 on Linux, but now Travis tells us that doing that creates a problem for Windows, again python3.6.

My guess this is either something to do with the @delay decorator, or with vaexfast.grid_find_edges.

I think I have narrowed down the issue there.. @maartenbreddels can you please take a quick look?

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

Successfully merging this pull request may close these issues.

2 participants