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

simplify dask.array to numpy route #226

Merged
merged 4 commits into from Jun 14, 2015

Conversation

Projects
None yet
2 participants
@mrocklin
Member

mrocklin commented Jun 14, 2015

Fixes #225

@cpcloud

This comment has been minimized.

Member

cpcloud commented Jun 14, 2015

discover also needs to be changed to from_numpy(a.shape, a.dtype)

@mrocklin

This comment has been minimized.

Member

mrocklin commented Jun 14, 2015

done

@cpcloud

This comment has been minimized.

Member

cpcloud commented Jun 14, 2015

we're not installing from source with travis, can you remove the dask from the conda install line and add a pip install git+git://github.com/ContinuumIO/dask.git in there

@mrocklin

This comment has been minimized.

Member

mrocklin commented Jun 14, 2015

done

@cpcloud

This comment has been minimized.

Member

cpcloud commented Jun 14, 2015

thx

.travis.yml Outdated
@@ -37,6 +37,7 @@ install:
# Install various deps
- pip install sas7bdat
- pip install git://github.com/ContinuumIO/dask.git --upgrade

This comment has been minimized.

@cpcloud

cpcloud Jun 14, 2015

Member

sorry my original line was wrong, it needs a git+ before the git://

@mrocklin mrocklin force-pushed the mrocklin:dask-cleanup branch from 87f6e81 to 877f77a Jun 14, 2015

cpcloud added a commit that referenced this pull request Jun 14, 2015

Merge pull request #226 from mrocklin/dask-cleanup
simplify dask.array to numpy route

@cpcloud cpcloud merged commit 95e378a into blaze:master Jun 14, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cpcloud

This comment has been minimized.

Member

cpcloud commented Jun 14, 2015

thanks @mrocklin

@mrocklin mrocklin deleted the mrocklin:dask-cleanup branch Jun 14, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment