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

implement mp.Connection with async streams #1640

Merged
merged 5 commits into from
May 7, 2024
Merged

Conversation

technillogue
Copy link
Contributor

@technillogue technillogue commented May 1, 2024

Yet Another Step towards merging #1530 into #1512. this drops that stupid race() function and the threaded mp.Connection setup, because those were found to be somewhat slow when profiling and it's generally simpler and less deadlock-prone to do it this way. we instead just implement most of the same stuff as mp.Connection, including a test that it matches the same binary format implemented by mp.Connection

@technillogue technillogue changed the base branch from main to async May 1, 2024 06:03
Signed-off-by: technillogue <technillogue@gmail.com>
Signed-off-by: technillogue <technillogue@gmail.com>
Signed-off-by: technillogue <technillogue@gmail.com>
@technillogue technillogue requested a review from a team May 2, 2024 22:39
Signed-off-by: technillogue <technillogue@gmail.com>
@technillogue technillogue marked this pull request as ready for review May 2, 2024 22:41
Signed-off-by: technillogue <technillogue@gmail.com>
@technillogue technillogue merged commit bb01c85 into async May 7, 2024
9 checks passed
@technillogue technillogue deleted the syl/aioconnection branch May 7, 2024 16:40
technillogue added a commit that referenced this pull request May 8, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request May 8, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request May 8, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jul 18, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
mattt pushed a commit that referenced this pull request Jul 18, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@gmail.com>
mattt pushed a commit that referenced this pull request Jul 19, 2024
* wip

* some tweaks

* ignore some type errors

* test connection roundtrip

* add notes from python source code

Signed-off-by: technillogue <technillogue@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