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

Optimize memory usage during broadcasts #1233

Merged
merged 1 commit into from
Sep 3, 2023

Conversation

miguelgrinberg
Copy link
Owner

No description provided.

@miguelgrinberg miguelgrinberg force-pushed the broadcast-optimizations branch 2 times, most recently from 179f46c to 5918e22 Compare September 1, 2023 22:27
@codecov-commenter
Copy link

codecov-commenter commented Sep 1, 2023

Codecov Report

Merging #1233 (7c15690) into main (f49d65a) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main     #1233   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines         1718      1740   +22     
  Branches       323       332    +9     
=========================================
+ Hits          1718      1740   +22     
Files Changed Coverage Δ
src/socketio/asyncio_manager.py 100.00% <100.00%> (ø)
src/socketio/asyncio_server.py 100.00% <100.00%> (ø)
src/socketio/base_manager.py 100.00% <100.00%> (ø)
src/socketio/server.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@miguelgrinberg miguelgrinberg force-pushed the broadcast-optimizations branch 2 times, most recently from f3129b0 to 79589f2 Compare September 3, 2023 15:23
@miguelgrinberg miguelgrinberg merged commit bf11ad3 into main Sep 3, 2023
20 checks passed
@miguelgrinberg miguelgrinberg deleted the broadcast-optimizations branch September 3, 2023 17:51
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.

None yet

2 participants