Skip to content

Add global variable to Scheduler on compiled state#2

Merged
jrbourbeau merged 2 commits intojrbourbeau:check-cythonizedfrom
jakirkham:add_cy_var_sched
Apr 28, 2021
Merged

Add global variable to Scheduler on compiled state#2
jrbourbeau merged 2 commits intojrbourbeau:check-cythonizedfrom
jakirkham:add_cy_var_sched

Conversation

@jakirkham
Copy link

No description provided.

This should make it easy to tell whether the scheduler was compiled with
Cython or not.
Copy link
Owner

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, this is much cleaner

@jrbourbeau jrbourbeau merged commit 8e2be5d into jrbourbeau:check-cythonized Apr 28, 2021
@jakirkham jakirkham deleted the add_cy_var_sched branch April 28, 2021 20:21
jrbourbeau added a commit that referenced this pull request May 3, 2021
* Check when scheduler is cythonized in CI

* Update cythonize step

* Update test

* Run full test suite again

* Add global variable to Scheduler on compiled state (#2)

* Add `COMPILED` global variable to scheduler

This should make it easy to tell whether the scheduler was compiled with
Cython or not.

* Simplify Cythonized Scheduler check

* lint

* Add skips to distributed/cli/tests/test_dask_spec.py

* xfail tests

* Move cythonized assert into test setup

* Remove unused CYTHONIZED env var

Co-authored-by: jakirkham <jakirkham@gmail.com>
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