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

Add Semaphore, Condition, Event #2007

Open
mrocklin opened this issue May 25, 2018 · 6 comments
Open

Add Semaphore, Condition, Event #2007

mrocklin opened this issue May 25, 2018 · 6 comments
Labels
good first issue Clearly described and easy to accomplish. Good for beginners to the project.

Comments

@mrocklin
Copy link
Member

Dask currently has some distributed coordination primitives like Locks and Queues that match the API of normal local Locks and Queues. There are other primitives like these that we do not yet support, notable Semaphore, Condition, and Event.

These might be good training for people who want to become familiar with extending the dask.distributed system. The files lock.py, queues.py, and variable.py might be a good model for development here.

@mrocklin mrocklin added the good first issue Clearly described and easy to accomplish. Good for beginners to the project. label Jul 10, 2018
@palash25
Copy link

Hi, I've been writing python for some time but I'm new to distributed computing. This seems like the right issue to get started. Can I work on this?

@mrocklin
Copy link
Member Author

mrocklin commented Jul 10, 2018 via email

@palash25
Copy link

palash25 commented Jul 10, 2018

Thanks will try to submit a PR soon

@andersy005
Copy link
Member

I am interested in working on the other features namely (Event, Condition), and wanted to make sure that I am not duplicating work that has already been done.

@mrocklin
Copy link
Member Author

It appears that #2690 is/was meant to implement one (Semaphore) of the missing features listed here. Are there any updates on the status of #2690?

All communication is happening on github. I recommend checking in on that PR.

#3397 mentions Events and Conditions. Is it the case that these two features have also been implemented already?

This is different. Dask uses Tornado/asyncio objects within its codebase. We would also like to expose distributed versions of these objects to users so that they can make their own applications.

@andersy005
Copy link
Member

All communication is happening on github. I recommend checking in on that PR.

👍 will do

This is different. Dask uses Tornado/asyncio objects within its codebase. We would also like to expose distributed versions of these objects to users so that they can make their own applications.

Thank you for the clarification!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Clearly described and easy to accomplish. Good for beginners to the project.
Projects
None yet
Development

No branches or pull requests

3 participants