refactor(katana-primitives): move class related types to a separate module #1560

kariy · 2024-02-19T17:21:41Z

No description provided.

kariy · 2024-02-19T17:21:52Z

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

codecov · 2024-02-19T17:32:51Z

Codecov Report

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

❗ No coverage uploaded for pull request base (dev/katana@66613e3). Click here to learn what that means.

❗ Current head ee94cdf differs from pull request most recent head 8999d82. Consider uploading reports for the commit 8999d82 to get more accurate results

Files	Patch %	Lines
crates/katana/executor/src/blockifier/state.rs	39.65%	35 Missing ⚠️
crates/katana/primitives/src/conversion/rpc.rs	88.88%	12 Missing ⚠️
crates/katana/storage/provider/src/lib.rs	0.00%	2 Missing ⚠️
crates/katana/primitives/src/transaction.rs	0.00%	1 Missing ⚠️
.../katana/storage/provider/src/providers/db/state.rs	85.71%	1 Missing ⚠️
...ana/storage/provider/src/providers/fork/backend.rs	0.00%	1 Missing ⚠️
.../katana/storage/provider/src/providers/fork/mod.rs	0.00%	1 Missing ⚠️
...na/storage/provider/src/providers/in_memory/mod.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             dev/katana    #1560   +/-   ##
=============================================
  Coverage              ?   69.95%           
=============================================
  Files                 ?      262           
  Lines                 ?    24971           
  Branches              ?        0           
=============================================
  Hits                  ?    17469           
  Misses                ?     7502           
  Partials              ?        0

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

kariy · 2024-02-29T06:56:15Z

Merge activity

Feb 29, 1:56 AM EST: @kariy started a stack merge that includes this pull request via Graphite.
Feb 29, 1:57 AM EST: Graphite rebased this pull request as part of a merge.
Feb 29, 1:58 AM EST: @kariy merged this pull request with Graphite.

…odule

