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 63 commits into
base: main
Choose a base branch
from
Draft

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

wants to merge 63 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.
matt-o-how and others added 25 commits October 21, 2024 16:24
@matt-o-how
start work on tests
406c0b5
@matt-o-how
fix all but signature in test
cf4077d
@matt-o-how
fix test
db265a9
@matt-o-how
black, isort, flake8
7784b83
@matt-o-how
migrate changes to new files / folders
62b2c16
@matt-o-how
mypy fixes
f0fbd5e
@matt-o-how
remove unused imports
bb392dc
@matt-o-how
add bls_member to deployed_puzzle_hashes
2093c73
@matt-o-how
no memo for BLS Member
d544e24
@matt-o-how
change comments on BLSMember
81a77c5
@matt-o-how
chialispp bls_member
80516a0
@matt-o-how
update test comment
0ab7039
@matt-o-how
black again
5613000
@matt-o-how
pylint disable
e21e3f8
@matt-o-how
add a more minimal test for blsmember
81b73aa
@matt-o-how
fix quex comments
b6e3519
@matt-o-how
flake8
b34ba20
@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

github-actions bot commented Nov 5, 2024

File Coverage Missing Lines
chia/wallet/puzzles/custody/custody_architecture.py 97.6% lines 178, 273, 308, 323, 339
Total Missing Coverage
465 lines 5 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