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

Fix URL RPC connect & async interface handling #127

Closed
wants to merge 9 commits into from

Conversation

smartgoo
Copy link
Collaborator

No description provided.

someone235 and others added 9 commits November 24, 2024 16:23
* Enable payloads for non coinbase transactions

* Add payload hash to sighash

* test reflects enabling payload

* Enhance benchmarking: add payload size variations

Refactored `mock_tx` to `mock_tx_with_payload` to support custom payload sizes. Introduced new benchmark function `benchmark_check_scripts_with_payload` to test performance with varying payload sizes. Commented out the old benchmark function to focus on payload-based tests.

* Enhance script checking benchmarks

Added benchmarks to evaluate script checking performance with varying payload sizes and input counts. This helps in understanding the impact of transaction payload size on validation and the relationship between input count and payload processing overhead.

* Add new test case for transaction hashing and refactor code

This commit introduces a new test case to verify that transaction IDs and hashes change with payload modifications. Additionally, code readability and consistency are improved by refactoring multi-line expressions into single lines where appropriate.

* Add payload activation test for transactions

This commit introduces a new integration test to validate the enforcement of payload activation rules at a specified DAA score. The test ensures that transactions with large payloads are rejected before activation and accepted afterward, maintaining consensus integrity.

* style: fmt

* test: add test that checks that payload change reflects sighash

* rename test

* Don't ever skip utxo_free_tx_validation

* lints

---------

Co-authored-by: max143672 <[email protected]>
* cargo fix all new 1.83 lints except for static_mut_refs

* use LazyLock for CONTEXT

* allow static_mut_refs for wallet storage paths

* bump MSRV to 1.82 and use the recent is_none_or
* Search for the full consecutive window covering all sampled blocks

* unrelated: reachability docs

* Avoid searching for the cover if window is not sampled

* cleanup WindowType::FullDifficultyWindow

* rename

* Fix cache origin issue and simplify cache management

* prevent access to block window cache get w/o specifying origin

* Suggested refactor for determining lock time type prior the call (to avoid leaking logic out of the TransactionValidator)

* long due renames

* renames and comments

* move window "cover" logic into WindowManager

* unrelated technical debt:  make sure to run par_iter within the context of an existing thread pool (avoid creating a global thread pool if possible)
…aspanet#599)

* Track the average transaction mass throughout the mempool's lifespan using a decaying formulae

* notebook analysis

* finalize notebook

* relax feerate estimator test comparisons to avoid arbitrary CI failures

* review comment
@smartgoo smartgoo closed this Nov 29, 2024
@smartgoo smartgoo deleted the py-async-fix branch November 29, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants