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

README Introduction example text #96

Closed
wants to merge 1 commit into from

Conversation

jamesdbrock
Copy link

@jamesdbrock jamesdbrock commented Oct 13, 2023

Great package, thanks for maintaining it.

It seems to have no documentation though? Here's some slight documentation improvement.

@hosaka
Copy link
Collaborator

hosaka commented Oct 13, 2023

Related to #69

Hi @jamesdbrock, thanks for bringing this up. I haven't had time to address the lack of documentation. If you can expand a bit on your changes: perhaps add some basic examples to examples/ and add more structure to the readme, that would be great.

@jamesdbrock
Copy link
Author

If you can expand a bit on your changes: perhaps add some basic examples to examples/ and add more structure to the readme, that would be great.

I'd love to, but can we merge this PR first and then I'll add more PRs for more documentation improvements?

@hosaka
Copy link
Collaborator

hosaka commented Oct 17, 2023

I think the addition of "can be used with set_event_loop" is not quite enough to pass as documentation improvements. The QEventLoop can be used with a bunch of other asyncio methods too. Rather than giving a one liner, I think we're better off having some toy examples that demonstrate the use of async slots, async timers etc. right in the README.

If you find time to add more of these to this PR, please do so and I will review it again.

@jamesdbrock
Copy link
Author

Here's the README text introduced by this PR:


qasync allows coroutines to be used in PyQt/PySide applications by providing class QEventLoop, which is
an implementation of the PEP 3156 AbstractEventLoop and can be used in set_event_loop.

import asyncio
import qasync

asyncio.set_event_loop(qasync.QEventLoop())

@hosaka
Copy link
Collaborator

hosaka commented Nov 17, 2023

Superseded by #102

@hosaka hosaka closed this Nov 17, 2023
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