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

Node side RBS support #2464

Merged
merged 5 commits into from
Nov 29, 2024
Merged

Conversation

maqi
Copy link
Member

@maqi maqi commented Nov 26, 2024

Description

The network density can be calculated via sampling or equation approach.
For the performance consideration, the equation calculated network_density will be used by node for:

  • define responsible range
  • define replicate candidates
    Meanwhile, the sampling calculated network_density can be used by client for:
  • define the range of candidates to quoting with and upload record to

This PR is mainly the node-side RBS support, which paves the way for RBS deployment entirely (which client to be involved as well) without requiring breaking change to the live network in furture.

Related Issue

Fixes #<issue_number> (if applicable).

Type of Change

Please mark the types of changes made in this pull request.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@maqi maqi force-pushed the use_sampling_for_range_calculation branch 15 times, most recently from 15fcb70 to 69b8b9c Compare November 28, 2024 20:21
@maqi maqi removed the DoNotMerge label Nov 29, 2024
@maqi maqi changed the title Use sampling for range calculation Node side RBS support Nov 29, 2024
@maqi maqi force-pushed the use_sampling_for_range_calculation branch from 69b8b9c to b1b1d2d Compare November 29, 2024 13:46
@maqi maqi enabled auto-merge November 29, 2024 14:33
@jacderida jacderida disabled auto-merge November 29, 2024 14:48
@jacderida jacderida merged commit c6c8d43 into maidsafe:main Nov 29, 2024
30 checks passed
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