Skip to content

catchup: into main from long_lived/initial_datalayer @ cf34dfd8cd7d01f1f3e49df4173506c5ebbd716b - #1194

Closed
altendky wants to merge 469 commits into
mainfrom
catchup/main_from_long_lived_initial_datalayer_cf34dfd8cd7d01f1f3e49df4173506c5ebbd716b
Closed

catchup: into main from long_lived/initial_datalayer @ cf34dfd8cd7d01f1f3e49df4173506c5ebbd716b#1194
altendky wants to merge 469 commits into
mainfrom
catchup/main_from_long_lived_initial_datalayer_cf34dfd8cd7d01f1f3e49df4173506c5ebbd716b

Conversation

@altendky

@altendky altendky commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

Source hash: cf34dfd
Remaining commits: 0

…datalayer_from_main_ed05836d73c6e707b2b7d571f96d3b6ed603a27b

catchup: into long_lived/initial_datalayer from main  @ ed05836
…long_lived_initial_datalayer_from_main_0227c272b2c4dc890b4f668fdf568d69bc457c14
…datalayer_from_main_0227c272b2c4dc890b4f668fdf568d69bc457c14

catchup: into long_lived/initial_datalayer from main  @ 0227c27
make datalayer a submodule for the namespacing benefits
…s_datalayer

ban python `*` imports from `chia_rs.datalayer`
correct datalayer py-bindings settings for chia-protocol
this avoids blockchain tests directly accessing and thus copying `.key_to_index`
…long_lived_initial_datalayer_from_main_afbb1588ccf5be58b5fce098e30180c8e3e83cff
…datalayer_from_main_afbb1588ccf5be58b5fce098e30180c8e3e83cff

catchup: into long_lived/initial_datalayer from main  @ afbb158
…long_lived_initial_datalayer_from_main_7a103a64055471f2ca6ee860b06af2c8868a9190
altendky and others added 22 commits July 11, 2025 16:12
…long_lived_initial_datalayer_from_main_061bd6e192ceb90e3bb81ceff1fc69d674626eb7
…_datalayer_from_main_061bd6e192ceb90e3bb81ceff1fc69d674626eb7

catchup: into long_lived/initial_datalayer from main  @ 061bd6e
* Check root hash in get_missing_hashes.

* format.

* Try with expect.

* Lint.

* Add test.

* Remove mut.
* add more fuzzing for datalayer

* hint

* kinda tidy not really

* more

* stuff

* better

* drop prints

* stuff

* tidy

* catch up pyi

* fixup

* fix and add test case

* Update crates/chia-datalayer/src/merkle/iterators.rs

Co-authored-by: Arvid Norberg <arvid.norberg@gmail.com>

* back to immutable integrity check

* oops

---------

Co-authored-by: Arvid Norberg <arvid.norberg@gmail.com>
…long_lived_initial_datalayer_from_main_bfbc0f7b28e24babb17b4d7de9587725b4adcdf5
* tidy datalayer long lived

* actually run the script
…_datalayer_from_main_bfbc0f7b28e24babb17b4d7de9587725b4adcdf5

catchup: into long_lived/initial_datalayer from main  @ bfbc0f7
…long_lived_initial_datalayer_from_main_8703d67d83f513c90321f656ce5db2ee3544ecef
* CHIA-2072 create chia-fuzzing lib from clvm_rs utils (#1168)

For use by chia_rs and clvm_rs

* Switch fuzzing tests to use chia-fuzzing library (#1169)

All fuzzers use arbitrary now.

* bump to 0.28.0 (#1170)

---------

Co-authored-by: Jack Nelson <j.nelson@chia.net>
…long_lived_initial_datalayer_from_main_b5b4926b59411b4f68c373b04ba448a94ded0f79
…_datalayer_from_main_b5b4926b59411b4f68c373b04ba448a94ded0f79

catchup: into long_lived/initial_datalayer from main  @ b5b4926
…long_lived_initial_datalayer_from_main_7dd252c4cc3a0660bccdc9fdb1b92d7f43a32475
…_datalayer_from_main_7dd252c4cc3a0660bccdc9fdb1b92d7f43a32475

catchup: into long_lived/initial_datalayer from main  @ 7dd252c
…main_from_long_lived_initial_datalayer_cf34dfd8cd7d01f1f3e49df4173506c5ebbd716b
@altendky altendky added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog checkpoint_merge labels Aug 5, 2025
@coveralls-official

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16757985325

Details

  • 1915 of 2369 (80.84%) changed or added relevant lines in 10 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.8%) to 67.91%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/chia-datalayer/src/merkle/iterators.rs 140 149 93.96%
crates/chia-datalayer/src/merkle/dot.rs 84 95 88.42%
crates/chia-datalayer/src/merkle/proof_of_inclusion.rs 79 100 79.0%
wheel/src/api.rs 0 25 0.0%
crates/chia-datalayer/macro/src/lib.rs 0 50 0.0%
crates/chia-datalayer/src/merkle/format.rs 118 169 69.82%
crates/chia-datalayer/src/merkle/deltas.rs 246 331 74.32%
crates/chia-datalayer/src/merkle/blob.rs 1216 1418 85.75%
Totals Coverage Status
Change from base Build 16681737423: 1.8%
Covered Lines: 13468
Relevant Lines: 19832

💛 - Coveralls

@arvidn arvidn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you'd want this patch as well, run the fuzzers for a moment in CI.

diff --git a/.github/workflows/build-crate.yml b/.github/workflows/build-crate.yml
index 4a65e339..cb671c7d 100644
--- a/.github/workflows/build-crate.yml
+++ b/.github/workflows/build-crate.yml
@@ -114,3 +114,8 @@ jobs:
         run: |
           cd crates/clvm-traits
           cargo fuzz list | xargs -I "%" sh -c "cargo +nightly-2025-01-12 fuzz run % -- -max_total_time=20 || exit 255"
+
+      - name: Fuzz (chia-datalayer)
+        run: |
+          cd crates/chia-datalayer
+          cargo fuzz list | xargs -I "%" sh -c "cargo +nightly-2025-01-12 fuzz run % -- -max_total_time=20 || exit 255"

Comment thread crates/chia-datalayer/src/merkle/blob.rs
Comment thread crates/chia-datalayer/src/merkle/iterators.rs
Comment thread tests/test_datalayer.py
@altendky

altendky commented Aug 6, 2025

Copy link
Copy Markdown
Contributor Author

I think you'd want this patch as well, run the fuzzers for a moment in CI.

yes... i would like that. :| thanks.

@altendky
altendky marked this pull request as draft August 6, 2025 19:32
@altendky altendky closed this Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changed Required label for PR that categorizes merge commit message as "Changed" for changelog checkpoint_merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants