-
Notifications
You must be signed in to change notification settings - Fork 191
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
feat: native tmux-based network e2e #9036
Conversation
import subprocess | ||
import json | ||
import heapq | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bundled because useful
export VALIDATOR_DISABLED="true" | ||
export SEQ_MAX_SECONDS_BETWEEN_BLOCKS="0" | ||
export SEQ_MIN_TX_PER_BLOCK="1" | ||
export P2P_TCP_ANNOUNCE_ADDR="0.0.0.0:40400" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recall in the past i had issues where the announce addresses was not 127.0.0.1
. But I cannot recall why - writing this just incase
Changes to circuit sizes
🧾 Summary (100% most significant diffs)
Full diff report 👇
|
@@ -25,7 +25,7 @@ export class DeployProvenTx<TContract extends Contract = Contract> extends Prove | |||
*/ | |||
public override send(): DeploySentTx<TContract> { | |||
const promise = (() => { | |||
return this.wallet.sendTx(this); | |||
return this.wallet.sendTx(this.getPlainDataTx()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just checking that this was meant to be in this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is needed to fix serialization. Signed off by grego
For debugging