Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

Add smarter networking packet rate control#327

Open
BrendanAnnable wants to merge 13 commits into
masterfrom
annable/smarter-packet-rate
Open

Add smarter networking packet rate control#327
BrendanAnnable wants to merge 13 commits into
masterfrom
annable/smarter-packet-rate

Conversation

@BrendanAnnable

@BrendanAnnable BrendanAnnable commented Jul 6, 2020

Copy link
Copy Markdown
Member

Retrying #144

@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 6, 2020 13:47 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 6, 2020 13:50 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 6, 2020 23:02 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 6, 2020 23:09 Inactive

@TrentHouliston TrentHouliston left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we want to use rates, we probably don't care about latency

Comment thread src/server/nuclearnet/web_socket_proxy_nuclearnet_server.ts Outdated
Comment thread src/server/nuclearnet/web_socket_proxy_nuclearnet_server.ts Outdated
Comment thread src/server/nuclearnet/web_socket_proxy_nuclearnet_server.ts Outdated
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 6, 2020 23:39 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 6, 2020 23:41 Inactive
TrentHouliston
TrentHouliston previously approved these changes Jul 9, 2020

@TrentHouliston TrentHouliston left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 18, 2020 09:11 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 18, 2020 09:19 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-annable-smarter-gxus4o July 18, 2020 09:46 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants