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

Test generating script first pass #8

Merged
merged 9 commits into from
Nov 21, 2023
Merged

Test generating script first pass #8

merged 9 commits into from
Nov 21, 2023

Conversation

pacu
Copy link
Collaborator

@pacu pacu commented Aug 17, 2023

this buils atop #7

This is the first pass over the script that creates the SFoE dataset.

There are several caveats that I've found:

  • Zcashd can't restore its own seed phrases. The "miner wallet" can't be used deterministically.
  • z_sendmany can't send several amounts to the same address, and I haven't figured how to break out notes.
  • I'm using t funds from the regtest miner for simplicity. Once I figure how to break a coinbase into many smaller shielded coinbases I can start sending orchard outputs in many different transactions.
  • for the sake of development I've reduced the fillers to 100 for now. doing 100k takes a while.

@pacu pacu mentioned this pull request Aug 17, 2023
Closed
@pacu
[WIP] fix key on after_info
8ef1bb1
@pacu
[WIP] Script to generate the test case
b9ab16d 
`get_testcase_blocks.py` dumps blocks in darksidewalletd format from
an already initialized zcashd instance.
`generate_sfoe.py` loads the test scenario into a zcashd dockerized
instance
`get_testcase_treestates.py` dumps the treestates needed to sync this
test case from darksidewalletd.
@pacu
[wip] fix shield coinbase and make break balances work
8cd86ac
@pacu
delete vscode file
ba316f2
@pacu
Copy link
Collaborator Author

pacu commented Aug 23, 2023

Update: The concrete test case construction take a lot of time even for 100 blocks in total. (original idea is 100k blocks long).

Will spin up another branch to try another approach

@pacu pacu marked this pull request as draft August 23, 2023 00:11
@pacu pacu marked this pull request as ready for review November 21, 2023 14:11
@pacu pacu merged commit 82f49fd into sfoe-dataset Nov 21, 2023
@pacu pacu deleted the test_gen_script branch November 21, 2023 14:11
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@pacu