-
Notifications
You must be signed in to change notification settings - Fork 383
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
L1 Fee Curie: add unit test #1313
Conversation
…-circuits into curie_unittest
…-circuits into curie_unittest
@DreamWuGit can merge develop |
will fix new test failure |
@lispc ready now. |
.assign(region, 0, Value::known(F::from(is_curie)))?; | ||
let rlp_signed_len = if is_curie == 1 { tx_signed_length } else { 0 }; | ||
self.tx_signed_length | ||
.assign(region, 0, Value::known(F::from(rlp_signed_len)))?; |
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 think it is ok to still assign tx_signed_length even when is_curie == 0? Not used anyway
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
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.
in begin_tx
, this tx_signed_length
is looked up from tx table, so can not be any value eventually in real circuit even not is_curie.
Description
update
TxL1FeeGadgetTestContainer
to support Curie l1 fee unit testsIssue Link
[link issue here]
Type of change