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.

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

Murisi/draft with multi tx masp vp rebased2 #3274

Closed
wants to merge 263 commits into from

Conversation

murisi
Copy link
Contributor

@murisi murisi commented May 20, 2024

Describe your changes

Checking the merge of #3264 into #3213 .

Indicate on which release or other PRs this topic is based on

#3264
#3213

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

grarco and others added 26 commits May 16, 2024 11:50
* grarco/ferveo-cleanup:
  Updates wrapper tx constructor documentation
  Changelog #3241
  Cleans up ferveo-related things
* tomas/lint-dbg-and-prints:
  changelog: add #3257
  fix lint warnings
  add lints for dbg and prints (except for in tests)
* origin/brent/cleanup-dependencies:
  changelog: add #3218
  respond to comments
  upgrade tracing-log
  use updated fork of tiny-hderive
  use regex workspace
  de-duplicate itertools by upgrading itself and tonic-build
  upgrade ed25519-consensus
  replace dep `parse_duration` with `duration-str`
  update shlex
  update rustls
  update mio
  update libgit2-sys
  update h2
  update eyre
  remove unused dependencies
* origin/tiago/dont-export-wasm-mem:
  Changelog for #3258
  Remove unused WASM memory export
Namada 0.35.1 is a patch release that fixes a couple build issues with the last minor release.
* origin/yuji/tendermint-0.36:
  check event attribute parsing result
  fix test
  update masp
  add changelog
  for clippy
  ibc-rs 0.53
  bump to tendermint-rs 0.36
* origin/grarco/tx-batch:
  Restores `batch_size` arg for `fetch`
  Refactors `dispatch_tx` to return a result
  Refactors masp events to avoid dynamic attributes
  Short-circuit atomic batches at the first failure
  Implements a `get_tx_data` method on `Ctx`
  Refactors validity booleans into a `ValidityFlags` struct
  Refactors buckets in write log with iterator chains
  Misc refactors, fixes typo
  Improves unit tests for batches
  Refactors match on inner tx result
  Preserve txs order when constructing batch in the sdk
  Drops duplicated events for errored batch
  More unit tests for tx batching
  Appends inner tx result even in case of out of gas
  Better handling of batch errors
  Restores shielded sync struct in the sdk
  Reverts wrong shielded sync cli changes
  Removes broken batch section optimization
  Extracts ok tx result evaluation to a separate function
  Changes error code for batches. Fixes missing events
  Refactors `evaluate_tx_result`. Updates stats
  Renames `TxInfo` and fixes write log commit in `finalize_block`
  Refactors stats accounting
  Reverts `TryFrom` impls for ethereum data to `Tx`
  Changelog #1356
  Adds unit tests for tx batches
  Updates tx types comparisons and avoid duplicated signatures for batches
  Updates error message in ibc test
  Fixes bug in wrapper keys
  Fixes fee payemnt logic and unit tests
  Recomputes signatures for localnet genesis
  Clippy fix
  Removes unused `Ciphertext` section
  Updates `add_inner_tx` to avoid duplicated sections
  Adds new `mk_tx_batch` for unit testing
  Fixes tx interface to attach a new inner tx
  Updates comments
  Adds an SDK function to construct tx batches
  Renames `Commitments` and update docs
  Refactors the batch write log
  Refactors tx result handling in `finalize_block`
  Removes custom borsh impls for `BatchedTx` and `BatchedTxRef`
  More refactoring
  Misc refactoring
  Fixes benchmarks
  Fixes integration tests
  Fixes wasm tests
  Refactors test envs
  Fixes serialization of tx result and unit tests
  Rebuilds wasm for tests
  Recomputes genesis signatures. Fixes missing commitments in tests
  Updates `VpEval`
  Fixes wasm for test. Clippy + fmt
  Updates wasm interface and codes
  Renames batched txs structs
  Misc updates to write log. Updates tx result handling in `finalize_block`
  Updates `WriteLog` to support tx batches
  Improves handling of txs' results in `dispatch_tx` and `finalize_block`
  Introduces `OwnedBatchedTx` for benchmarks. Fixes benches and tests
  Generic `TxResult`
  Updates contexts for batched transactions. Misc updates to protocol
  Updates logs and tx result to support batching
  Adds `BatchedTx` struct and updated the native vps and transactions execution
  Updates core tx methods and masp client functions
  Adds multiple tx commitments in `Header`
* origin/grarco/masp-fees:
  Changelog #3217
  Assigns issues to TODOs
  Removes `wrapper_changed_keys` from `TxResult`
  Removes fee unshielding e2e test
  Removes fee unshielding integration tests
  Recomputes tx test fixture
  Recomputes wasm for tests modules
  Updates TODO comments
  Reworks sdk fee validation
  Renames `wrapper_fee_check`
  Removes error/functions related to fee unshielding
  Removes `descriptions_limit` protocol param
  Removes redundandt wrapper types
  Removes fee unshielding cli and tx args
  Removes `unshield_section_hash` from `WrapperTx`. Updates the relative functions. Removes event for masp wrapper tx
Copy link

codecov bot commented May 20, 2024

Codecov Report

Attention: Patch coverage is 53.57275% with 1254 lines in your changes are missing coverage. Please review.

Project coverage is 60.54%. Comparing base (4ed6229) to head (91f8621).

Current head 91f8621 differs from pull request most recent head dc6c44e

Please upload reports for the commit dc6c44e to get more accurate results.

Files Patch % Lines
crates/namada/src/ledger/native_vp/masp.rs 0.00% 408 Missing ⚠️
crates/core/src/uint.rs 1.60% 123 Missing ⚠️
crates/sdk/src/masp.rs 0.00% 96 Missing ⚠️
crates/namada/src/ledger/protocol/mod.rs 60.79% 69 Missing ⚠️
...s/apps/src/lib/node/ledger/shell/finalize_block.rs 87.02% 62 Missing ⚠️
crates/apps/src/lib/bench_utils.rs 0.00% 52 Missing ⚠️
crates/sdk/src/rpc.rs 0.00% 46 Missing ⚠️
crates/events/src/extend.rs 0.00% 33 Missing ⚠️
crates/namada/src/vm/host_env.rs 82.14% 30 Missing ⚠️
crates/apps/src/lib/config/genesis/chain.rs 12.90% 27 Missing ⚠️
... and 45 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3274      +/-   ##
==========================================
+ Coverage   60.24%   60.54%   +0.29%     
==========================================
  Files         303      301       -2     
  Lines       93191    93926     +735     
==========================================
+ Hits        56145    56867     +722     
- Misses      37046    37059      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@murisi murisi force-pushed the murisi/draft-with-multi-tx-masp-vp-rebased2 branch from 91f8621 to dc6c44e Compare May 21, 2024 13:09
@Fraccaman Fraccaman closed this May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants