Closed aarmoa closed 3 months ago
The recent changes improve gas limit estimation in the pyinjective
trading system by increasing various gas limits for order types and optimizing test cases to utilize defined constants instead of hardcoded values. This enhances maintainability, clarity, and accuracy across the codebase and tests, ensuring alignment with the latest definitions.
Files | Change Summary |
---|---|
pyinjective/core/gas_limit_estimator.py |
Increased gas limits for spot and derivative orders, modified multipliers, and adjusted constants in multiple estimator classes, indicating an overall increase. |
tests/core/test_gas_limit_estimator.py , tests/core/test_message_based_transaction_fee_calculator.py |
Replaced hardcoded gas limit values with constants from respective estimator classes to enhance maintainability and accuracy of test cases. |
CHANGELOG.md , pyproject.toml |
Updated version information and documented changes related to gas cost estimation and the transition to version 1.6.3. |
π° In the land of gas limits, oh so bright,
Weβve tweaked the numbers, made them just right.
Orders and messages dance with such grace,
Constants now guide them, keeping up pace.
Hops of improvement, a joyous new day,
Letβs celebrate changes, hip-hip-hooray! πΌβ¨
Solves CHAIN-172
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Chores