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

Dask order uses task_spec #8842

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

fjetter
Copy link
Member

@fjetter fjetter commented Aug 26, 2024

This removes the need to materialize and copy the dependencies to a dict.

Copy link
Contributor

github-actions bot commented Aug 26, 2024

Unit Test Results

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

    18 files   -      7      18 suites   - 7   6h 14m 39s ⏱️ - 4h 9m 38s
 4 128 tests ±     0   3 991 ✅  -     24    112 💤 +  2  24 ❌ +21  1 🔥 +1 
34 557 runs   - 13 111  32 921 ✅  - 12 626  1 588 💤  - 526  47 ❌ +40  1 🔥 +1 

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

Results for commit 82ff9fc. ± Comparison against base commit 9da5824.

This pull request skips 2 tests.
distributed.shuffle.tests.test_merge ‑ test_minimal_version
distributed.tests.test_client ‑ test_computation_object_code_not_available

♻️ This comment has been updated with latest results.

@fjetter fjetter force-pushed the dask_order_task_spec branch from 0ef6e89 to 82ff9fc Compare October 30, 2024 13:13
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.

1 participant