Skip to content

Conversation

@topointon-jump
Copy link
Contributor

@topointon-jump topointon-jump commented Nov 7, 2025

Eliminate several failure modes when sending vote accounts from replay->tower.

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.06363 s 0.063782 s 0.239%
backtest mainnet-368528500-perf snapshot load 2.146 s 2.14 s -0.280%
backtest mainnet-368528500-perf total elapsed 63.630275 s 63.78239 s 0.239%
firedancer mem usage with mainnet.toml 1024.14 GiB 1024.14 GiB 0.000%

@topointon-jump topointon-jump force-pushed the topointon/vote_states_fail branch from f739e7e to 948e117 Compare November 7, 2025 20:32
@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.074302 s 0.170018 s 128.820% ⚠️
backtest mainnet-368528500-perf snapshot load 2.659 s 2.647 s -0.451%
backtest mainnet-368528500-perf total elapsed 74.301883 s 170.018185 s 128.821% ⚠️
firedancer mem usage with mainnet.toml 991.14 GiB 991.14 GiB 0.000%

@topointon-jump topointon-jump force-pushed the topointon/vote_states_fail branch from 948e117 to 29a9afd Compare November 7, 2025 21:33
@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.052729 s 0.052166 s -1.068%
backtest mainnet-368528500-perf snapshot load 1.707 s 1.659 s -2.812%
backtest mainnet-368528500-perf total elapsed 52.729387 s 52.165898 s -1.069%
firedancer mem usage with mainnet.toml 991.14 GiB 991.14 GiB 0.000%

@ibhatt-jumptrading
Copy link
Contributor

ibhatt-jumptrading commented Nov 7, 2025

i was thinking about this a little bit, we could just cross reference with the current vote states because that stays up to date so if an account is not in vote_states but is in vote_states_prev, we get rid of it. So I propose we just add stakes_t_1 to fd_vote_states.c/h. This makes it so we can reliably just reference the current vote states for the source of truth and don't need to rely on funk failure modes or introspecting on the actual account data. This should be a quick fix: you just need to copy how the vote states has the stake values from t-2

@topointon-jump topointon-jump force-pushed the topointon/vote_states_fail branch from 29a9afd to 4aa3022 Compare November 8, 2025 18:14
@github-actions
Copy link

github-actions bot commented Nov 8, 2025

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.073011 s 0.073088 s 0.105%
backtest mainnet-368528500-perf snapshot load 2.686 s 2.694 s 0.298%
backtest mainnet-368528500-perf total elapsed 73.011141 s 73.088259 s 0.106%
firedancer mem usage with mainnet.toml 991.14 GiB 992.14 GiB 0.101%

@topointon-jump topointon-jump force-pushed the topointon/vote_states_fail branch from 4aa3022 to ded967d Compare November 8, 2025 18:43
@topointon-jump topointon-jump changed the title replay: fix behaviour when we fail to fetch vote state from funk replay: improve reliability of replay to tower vote states sending Nov 8, 2025
@github-actions
Copy link

github-actions bot commented Nov 8, 2025

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.060051 s 0.051055 s -14.981%
backtest mainnet-368528500-perf snapshot load 1.782 s 1.376 s -22.783%
backtest mainnet-368528500-perf total elapsed 60.050677 s 51.055305 s -14.980%
firedancer mem usage with mainnet.toml 991.14 GiB 992.14 GiB 0.101%

@topointon-jump topointon-jump force-pushed the topointon/vote_states_fail branch from ded967d to 7961723 Compare November 8, 2025 18:52
@github-actions
Copy link

github-actions bot commented Nov 8, 2025

Performance Measurements ⏳

Suite Baseline New Change
backtest mainnet-368528500-perf per slot 0.072978 s 0.073203 s 0.308%
backtest mainnet-368528500-perf snapshot load 2.696 s 2.653 s -1.595%
backtest mainnet-368528500-perf total elapsed 72.977749 s 73.203475 s 0.309%
firedancer mem usage with mainnet.toml 991.14 GiB 992.14 GiB 0.101%

@topointon-jump
Copy link
Contributor Author

updated this so that fd_replay_out_vote_tower_from_funk can never fail (cc @ibhatt-jumptrading @mmcgee-jump)

@topointon-jump topointon-jump merged commit b2365c8 into main Nov 9, 2025
12 checks passed
@topointon-jump topointon-jump deleted the topointon/vote_states_fail branch November 9, 2025 23:57
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.

4 participants