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

catchup: into long_lived/datalayer_merkle_blob from main @ 084599bf2b51dbc7beef983d88e44724c24b7698 #19221

Open
wants to merge 31 commits into
base: long_lived/datalayer_merkle_blob
Choose a base branch
from

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Feb 3, 2025

Source hash: 084599b
Remaining commits: 0

ChiaMineJP and others added 30 commits January 22, 2025 23:57
seeing 25% hang rate and top successful time of 22 minutes so let's fail a bit quicker on the hangs.

https://github.com/Chia-Network/chia-blockchain/actions/runs/12712771698/usage
Remove no longer needed get_block_path.
* Add singleton_records to action scopes

* Extract SingletonRecord to its own file

* Fix test_lifecycle

* Fix wallet action scope test mock
…equest_unfinished_block2 (#19141)

Fix a typo when picking the best unfinished block in test_request_unfinished_block2.
Remove no longer needed has_peers_with_peak.
* Swap out `Payment` for `CreateCoin`

* Fix mypy-exclusions

* tweak
Allow coin selection of 0 value coins
* pyproject.toml consistency

* fixup
* Add some extra safety into `create_message_spend`

* Fix a poorly named variable
* move augmented to caller

* use undelying height_to_hash to check main chain
* dump shellcheck and actionlint versions in ci

* forward slashes

Co-authored-by: Kyle Altendorf <[email protected]>
…nt/main_from_release_2.5.1_26217db138ff4642973e220fd01fc7cada1fe345
…h_replaced_generator (#19139)

Fix a plot ID calculation typo in test_unfinished_block_with_replaced_generator.
Add config constant support to `chia db validate`
- [Fixed an issue where input focus is lost on offer builder
page](Chia-Network/chia-blockchain-gui#2564)
- [Fixed moderate severity vulnerability in one of npm
dependencies.](Chia-Network/chia-blockchain-gui#2576)
- Added [log
viewer](Chia-Network/chia-blockchain-gui#2565)
- Updated various npm dependencies

The pin is set to
https://github.com/Chia-Network/chia-blockchain-gui/tree/a3b94b23a6963238bffe8fce251f4dd4cc1819cc
…nt/main_from_release_2.5.1_9343b143b042bec38f6a7845e45e2b5f550aaeb0
Delete chia/simulator/simulator_constants.py

the keychain seems unneeded, it might be wrong given the 'new consensus' comment, and it prints a python literal not a hex string or somesuch.  and it uses the `DEFAULT_ROOT_PATH` global i'm trying to get rid of.  oh, plus it's a directly runnable file that ought to be available via some `chia` subcommand instead, if it sticks around.
updated chia_rs to latest version (0.19.0)
…nt/main_from_release_2.5.1_ec77339dfe9b809b8f80baeff588ab11fa518d22
…long_lived_datalayer_merkle_blob_from_main_084599bf2b51dbc7beef983d88e44724c24b7698
@altendky altendky requested a review from a team as a code owner February 3, 2025 15:55
@altendky altendky added checkpoint_merge Changed Required label for PR that categorizes merge commit message as "Changed" for changelog labels Feb 3, 2025
Copy link

socket-security bot commented Feb 3, 2025

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: pypi/[email protected]

View full report↗︎

Copy link

coveralls-official bot commented Feb 3, 2025

Pull Request Test Coverage Report for Build 13120119837

Details

  • 173 of 183 (94.54%) changed or added relevant lines in 41 files are covered.
  • 27 unchanged lines in 9 files lost coverage.
  • Overall coverage increased (+0.04%) to 91.498%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chia/_tests/wallet/rpc/config.py 0 1 0.0%
chia/_tests/core/full_node/test_full_node.py 1 3 33.33%
chia/wallet/did_wallet/did_wallet_puzzles.py 6 8 75.0%
chia/cmds/db_validate_func.py 7 12 58.33%
Files with Coverage Reduction New Missed Lines %
chia/_tests/simulation/test_simulation.py 1 96.45%
chia/full_node/full_node_api.py 1 84.38%
chia/wallet/puzzles/puzzle_utils.py 1 76.19%
chia/wallet/wallet_node.py 1 88.25%
chia/timelord/timelord.py 2 78.2%
chia/timelord/timelord_launcher.py 2 89.29%
chia/rpc/rpc_server.py 4 88.54%
chia/server/node_discovery.py 7 80.95%
chia/full_node/full_node.py 8 86.17%
Totals Coverage Status
Change from base Build 13075024935: 0.04%
Covered Lines: 105519
Relevant Lines: 115144

💛 - Coveralls

Copy link
Contributor

github-actions bot commented Feb 3, 2025

File Coverage Missing Lines
chia/_tests/core/full_node/test_full_node.py 33.3% lines 1474-1475
chia/cmds/db_validate_func.py 58.3% lines 21, 23-25, 29
chia/wallet/did_wallet/did_wallet_puzzles.py 75.0% lines 86, 88
Total Missing Coverage
182 lines 9 lines 95%

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 coverage-diff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants