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

aurel/hawk-main: Start HNSW MPC node #942

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

aurel/hawk-main: Start HNSW MPC node #942

wants to merge 2 commits into from

Conversation

naure
Copy link
Collaborator

@naure naure commented Jan 15, 2025

  • Create the main node program and a test of it.
  • This is mostly based on the test code starting at Aby3Store::shared_random_setup_with_grpc

Test with cargo test -p iris-mpc-cpu -- hawk_main --nocapture

Or:

cargo run -p iris-mpc-cpu --bin hawk_main -- --party-index=0 &
cargo run -p iris-mpc-cpu --bin hawk_main -- --party-index=1 &
cargo run -p iris-mpc-cpu --bin hawk_main -- --party-index=2

@naure naure requested review from bgillesp and iliailia January 15, 2025 14:23
let my_identity = identities[my_index].clone();
let my_address = addresses[my_index].clone();

println!("🦅 Starting Hawk node {my_index}");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! ;)

storage: iris_store,
owner: my_identity,
};
assert_eq!(aby3_store.get_owner_index(), my_index);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall it return Err perhaps?

// ---- Shared setup ----

let n_parties = 3;
let port_start = 40000;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if you add more async tests, then these ports will be congested. So I'd add addresses/ports to HawkArgs and use this func to generate them for tests.

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