Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

REFACTOR-#4696: Merge dask and ray virtual partition code.#4759

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
mvashishtha wants to merge2 commits intomodin-project:master
base:master
Choose a base branch
Loading
frommvashishtha:4696-merge-dask-ray-virtual-partitions

Conversation

@mvashishtha
Copy link
Collaborator

What do these changes do?

REFACTOR-#4696: Merge dask and ray virtual partition code.

  • commit message follows format outlinedhere
  • passesflake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
  • passesblack --check modin/ asv_bench/benchmarks scripts/doc_checker.py
  • signed commit withgit commit -s
  • ResolvesREFACTOR: merge dask and ray virtual partition code #4696
  • tests added and passing
  • module layout described atdocs/development/architecture.rst is up-to-date
  • added (Issue Number: PR title (PR Number)) and github username to release notes for next major release

mvashishtha added2 commitsAugust 2, 2022 17:37
Signed-off-by: mvashishtha <mahesh@ponder.io>
@mvashishthamvashishtha requested a review froma team as acode ownerAugust 2, 2022 22:56
@lgtm-com
Copy link

lgtm-combot commentedAug 2, 2022

This pull requestintroduces 1 alert when merging3a73fab intoccda567 -view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

@mvashishthamvashishtha marked this pull request as draftAugust 2, 2022 23:34
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@devin-petersohndevin-petersohnAwaiting requested review from devin-petersohndevin-petersohn will be requested when the pull request is marked ready for reviewdevin-petersohn is a code owner

@RehanSDRehanSDAwaiting requested review from RehanSDRehanSD will be requested when the pull request is marked ready for reviewRehanSD is a code owner

@YarShevYarShevAwaiting requested review from YarShevYarShev will be requested when the pull request is marked ready for reviewYarShev is a code owner

@vnlitvinovvnlitvinovAwaiting requested review from vnlitvinovvnlitvinov will be requested when the pull request is marked ready for reviewvnlitvinov is a code owner

@anmyachevanmyachevAwaiting requested review from anmyachevanmyachev will be requested when the pull request is marked ready for reviewanmyachev is a code owner

@dchigarevdchigarevAwaiting requested review from dchigarevdchigarev will be requested when the pull request is marked ready for reviewdchigarev is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

REFACTOR: merge dask and ray virtual partition code

1 participant

@mvashishtha
[8]ページ先頭
©2009-2025 Movatter.jp