[WIP] Fix Cythonized Scheduler test failures#4767
Closed
[WIP] Fix Cythonized Scheduler test failures#4767
Conversation
cf26517 to
1fb113e
Compare
Member
Author
|
Was able to fix a couple of these without too much difficulty. Broke out in PR ( #4768 ). |
Closed
Member
Author
|
One avenue to addressing these remaining issues would be to continue with splitting apart |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reverts the tests skipped/xfailed in PR ( #4764 ) due to issues encountered with the Cythonized Scheduler. Doing this to fix those tests and get them passing in the Cythonized Scheduler case as well.
Note: This PR mainly exists for tracking purposes while we sort out the different tests failures.
black distributed/flake8 distributed/isort distributed