merge #8
Merged
merge #8
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* streamable: Merge `strictdataclass` into `Streamable` class * tests: Test not supported streamable types * streamable: Reorder decorators * streamable: Simplify streamable decorator and force correct usage/syntax * streamable: Just move some stuff around in the file * streamable: Improve syntax error messages * mypy: Drop `type_checking.py` and `test_type_checking.py` from exclusion * streamable: Use cached fields instead of `__annotations__` This is now possible after merging `__post_init__` into `Streamable` * Introduce `DefinitionError` as `StreamableError` * `/t` -> ` `
* expose farm block api to RPC for simulator * lint * pre-commit lint
* 2.7 seconds -> 0.45 seconds * Merge * Work on create_plots refactor * Try to fix tests * Try to fix tests * Use new functions * Fix block_tools by adding dir * Extra argument * Try to fix cyclic import * isort * Drop warning * Some cleanups around `exclude_final_dir` and directory adding * Cleanup `min_mainnet_k_size` checks * Drop unrelated changes * Fixes after rebase * Fix cyclic import * Update tests/block_tools.py Co-authored-by: dustinface <[email protected]> * Update tests/block_tools.py Co-authored-by: dustinface <[email protected]> Co-authored-by: xdustinface <[email protected]> Co-authored-by: dustinface <[email protected]>
* Added steps, when building the chiavdf wheel for macos and rhel-based systems, to install cmake and/or gmp.
* Mark the github workspace as safe * Move the git config step after git is installed in the test containers
…ex, but caching puzzle_hash -> secret key (which is the lookup we're actually interested in). This avoids duplicating the actual derivation (#11154)
…re-releases that could be returned by listReleases (#11165)
* Build cli only version of debs * Export the vars needed by j2 * Fix paths * Add symlink to chia in /usr/local/bin/ * Upload the cli only debs to s3 * Add init.py * Ensure SHA is on the dev build for amd64
…lso add feature keep_going_until_tx_block. (#11185)
* Fix a few additional flaky tests * Lint * Update tests/core/full_node/stores/test_full_node_store.py Co-authored-by: Jeff <[email protected]> * Fix merge conflict * Remove unnecessary function Co-authored-by: Jeff <[email protected]>
* port 0 to fix flakiness * Try fixing setup_full_system * Try fixing setup_full_system, and lint * More attempts to fix * No more calls to get random ports in setup_nodes * Revert accidental changes * Timelord extra arg * Try with port 0 * Fix daemon test, and lint * Try without 0.0.0.0 * Back to 0.0.0.0 * Try a few timelord changes to get test running * Increase timeout again * Use the correct interface to get the port * INFO logging to debug issue * Revert "INFO logging to debug issue" This reverts commit 7c379e5. * Fix advertised port log * Add extra log * Logging back * Rollback the timelord changes * Try port 0 timelord * Revert "Try port 0 timelord" This reverts commit 4997faf. * Try full green, change ordering * Remove unused var * speed up simulation and cleanup * Now try without the port config * Fix a flaky call to get_event_loop * Try getting the port dynamically * No dynamic port * Try changing the ordering * Try adding a sleep * Back to what works * Timelord before vdf clients * Dynamic port for 1st timelord * Revert "Dynamic port for 1st timelord" This reverts commit 0f322a1. * Revert "Timelord before vdf clients" This reverts commit 3286c34. * Revert "Back to what works" This reverts commit 30380df. * Revert "Try adding a sleep" This reverts commit 9212b66. * Revert "Try changing the ordering" This reverts commit a62597d. * Revert "No dynamic port" This reverts commit 5d2e157. * Revert "Try getting the port dynamically" This reverts commit ef9cd75. * Revert "Fix a flaky call to get_event_loop" This reverts commit 01a000f. * Try one to 0 * Just not 0 * Don't get port dynamically * Cleanup a bit * Fix * Some cleanup work * Some cleanup work * Fix daemon test * Cleanup * Remove arguments
* factor out as_list() from Coin into free function. Remove unused name_str() from Coin. Minor optimization of hash_coin_ids() for common case. * extend Coin unit test
…11589) * add tests.util.misc.assert_maximum_duration for benchmark assertions * fix some imports * more * lint * correct duration * hint fixup * rename to caller_file_and_line() * set default timer to thread_time() * extract gc manager and default to disabling * self calibrate overhead * wrap none-ness behind .results() method * add messages * from __future__ import annotations * yield out self * final from typing_extensions * split it * just use Future * tweak * tweak * correction in loosely hint checked file * all future indexing in hints * union for the hints * rename * assert_runtime() * another rename... * message -> label * label all results with test names * oops
* check for removals before calling function * move condition to function
* Bump actions/setup-python from 2 to 3 Bumps [actions/setup-python](https://github.com/actions/setup-python) from 2 to 3. - [Release notes](https://github.com/actions/setup-python/releases) - [Commits](actions/setup-python@v2...v3) --- updated-dependencies: - dependency-name: actions/setup-python dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * Fixup templates for python@v3 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Gene Hoffman <[email protected]>
This is a follow up after #11552 which basically used the initial parts of `test_cat_endpoints` to make `test_offer_endpoints` working. Now this PR drops a lot of the stuff we can assume to be not needed in this test because its tested in `test_cat_endpoints` and it adds some general improvments to it.
…ption. Normally this fails with an error code (#11566)
…r` (#11620) * hint Harvester.server as Optional[ChiaServer] * use a Protocol instead of Callable * oh right, Optional
Make sure it exists, is a directory and hasn't been added already.
* streamable: Cache convert functions for dict -> dataclass conversion * tests: Test `dataclass_from_dict` with non-streamable classes * `Any` -> `object` Co-authored-by: Kyle Altendorf <[email protected]> * Move comment into `dataclass_from_dict` Co-authored-by: Kyle Altendorf <[email protected]>
* changes for testscenario * revert change * implement sqlite version dependent variable * use parameter at module level, not as instance var * linefix * isort, black
* add get commands * remove get cmd * add address check via coin to show. * isort lint * fix circular import * fix cli qwerk * finish other mariano requests isort too * some tweaks * undo all wallet funcs changes * Update show.py * address changes * lint and fix * reset back submodule * removing new commands * remove exit_node command Also do a lot of cleanup * lint fix * lint on wallet_funcs * bug swatting * make another separate function fix a neat bug too * spicy lint * factor out last functions
* Remove is not None and length assertion in select_coins() The `coins is not None` check seems unneeded since `select_coins()` is hinted to return `Set[Coin]`. The length requirement needlessly restricts from being able to request a zero amount. Either situation would either trigger an exception in the next assert, or trigger the assert itself for non-zero amounts requested. * also remove from cat_wallet.py
* log error for failed service start * remove testing 1/0
* Add the `node_id` to the logs in `Receiver` * Log all processed messages * Log reset calls * Improve `reciever.Sync` and `delta.Delta` logging
* Use `get_harvesters_summary` in `chia farm summary` * Add initial plot loading indication to `chia farm summary`
Bumps [github/super-linter](https://github.com/github/super-linter) from 4.9.2 to 4.9.3. - [Release notes](https://github.com/github/super-linter/releases) - [Changelog](https://github.com/github/super-linter/blob/main/docs/release-process.md) - [Commits](github/super-linter@v4.9.2...v4.9.3) --- updated-dependencies: - dependency-name: github/super-linter dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
neurosis69
merged commit d886bd1
into
neurosis69:replace_asterisk_wallet_transaction_store
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.