…odule (#1560)

…odule (dojoengine#1560)

…odule (#1560)

* chore: bump dependencies * fix: update cairo-lang * fix: add missing run_profiler * fix: fmt * update cairo-vm rev * wip * wip * fmt * fix: ensure semantics analyzer plugin is registered * chore(katana): make ethers non optional since U256 is used outside messaging * fix: run CAIRO_FIX_TESTS * fix(dojo-core): remove compiler warnings * git: add sozo generated bindings to gitignore * fix: fix class hash * fix: class hash generated code * fix(dojo-core): fix tests and compilation warnings * chore: integrate snos on cairo2.5 * chore: update Cargo.lock * wip: work on integration of katana types * wip: fetch strategy * fix * feat: add parsing of state diff for genesis block * wip * wip * fix: rework provider for saya and RPC implem * wip * wip: snos types * wip txs * wip: fix snos parsing need to complete inputs * chore: switch back to git snos * chore: bump blockifier to use Serde on TransactionExecutionInfo * feat: add saya RPC namespace * feat: add transaction execution info to in memory db * fix: remove unused code covered by other test * fix: add logic for small requests to get transactions executions info * fix: fetch transactions executions info instead of running blockifier * fix: fmt and clippy * docs: fix unused type * fix: add tests for new transactions execution retrieval * fix: add correct test to check no pending support * fix: restore unchanged file * fix: rename new provider trait to TransactionTraceProvider * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * adjust new types for Katana blockchain and SNOS * fix: clippy * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * feat: add new primitive for trace and blockifier conversions * feat: add TransactionExecutionInfo convert from blockifier * feat: add executor trait method to retrieve the execution info * fix: ensure tx execution info is passed in block producer * chore: restore dependency on blockifier as we use TxExecInfo primitive * refactor(katana): decouple `katana-primitives` from any executor impls (#1637) * fix: clippy / fmt * merge * fix: clippy --------- Co-authored-by: Kariy <[email protected]> Co-authored-by: Ammar Arif <[email protected]>

…odule (#1560)

* chore: bump dependencies * fix: update cairo-lang * fix: add missing run_profiler * fix: fmt * update cairo-vm rev * wip * wip * fmt * fix: ensure semantics analyzer plugin is registered * chore(katana): make ethers non optional since U256 is used outside messaging * fix: run CAIRO_FIX_TESTS * fix(dojo-core): remove compiler warnings * git: add sozo generated bindings to gitignore * fix: fix class hash * fix: class hash generated code * fix(dojo-core): fix tests and compilation warnings * chore: integrate snos on cairo2.5 * chore: update Cargo.lock * wip: work on integration of katana types * wip: fetch strategy * fix * feat: add parsing of state diff for genesis block * wip * wip * fix: rework provider for saya and RPC implem * wip * wip: snos types * wip txs * wip: fix snos parsing need to complete inputs * chore: switch back to git snos * chore: bump blockifier to use Serde on TransactionExecutionInfo * feat: add saya RPC namespace * feat: add transaction execution info to in memory db * fix: remove unused code covered by other test * fix: add logic for small requests to get transactions executions info * fix: fetch transactions executions info instead of running blockifier * fix: fmt and clippy * docs: fix unused type * fix: add tests for new transactions execution retrieval * fix: add correct test to check no pending support * fix: restore unchanged file * fix: rename new provider trait to TransactionTraceProvider * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * adjust new types for Katana blockchain and SNOS * fix: clippy * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * feat: add new primitive for trace and blockifier conversions * feat: add TransactionExecutionInfo convert from blockifier * feat: add executor trait method to retrieve the execution info * fix: ensure tx execution info is passed in block producer * chore: restore dependency on blockifier as we use TxExecInfo primitive * refactor(katana): decouple `katana-primitives` from any executor impls (#1637) * fix: clippy / fmt * merge * fix: clippy --------- Co-authored-by: Kariy <[email protected]> Co-authored-by: Ammar Arif <[email protected]>

…namespace (#1697) * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * refactor(katana): decouple `katana-primitives` from any executor impls (#1637) * test(katana-executor): improve tests and add test for state (#1649) - add tests for each executor's CachedState - improve test in executor.rs - fix sir not enough gas error - fix test in simulate.rs * feat(saya): load data from Katana to run `SNOS` (#1535) * chore: bump dependencies * fix: update cairo-lang * fix: add missing run_profiler * fix: fmt * update cairo-vm rev * wip * wip * fmt * fix: ensure semantics analyzer plugin is registered * chore(katana): make ethers non optional since U256 is used outside messaging * fix: run CAIRO_FIX_TESTS * fix(dojo-core): remove compiler warnings * git: add sozo generated bindings to gitignore * fix: fix class hash * fix: class hash generated code * fix(dojo-core): fix tests and compilation warnings * chore: integrate snos on cairo2.5 * chore: update Cargo.lock * wip: work on integration of katana types * wip: fetch strategy * fix * feat: add parsing of state diff for genesis block * wip * wip * fix: rework provider for saya and RPC implem * wip * wip: snos types * wip txs * wip: fix snos parsing need to complete inputs * chore: switch back to git snos * chore: bump blockifier to use Serde on TransactionExecutionInfo * feat: add saya RPC namespace * feat: add transaction execution info to in memory db * fix: remove unused code covered by other test * fix: add logic for small requests to get transactions executions info * fix: fetch transactions executions info instead of running blockifier * fix: fmt and clippy * docs: fix unused type * fix: add tests for new transactions execution retrieval * fix: add correct test to check no pending support * fix: restore unchanged file * fix: rename new provider trait to TransactionTraceProvider * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * adjust new types for Katana blockchain and SNOS * fix: clippy * refactor(katana): change stored compiled class (#1559) Both `blockifier` and `starknet_in_rust` use their own proprietary contract class types. So need to have a type where the types from those two crates can be derived from. * refactor(katana-primitives): move class related types to a separate module (#1560) * Katana executor traits (#1561) * feat(katana-executor): add `blockifier` executor implementations (#1562) implement the executor traits introduced in #1561 based on `blockifier` * Refactor Katana to using the new executor traits (#1563) integrate the new executor traits introduced in #1561 to katana * feat(katana-executor): add starknet_in_rust executor (#1564) * doc(katana-executor): add README * test(katana-executor): add test for transaction simulation (#1593) * test(katana-executor): refactor simulate tests (#1606) refactor tests * feat: add new primitive for trace and blockifier conversions * feat: add TransactionExecutionInfo convert from blockifier * feat: add executor trait method to retrieve the execution info * fix: ensure tx execution info is passed in block producer * chore: restore dependency on blockifier as we use TxExecInfo primitive * refactor(katana): decouple `katana-primitives` from any executor impls (#1637) * fix: clippy / fmt * merge * fix: clippy --------- Co-authored-by: Kariy <[email protected]> Co-authored-by: Ammar Arif <[email protected]> * feat(katana-primitives): add fee info type * Refactor executor output * fix(katana): add missing contract address in `CallInfo` * Remove `native` feature for now * Limit nextest build jobs * refactor(katana-executor): add logs for transaction events and resources * fix(katana-executor): return reverted tx as error in estimate fee --------- Co-authored-by: glihm <[email protected]>

kariy mentioned this pull request Feb 19, 2024

refactor(katana): change stored compiled class #1559

Merged

This was referenced Feb 19, 2024

Katana executor traits #1561

Merged

feat(katana-executor): add blockifier executor implementations #1562

Merged

Refactor Katana to using the new executor traits #1563

Merged

feat(katana-executor): add starknet_in_rust executor #1564

Merged

glihm approved these changes Feb 19, 2024

View reviewed changes

kariy force-pushed the 02-19-refactor_katana-primitives_move_class_related_types_to_a_separate_module branch from c4a3f84 to ee94cdf Compare February 20, 2024 07:18

Base automatically changed from 02-19-refactor_katana_change_stored_compiled_class to dev/katana February 29, 2024 06:56

refactor(katana-primitives): move class related types to a separate m…

8999d82

…odule

kariy force-pushed the 02-19-refactor_katana-primitives_move_class_related_types_to_a_separate_module branch from ee94cdf to 8999d82 Compare February 29, 2024 06:56

kariy merged commit 2b29d23 into dev/katana Feb 29, 2024
10 checks passed

kariy deleted the 02-19-refactor_katana-primitives_move_class_related_types_to_a_separate_module branch February 29, 2024 06:58

kariy added a commit that referenced this pull request Feb 29, 2024

refactor(katana-primitives): move class related types to a separate m…

6a8564a

…odule (#1560)

kariy mentioned this pull request Mar 1, 2024

test(katana-executor): add test for transaction simulation #1593

Merged

kariy added a commit that referenced this pull request Mar 4, 2024

refactor(katana-primitives): move class related types to a separate m…

a03c768

…odule (#1560)

kariy added a commit to kariy/dojo that referenced this pull request Mar 4, 2024

refactor(katana-primitives): move class related types to a separate m…

73a49c8

…odule (dojoengine#1560)

kariy added a commit that referenced this pull request Mar 5, 2024

refactor(katana-primitives): move class related types to a separate m…

24ed717

…odule (#1560)

kariy added a commit that referenced this pull request Mar 7, 2024

refactor(katana-primitives): move class related types to a separate m…

d71c246

…odule (#1560)

kariy added a commit that referenced this pull request Mar 11, 2024

refactor(katana-primitives): move class related types to a separate m…

7233da7

…odule (#1560)

kariy added a commit that referenced this pull request Mar 18, 2024

refactor(katana-primitives): move class related types to a separate m…

337b654

…odule (#1560)

kariy mentioned this pull request Mar 25, 2024

Merge dev/katana branch #1697

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor(katana-primitives): move class related types to a separate module #1560

refactor(katana-primitives): move class related types to a separate module #1560

kariy commented Feb 19, 2024

kariy commented Feb 19, 2024 •

edited

Loading

codecov bot commented Feb 19, 2024 •

edited

Loading

kariy commented Feb 29, 2024 •

edited

Loading

refactor(katana-primitives): move class related types to a separate module #1560

refactor(katana-primitives): move class related types to a separate module #1560

Conversation

kariy commented Feb 19, 2024

kariy commented Feb 19, 2024 • edited Loading

codecov bot commented Feb 19, 2024 • edited Loading

Codecov Report

kariy commented Feb 29, 2024 • edited Loading

Merge activity

kariy commented Feb 19, 2024 •

edited

Loading

codecov bot commented Feb 19, 2024 •

edited

Loading

kariy commented Feb 29, 2024 •

edited

Loading