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

Update bag repartition docstring #5772

Merged
merged 1 commit into from Jan 8, 2020

Conversation

Timost
Copy link
Contributor

@Timost Timost commented Jan 8, 2020

repartition can be used both for reducing and increasing the number of partitions. The docstring only mentioned reducing the number of partitions

Fixes #5570

  • Tests added / passed
  • Passes black dask / flake8 dask

repartition can be used both for reducing and increasing the number of partitions. The docstring only mentioned reducing the number of partitions
@Timost Timost force-pushed the update-bag-repartition-docstring branch from 6b42461 to b415161 Compare January 8, 2020 12:51
@quasiben
Copy link
Member

quasiben commented Jan 8, 2020

Thanks for docstring updates @Timost . I see this is your first time contributing, welcome!

@quasiben quasiben merged commit 8530bdd into dask:master Jan 8, 2020
@Timost
Copy link
Contributor Author

Timost commented Jan 8, 2020

My pleasure ! Not sure why the CI failed the second time though.

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.

Bag repartition can increase, docstring implies otherwise
2 participants