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 intomasterfrom
annable/smarter-packet-rate
Open

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

Conversation

@BrendanAnnable
Copy link
Member

@BrendanAnnable BrendanAnnable commented Jul 6, 2020

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
Copy link
Member

@TrentHouliston TrentHouliston left a comment

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

@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
Copy link
Member

@TrentHouliston TrentHouliston left a comment

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