Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Bench exchange tweaks #3957

Merged
merged 2 commits into from
Apr 23, 2019
Merged

Conversation

jackcmay
Copy link
Contributor

Problem

Summary of Changes

Tweaked the bench-exchange to focus on perf

  • Different chunk lenghts for funding vs trading
  • Remove double chunk in trader
  • Don't attempt to capture all trades sent, instead focus on sampling, its worse if the sampler keeps sampling after trading stops and incorporates zero tx samples.

Fixes #

@codecov
Copy link

codecov bot commented Apr 23, 2019

Codecov Report

Merging #3957 into master will decrease coverage by <.1%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master   #3957     +/-   ##
========================================
- Coverage    77.8%   77.8%   -0.1%     
========================================
  Files         162     162             
  Lines       27470   27471      +1     
========================================
- Hits        21399   21390      -9     
- Misses       6071    6081     +10

1 similar comment
@codecov
Copy link

codecov bot commented Apr 23, 2019

Codecov Report

Merging #3957 into master will decrease coverage by <.1%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master   #3957     +/-   ##
========================================
- Coverage    77.8%   77.8%   -0.1%     
========================================
  Files         162     162             
  Lines       27470   27471      +1     
========================================
- Hits        21399   21390      -9     
- Misses       6071    6081     +10

@jackcmay jackcmay merged commit f2e2106 into solana-labs:master Apr 23, 2019
@jackcmay jackcmay deleted the bench-exchange-tweaks branch April 23, 2019 23:48
tao-stones added a commit to tao-stones/solana that referenced this pull request Dec 9, 2024
…after SIMD-170 (solana-labs#3957)

Add tests to document how cost tracker counts and adjusts before and
after SIMD-170.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant