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.

Sign up for GitHub

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

Jump to bottom

[CHIA-1552] Proposed set of drivers for multi-sig custody #18686

Draft
wants to merge 80 commits into
base: main
Choose a base branch
from
Draft

[CHIA-1552] Proposed set of drivers for multi-sig custody #18686

wants to merge 80 commits into from

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Oct 9, 2024 •
edited
Loading

description coming soon

@Quexington Quexington added clvm Added Required label for PR that categorizes merge commit message as "Added" for changelog labels Oct 9, 2024
@Chia-Network Chia-Network deleted a comment from github-actions bot Oct 10, 2024
@Chia-Network Chia-Network deleted a comment from coveralls-official bot Oct 10, 2024
@Quexington
[CHIA-1619] Introduce a 1 of N optimization for multi sig drivers (#1…
c546246 
…8690)

An optimization of inner puzzle construction when an M of N is involved
and M=1
@Quexington
Add optimization for NofN
745a45e
@Quexington
[CHIA-1622] Introduce an N of N optimization for multi sig drivers (#…
3705bfa 
…18694)

An optimization of inner puzzle construction when an M of N is involved
and M=N
@Quexington
Add support for PuzzleWithRestrictions with no MofN
b976541
@Quexington
Add support for PuzzleWithRestrictions with no MofN (#18709)
3db92f4 
A small bug where the API of member puzzles was not taken into account
when they are the only puzzle in the stack.
Quexington and others added 8 commits October 22, 2024 14:41
@Quexington
Add basic framework for delegated puzzle validators
8aa659c
@matt-o-how
code coverage for memo()
73667f2
@matt-o-how
don't reuse variable of different type
8ba7685
@Quexington
Add a BLSMember puzzle, a corresponding class, and a test (#18738)
805c41a 
### Purpose:

Adds the BLSMember puzzle which simply has a curried key and then
returns an `AGG_SIG_ME` of the delegated puzzle hash which is passed in
as a Truth.

### Testing Notes:

added a new test called `test_2_of_4_bls_members` in
`test_custody_architecture.py`
@Quexington
coverage
f4c12f8
@Quexington
Merge remote-tracking branch 'origin/quex.multi_sig_chialisp_drivers'…
6c80e86 
… into quex.basic_dpuz_validation_framework
@Quexington
pre-commit
e6e749d
@Quexington
Add basic framework for delegated puzzle validators (#18751)
7520fd6 
This adds a basic toolkit for writing restrictions for specific subtrees
in a `PuzzleWithRestrictions`. No actual restrictions are included,
those will come in a follow up PR.
@Quexington Quexington changed the title [CHIA-1522] Proposed set of drivers for multi-sig custody [CHIA-1552] Proposed set of drivers for multi-sig custody Oct 24, 2024
@github-actions GitHub Actions
Copy link
Contributor

File Coverage Missing Lines
chia/wallet/puzzles/custody/custody_architecture.py 97.6% lines 178, 273, 308, 323, 339
chia/wallet/puzzles/custody/member_puzzles/member_puzzles.py 97.6% lines 44
chia/wallet/puzzles/custody/restriction_puzzles/restrictions.py 94.7% lines 21
Total Missing Coverage
565 lines 7 lines 98%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Added Required label for PR that categorizes merge commit message as "Added" for changelog clvm coverage-diff
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Quexington @matt-o-how