chore(batcher): remove the duplicated papyrus_reader from native_blockifier

[Yael-Starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[Yael-Starkware]

• chore(batcher): remove the duplicated papyrus_reader from native_blockifier #1640 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Yael-Starkware and the rest of your teammates on Graphite

[github-actions]

Artifacts upload triggered. View details here

[Yael-Starkware]

changes this PRS contains:

1. papyrus_reader was deleted from native_blockifier (since it was previously duplicated to the batcher).
2. native_blockifier now uses papyrus_reader from the batcher.
3. papyrus_reader_test was split. It had 2 tests: one of them went to the batcher , and the other to py_block_executor_test - since it only used py_block_executor functionality.
4. large_compiled_contract.json moved to a new native_blockifier/resources directory.

Reviewable status: 0 of 10 files reviewed, all discussions resolved (waiting on @alonh5 and @noaov1)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.99%. Comparing base (e3165c4) to head (8b3e9bc).
Report is 290 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1640       +/-   ##
===========================================
+ Coverage   40.10%   76.99%   +36.89%     
===========================================
  Files          26      364      +338     
  Lines        1895    39076    +37181     
  Branches     1895    39076    +37181     
===========================================
+ Hits          760    30087    +29327     
- Misses       1100     6744     +5644     
- Partials       35     2245     +2210     

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

[noaov1]

Reviewed 10 of 10 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5 and @Yoni-Starkware)

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Artifacts upload triggered. View details here

[github-actions]

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [33.896 ms 33.947 ms 34.004 ms]
change: [-3.5763% -2.3104% -1.2792%] (p = 0.00 < 0.05)
Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
3 (3.00%) high mild
1 (1.00%) high severe

[Yoni-Starkware]

Reviewed 3 of 10 files at r1, 8 of 12 files at r2, 4 of 4 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)