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

Make hinted and optimistic AEC allocation configurable #4324

Closed

Conversation

gr0vity-dev
Copy link
Contributor

During the beta tests I noticed that modifying active_elections_hinted_limit_percentage in the config-node.toml isn't applied correctly.

This PR adds

  • active_elections_hinted_limit_percentage
  • active_elections_optimistic_limit_percentage
    to nodeconfig.cpp

- active_elections_hinted_limit_percentage
- active_elections_optimistic_limit_percentage
@pwojcikdev
Copy link
Contributor

I think we should make a separate config subcategory at [node.active_elections] with a dedicated active_elections_config class. Instead of prefixing new entries with active_elections_... put them in this config file for better long term organisation.

@gr0vity-dev
Copy link
Contributor Author

Closed in favor of #4604

@gr0vity-dev gr0vity-dev closed this May 6, 2024
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