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.

Sign up for GitHub

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

Jump to bottom

fix vectorize bug #102

Closed
wants to merge 4 commits into from
Closed

fix vectorize bug #102

wants to merge 4 commits into from

Conversation

bradyrx
Copy link
Owner

@bradyrx bradyrx commented Nov 13, 2020

Description

Fixes bug when passing a single dimension dask array to apply_ufunc with vectorize=True. Should fix breaks in #99.

@luke-gregor
@aaronspring

Fixes #100

@bradyrx bradyrx mentioned this pull request Nov 13, 2020
Closed
@bradyrx
Copy link
Owner Author

bradyrx commented Nov 13, 2020

Tests passed.. just a linting issue.

@bradyrx
Copy link
Owner Author

bradyrx commented Nov 13, 2020

Deleted run_linter.sh for now since I'll be switching over to our Github Actions testing.

@bradyrx
Copy link
Owner Author

bradyrx commented Nov 13, 2020

I'm going to ignore the RTD error for now, since #103 could fix this stuff. Will merge all these updates into #99.

@bradyrx
Copy link
Owner Author

bradyrx commented Nov 13, 2020

Actually going to close this in favor of #103 since I need to fix this there for the GA to pass.

@bradyrx bradyrx closed this Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

stats.rm_poly tests fail when dask array passed
1 participant
@bradyrx