Skip to content

Cloudpickle register by value#6466

Merged
fjetter merged 5 commits intodask:mainfrom
ian-r-rose:cloudpickle-register-by-value
Jun 28, 2022
Merged

Cloudpickle register by value#6466
fjetter merged 5 commits intodask:mainfrom
ian-r-rose:cloudpickle-register-by-value

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.

2 participants