Tao Zhu

Results 35 issues of Tao Zhu

### SIMD https://github.com/solana-foundation/solana-improvement-documents/pull/96 ### Description Reward 100% of priority fee to validator. The implementation of this proposal necessitates the use of a feature gate. Although there will be no alteration...

feature-gate

#### Problem Stake-accounts tests don't handle transaction fee. #### Summary of Changes - explicitly sets fee rate to zero. Fixes #

#### Problem Tests expect zero transaction fee without be explicit about it. #### Summary of Changes - explicitly sets up tests with zero fee rate Fixes #

#### Problem FeeStructure used in tests should be initialized with intended `lamports_per_signature`. Using `FeeStructure::default()` only works if `lamports_per_signature == 0 or 5_000`. #### Summary of Changes - Initialize FeeStructure with...

#### Problem TestValidator can be created with custom specified target_lamportds_per_signature ``` pub fn with_custom_fees( mint_address: Pubkey, target_lamports_per_signature: u64, faucet_addr: Option, socket_addr_space: SocketAddrSpace, ) -> Self ``` Cli tests uses `with_custom_fees()`...

#### Problem Some bank tests are set up with no-fee no-rent, others requires rent but no-fee, yet others are OK with FeeRateGovernor::default() and Rent::default(). In preparation of removing a multiplier...

#### Problem `congestion_multiplier` variable in `sdk::fee_structure::calculate_fee(...)` does not have to do with congestion, it actually zeros out transaction fee for tests that don't handle transaction fees - those tests sets...

#### Problem nonce account holds a `fee_calculator` that provides `lamports_per_signature`. However this `lamports_per_signature` is not used: 1. `calculate_fee(...)` uses `fee_structure.lamports_per_siganture` to calculate signature fee; 2. `process_message(...)` passes last block_hash's `lamports_per_signature`...

#### Problem Cost of signatures can be cleaner #### Summary of Changes - refactor to simplify it by using SanitizedMessage Fixes #

community
need:merge-assist

### SIMD none ### Description A carry-over: transaction fee is converted to `f64`, then `round()` then convert back to `u64`. This is unnecessary, and rounding large `f64` isn't accurate since...

feature-gate