Skip to content

Shuffle p2p #8813

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Shuffle p2p #8813

wants to merge 7 commits into from

Conversation

phofl
Copy link
Collaborator

@phofl phofl commented Aug 2, 2024

Closes #xxxx

  • Tests added / passed
  • Passes pre-commit run --all-files

@phofl phofl requested a review from fjetter as a code owner August 2, 2024 09:05
@phofl phofl marked this pull request as draft August 2, 2024 09:05
Copy link
Contributor

github-actions bot commented Aug 2, 2024

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    29 files  ±    0      29 suites  ±0   11h 24m 38s ⏱️ - 31m 24s
 4 100 tests  -     2   3 985 ✅  -     3    111 💤 ±  0  3 ❌ ±0  1 🔥 +1 
52 229 runs   - 3 264  50 092 ✅  - 2 966  2 131 💤  - 300  3 ❌  - 1  3 🔥 +3 

For more details on these failures and errors, see this check.

Results for commit 938ec20. ± Comparison against base commit 92fc0e2.

This pull request removes 8 and adds 6 tests. Note that renamed tests count towards both.
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_scheduler
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_scheduler_report_args[False]
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_scheduler_report_args[report_args0]
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_workers[1]
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_workers[False]
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_workers[True]
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_workers_report_args[False]
distributed.diagnostics.tests.test_memray ‑ test_basic_integration_workers_report_args[report_args0]
distributed.shuffle.tests.test_shuffle_array
distributed.shuffle.tests.test_shuffle_array ‑ test_shuffle[indexer0-chunks0]
distributed.shuffle.tests.test_shuffle_array ‑ test_shuffle[indexer1-chunks1]
distributed.shuffle.tests.test_shuffle_array ‑ test_shuffle[indexer2-chunks2]
distributed.shuffle.tests.test_shuffle_array ‑ test_shuffle[indexer3-chunks3]
distributed.shuffle.tests.test_shuffle_array ‑ test_shuffle_larger_array

♻️ This comment has been updated with latest results.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant