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

Configure arc size clamping per agent+space #34

Open
mattyg opened this issue Nov 28, 2024 · 1 comment
Open

Configure arc size clamping per agent+space #34

mattyg opened this issue Nov 28, 2024 · 1 comment
Labels

Comments

@mattyg
Copy link
Member

mattyg commented Nov 28, 2024

AC:

  • When a space is instantiated by an agent they pass in a config
  • The config includes an arc size clamping setting
  • If multiple agents in the same space pass in conflicting arc size clamping config, the applied config is the union of those

The end-goal of this is to enable holochain to specify arc clamping per-cell so that clones of a cell can have different arc clamping config.

@mattyg mattyg added the enhancement New feature or request label Nov 28, 2024
@mattyg mattyg added this to Holochain Nov 28, 2024
@github-project-automation github-project-automation bot moved this to Backlog in Holochain Nov 28, 2024
@mattyg mattyg moved this from Backlog to Ready for refinement in Holochain Nov 28, 2024
@neonphog
Copy link
Collaborator

neonphog commented Dec 4, 2024

I don't know how this is going to shake out in the holochain integration, but luckily we can set that aside for a moment.

K2 is going to work a little differently in that everyone will start with a zero arc (legacy was supposed to evolve into this at some point too, it just never did). So "clamping" to a full arc would be incorrect and we will not implement that.

We can still clamp to zero arcs, or maybe we want to have some kind of enum on the LocalAgent for:

pub enum ArcStrategy {
  /// The arc sizing algorithm will try to move
  /// the arc size toward this value.
  Target(u32),
  
  /// The arc sizing algorithm will take into acount
  /// this node's desire to be this factor greater
  /// or less than the ideal arc size determined by the DHT
  Factor(f64),
}

The gossip arcs already are the union of all current arcs claimed by local agents within the same space, so we don't have to specify that explicitly again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Ready for refinement
Development

No branches or pull requests

2 participants