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

Cloudpickle register by value #6466

Merged
merged 5 commits into from
Jun 28, 2022

Conversation

ian-r-rose
Copy link
Collaborator

Proposed fix for #6464

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

@github-actions
Copy link
Contributor

Unit Test Results

       15 files  ±0         15 suites  ±0   6h 26m 10s ⏱️ + 15m 29s
  2 820 tests +1    2 738 ✔️ ±0    81 💤 +1  1 ±0 
20 903 runs  +7  19 963 ✔️ +8  939 💤 ±0  1  - 1 

For more details on these failures, see this check.

Results for commit 8b82ca7. ± Comparison against base commit 1346671.

@ian-r-rose ian-r-rose force-pushed the cloudpickle-register-by-value branch from 8b82ca7 to 4e4ac45 Compare June 27, 2022 16:14
@github-actions
Copy link
Contributor

Unit Test Results

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

       15 files  +       3         15 suites  +3   10h 20m 22s ⏱️ + 3h 3m 3s
  2 898 tests +       1    2 811 ✔️ +     10    82 💤  -   12  5 +3 
21 465 runs  +4 917  20 502 ✔️ +4 763  958 💤 +151  5 +3 

For more details on these failures, see this check.

Results for commit 4e4ac45. ± Comparison against base commit a8eb3b2.

@fjetter fjetter merged commit 40c9420 into dask:main Jun 28, 2022
@ian-r-rose
Copy link
Collaborator Author

Thanks @fjetter !

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.

None yet

2 participants