-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
asyncio.Queue.task_done
documentation is ambiguous
#124433
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
Comments
No, an I think this is a good indicator that the excerpt under |
asyncio.Queue.task_done
documentation is ambiguous
In here it looks like the asyncio.queue is intended to be a "high-level API" to "distribute tasks" |
You can do that, but that isn't their sole purpose. |
"Task" here generally means "a thing to do", not " |
Yeah, that's why it's confusing in the context of |
Wouldn't hurt to say something on the subject in the docs then. |
Would you like to author the PR? (If not, I can deal with it.) |
Feel free. I don't know the subject well enough. |
) (cherry picked from commit 4322a31) Co-authored-by: Kumar Aditya <[email protected]>
) (cherry picked from commit 4322a31) Co-authored-by: Kumar Aditya <[email protected]>
…128672) gh-124433: fix docs for `asyncio.Queue.task_done` (GH-128669) (cherry picked from commit 4322a31) Co-authored-by: Kumar Aditya <[email protected]>
…128671) gh-124433: fix docs for `asyncio.Queue.task_done` (GH-128669) (cherry picked from commit 4322a31) Co-authored-by: Kumar Aditya <[email protected]>
Documentation
When reading https://docs.python.org/3/library/asyncio-queue.html the reader gets no clue to the fact that this object is (primarily) meant to contain tasks. (Because it is, isn't it? The task_done() method seems to indicate that)
Linked PRs
asyncio.Queue.task_done
#128669asyncio.Queue.task_done
(GH-128669) #128671asyncio.Queue.task_done
(GH-128669) #128672The text was updated successfully, but these errors were encountered: