Skip to content

feat(dkg): implement dkg/disk and dkg/share#285

Open
emlautarom1 wants to merge 43 commits intomainfrom
emlautarom1/dkg-disk-share
Open

feat(dkg): implement dkg/disk and dkg/share#285
emlautarom1 wants to merge 43 commits intomainfrom
emlautarom1/dkg-disk-share

Conversation

@emlautarom1
Copy link
Collaborator

Closes #252, Closes #257


This PR shows a limitation of the reference implementation module structure:

Until we figure out what is the best approach, the workaround is as follows:

  • Do not gate pluto_cluster::test_cluster behind the test flag
  • Disable clippy checks on this module (in particular due to the usage of .unwraps()).
  • Inline some methods from pluto_testutil to avoid pulling test dependencies.

With this, the only change to pluto_cluster's dependencies is swapping rand_core for rand which I believe is quite reasonable.

- Add `load_definition`
- Requires `shares`
- Adjust some pending TODOs.
- Fix `load_definition_file_does_not_exist` error
- Workaround to be used by dkg
- Impl `From<&Share> for ShareMsg`
- Make everything public
- Add docs to all exposed symbols
@varex83
Copy link
Collaborator

varex83 commented Mar 13, 2026

Until we figure out what is the best approach, the workaround is as follows

We can do it by so #[cfg(any(test, feature = "foo"))]

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.

Implement dkg/share Implement dkg/disk

2 participants