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
Implement AxisPartition for Dask #281
Conversation
+ using dask.delayed + simple case works
(mostly copy of pandas_on_python.py)
…sPartition The latter fails the test_var test.
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great addition thanks!
@devin-petersohn thanks for bringing us on board and sorry for being silent for a while. It was a nice project for our weekend hackathon - the skeleton you provided helped a lot. There may be a bug fix that @pcahyna has not pushed yet (he will do it today evening, maybe for a separate MR). I also wanted to go through the code once more before making an upstream pull request -- you were just too efficient 😄 |
No problem! Further pull requests are welcome! I tested the code locally and it works great for a first pass. |
No description provided.