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: long_lived/vault from main @ 07fcecd #17496

Conversation

Quexington
Copy link
Contributor

@Quexington Quexington commented Feb 5, 2024

AmineKhaldi and others added 30 commits January 30, 2024 09:31
…sync_untrusted_break (#17419)

Use Monkeypatching for register_interest_in_puzzle_hash in test_long_sync_untrusted_break.
ProposalInfo's passed and closed are not optional.
No need to readd consensus_mode fixture in DAO wallet tests.

It's already included at this point.
Annotate lineage_proof.py.
Annotate p2_delegated_conditions.py.
Annotate p2_puzzle_hash.py.
* spends are actually spent coins.
* Address unused values.
* puzzle is already a Program so there is no need to construct one from it in order to get its tree hash.
* Improve constructing parent_coin_info and amount values.
Improve did_get_info.

* Check optional values.
* Compute launcher_id only once and reuse it.
* Simplify computing solution.
* Compute coin state's coin ID only once.
* Improve constructing hints using only one dictionary lookup.
Remove no longer needed TestCATTrades class.
Produce flakiness in a type-friendly way in test_retry_store.
…_id in did_find_lost_did (#17427)

Properly construct bytes32 values for recovery_list_hash and launcher_id in did_find_lost_did.

recovery_list_hash.atom is optional bytes and launcher_id was bytes.
* Merge chia-blochchain-gui 2.2.0

* Updated GUI locales
…nt/main_from_release_2.2.0_880c1193bc7464c0313809d0d4254041950cc87f
Bumps [cryptography](https://github.com/pyca/cryptography) from 41.0.7 to 42.0.2.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@41.0.7...42.0.2)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [colorlog](https://github.com/borntyping/python-colorlog) from 6.8.0 to 6.8.2.
- [Release notes](https://github.com/borntyping/python-colorlog/releases)
- [Commits](borntyping/python-colorlog@v6.8.0...v6.8.2)

---
updated-dependencies:
- dependency-name: colorlog
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [coverage](https://github.com/nedbat/coveragepy) from 7.4.0 to 7.4.1.
- [Release notes](https://github.com/nedbat/coveragepy/releases)
- [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst)
- [Commits](nedbat/coveragepy@7.4.0...7.4.1)

---
updated-dependencies:
- dependency-name: coverage
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
fix `Install.ps1` for pwsh 7.4 (#17417)

* fix `Install.ps1` for pwsh 7.4

* fixup

* -not

Co-authored-by: Kyle Altendorf <sda@fstab.net>
Bumps [pytest](https://github.com/pytest-dev/pytest) from 7.4.4 to 8.0.0.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@7.4.4...8.0.0)

---
updated-dependencies:
- dependency-name: pytest
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tes (#17373)

Update "can not find parent coin" error log
…nt/main_from_release_2.2.0_1070ce8c363709c3f62ef32c92a6b81e6318d4ea
* Update pool url on redirect

* Add tests

* Only persist the new url when it's a permanent redirect

* Remove unused pool_info overrides
* use `psutil.cpu_affinity()` instead of `os.cpu_count()`

* go simple for darwin

* assert, yay!
Improve Offer's from_spend_bundle.

* Improve constructing nonce.
* Remove incorrect annotation for payment_args_list (it's an iterator not a list).
* Don't autopush transactions by default anywhere

* Remove ignore_max_send_amount

* Use TXConfig instead of pending_transactions for coin exclusion in spend_clawback_coins

* Address offline comments by @emlowe

* Rework max send amount to be based on coin quantity rather than amount

* merge fixes

* Inadvertent diffs

* Inadvertent change

* Inadvertent diff.

---------

Co-authored-by: Amine Khaldi <amine.khaldi@reactos.org>
* ensure signal objects for signal handlers

* Update misc.py

* black
Annotate test_wallet_sync.py.
Remove no longer needed TestTaproot class.
AmineKhaldi and others added 14 commits February 5, 2024 08:41
Speedup test_wallet_reorg_get_coinbase.
Speedup test_request_block_headers.
Speedup test_long_sync_untrusted_break.
Speedup test_wallet_reorg_sync.
Speedup test_short_batch_sync_wallet.
Speedup test_request_block_headers_rejected.
Speedup test_subscribe_for_hint_long_sync.
Speedup test_wallet_blockchain.
Speedup test_get_balance.
Speedup test_large_trade_record_query.
Speedup TradeStore's _get_new_trade_records_from_old.
Speedup TradeStore's add_trade_record.
Speedup test_almost_recent.
* Don't autopush transactions by default anywhere

* Remove ignore_max_send_amount

* Use TXConfig instead of pending_transactions for coin exclusion in spend_clawback_coins

* Unify transaction pushing into a single method

* Address offline comments by @emlowe

* Rework max send amount to be based on coin quantity rather than amount

* Inadvertent changes

* Test fixes

* Inadvertent change

* Bad import?

* Inadvertent changes?

* readd missing test
@Quexington Quexington added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes vault labels Feb 5, 2024
AmineKhaldi and others added 2 commits February 5, 2024 13:17
Remove no longer used dataclasses import.
…ult_from_main_c6f1a57c565fb3e66afe7991669df93958dbe049
@Quexington Quexington force-pushed the catchup/long_lived_vault_from_main_c6f1a57c565fb3e66afe7991669df93958dbe049 branch from 18d5ea9 to a4b5732 Compare February 6, 2024 15:23
@Quexington Quexington marked this pull request as ready for review February 6, 2024 16:14
@Quexington Quexington requested a review from a team as a code owner February 6, 2024 16:14
@Quexington Quexington changed the title catchup: long_lived/vault from main @ c6f1a57c565fb3e66afe7991669df93958dbe049 catchup: long_lived/vault from main @ 07fcecd Feb 6, 2024
Copy link
Contributor

github-actions bot commented Feb 6, 2024

File Coverage Missing Lines
chia/plotting/check_plots.py 0.0% lines 29, 60
chia/rpc/wallet_rpc_api.py 83.3% lines 2314, 2395, 3925
chia/timelord/timelord_api.py 83.3% lines 52, 55
chia/util/beta_metrics.py 50.0% lines 27
tests/farmer_harvester/test_farmer.py 98.4% lines 997
Total Missing Coverage
828 lines 9 lines 98%

Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

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

aok

@Quexington Quexington merged commit 640b154 into long_lived/vault Feb 8, 2024
265 of 266 checks passed
@Quexington Quexington deleted the catchup/long_lived_vault_from_main_c6f1a57c565fb3e66afe7991669df93958dbe049 branch February 8, 2024 16:46
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 coverage-diff Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes vault
Projects
None yet
Development

Successfully merging this pull request may close these issues.