You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm sure all the contributors here are across it, but just so the conda package doesn't get forgotten, it would be good for distributed to be as fast as possible and hence to be built with cython
Actually I'd recommend holding off on this for a bit. Am saying this for a few reasons:
A release is planned for next week and we only started doing this work this week
Only a relatively small amount of stuff has been Cythonized in master (more to come)
Outside of myself and some CI testing, not sure that it has seen broad enough usage yet
For these reasons, it seems reasonable to make this an opt-in thing for now as opposed to opt-out. Given there is a build flag, hopefully it's not too difficult for folks to opt-in.
That all being said, I would like to get more folks to try this and share feedback in the near future. Just not sure we are ready for conda-forge scale of folks trying this yet. Maybe in another release or so that will change. Hope that makes sense 🙂
I'm sure all the contributors here are across it, but just so the
conda
package doesn't get forgotten, it would be good fordistributed
to be as fast as possible and hence to be built withcython
xref: dask/distributed#4292
The text was updated successfully, but these errors were encountered: