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.

Sign up for GitHub

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

Jump to bottom

swarm: change MaxPO #1593

Merged
merged 3 commits into from
Apr 26, 2021
Merged

swarm: change MaxPO #1593

merged 3 commits into from
Apr 26, 2021

Conversation

acud
Copy link
Member

@acud acud commented Apr 23, 2021 •
edited
Loading

Long overdue but finally there - changing MaxPO to 31 (in other words 2^32 neighborhoods)

fixes #1442

@acud acud requested review from zelig and metacertain and removed request for zelig April 23, 2021 13:52
@ldeffenb ldeffenb mentioned this pull request Apr 23, 2021
Closed
@acud acud self-assigned this Apr 23, 2021
@acud acud added the ready for review The PR is ready to be reviewed label Apr 23, 2021
@acud acud merged commit 473d025 into master Apr 26, 2021
@acud acud deleted the max-po branch April 26, 2021 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@metacertain metacertain metacertain approved these changes

@zelig zelig zelig approved these changes

Assignees

@acud acud

Labels
pull-request ready for review The PR is ready to be reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

maxPO too low
3 participants
@acud @zelig @metacertain