User Details
User Details
- User Since
- Jun 12 2024, 16:34 (43 w, 4 d)
Fri, Apr 11
Fri, Apr 11
roqqit requested changes to D17832: [avalanche] Poll for the transactions that didn't make it into the last block.
roqqit committed rABCc50432fcd1a8: [avalanche] Wait for the local winner instead of immediately asserting (authored by roqqit).
[avalanche] Wait for the local winner instead of immediately asserting
Thu, Apr 10
Thu, Apr 10
roqqit requested review of D17923: [avalanche] Wait for the local winner instead of immediately asserting.
roqqit committed rABCe0776c77fd38: [avalanche] Remove requirement that local staking rewards have to be computed… (authored by roqqit).
[avalanche] Remove requirement that local staking rewards have to be computed…
Wed, Apr 9
Wed, Apr 9
Add Shibusawa activation logic
Tue, Apr 8
Tue, Apr 8
roqqit requested review of D17912: Add Shibusawa activation logic.
Mon, Apr 7
Mon, Apr 7
roqqit committed rABCb0a72edbe957: [avalanche] Add debug logging when local contenders are set (authored by roqqit).
[avalanche] Add debug logging when local contenders are set
roqqit updated the diff for D17905: [avalanche] Add debug logging when local contenders are set.
Add block height
roqqit requested review of D17905: [avalanche] Add debug logging when local contenders are set.
Thu, Apr 3
Thu, Apr 3
roqqit requested changes to D17890: validation: pass ChainstateRole for validationinterface calls.
roqqit updated the summary of D17890: validation: pass ChainstateRole for validationinterface calls.
Wed, Apr 2
Wed, Apr 2
roqqit committed rABCee751ff92ab4: [avalanche] Cleanup unnecessary API passthrough of getWinners (authored by roqqit).
[avalanche] Cleanup unnecessary API passthrough of getWinners
roqqit committed rABCde65c9fdd05b: [avalanche] Clarify separation of responsibilities for setting local winner… (authored by roqqit).
[avalanche] Clarify separation of responsibilities for setting local winner…
roqqit committed rABCff86651f6506: [avalanche] Cleanup unused addStakeContender in Processor (authored by roqqit).
[avalanche] Cleanup unused addStakeContender in Processor
roqqit committed rABCa17bb89fa414: [avalanche] Add test coverage to ensure immature proofs are handled correctly… (authored by roqqit).
[avalanche] Add test coverage to ensure immature proofs are handled correctly…
roqqit requested review of D17882: [avalanche] Cleanup unnecessary API passthrough of getWinners.
roqqit requested review of D17880: [avalanche] Cleanup unused addStakeContender in Processor.
Mon, Mar 31
Mon, Mar 31
roqqit committed rABC09cc323fb712: [avalanche] Add stake contenders to the cache every time a proof is registered (authored by roqqit).
[avalanche] Add stake contenders to the cache every time a proof is registered
Thu, Mar 27
Thu, Mar 27
roqqit committed rABCaff18f3253ee: [avalanche] Make sure to only promote contenders once for each proofid (authored by roqqit).
[avalanche] Make sure to only promote contenders once for each proofid
roqqit committed rABC6fb44405d40d: [avalanche] Add some logging when stake contenders are cached or promoted (authored by roqqit).
[avalanche] Add some logging when stake contenders are cached or promoted
roqqit updated the diff for D17861: [avalanche] Make sure to only promote contenders once for each proofid.
remove the set because it is not really needed
roqqit updated the diff for D17860: [avalanche] Add some logging when stake contenders are cached or promoted.
feedback
Wed, Mar 26
Wed, Mar 26
roqqit updated the test plan for D17860: [avalanche] Add some logging when stake contenders are cached or promoted.
roqqit requested review of D17861: [avalanche] Make sure to only promote contenders once for each proofid.
roqqit requested review of D17860: [avalanche] Add some logging when stake contenders are cached or promoted.
Fri, Mar 21
Fri, Mar 21
roqqit committed rABC4271d8338029: [avalanche] Fix flaky proof rank comparison in processor_tests (authored by roqqit).
[avalanche] Fix flaky proof rank comparison in processor_tests
Thu, Mar 20
Thu, Mar 20
roqqit committed rABC892275701a2b: [avalanche] Refactor helpers to build avaproofs messages (authored by roqqit).
[avalanche] Refactor helpers to build avaproofs messages
roqqit committed rABC2b95ae7e2f43: [avalanche] Cleanup unused prefilled proof index set (authored by roqqit).
[avalanche] Cleanup unused prefilled proof index set
roqqit requested review of D17840: [avalanche] Fix flaky proof rank comparison in processor_tests.
Wed, Mar 19
Wed, Mar 19
roqqit requested changes to D17832: [avalanche] Poll for the transactions that didn't make it into the last block.
back to you
This looks a bit odd to move it to the callsite but since it is moving in the right direction with regards to the processor FIXME, it is probably a small step in the right direction.
roqqit requested review of D17834: [avalanche] Refactor helpers to build avaproofs messages.
roqqit requested review of D17833: [avalanche] Cleanup unused prefilled proof index set.
Tue, Mar 18
Tue, Mar 18
roqqit committed rABCf7b0bdf7c762: [electrum] Fix typo in trezor firmware warning text (authored by roqqit).
[electrum] Fix typo in trezor firmware warning text
roqqit requested review of D17818: [electrum] Fix typo in trezor firmware warning text.
Mon, Mar 17
Mon, Mar 17
roqqit committed rABCcf38f5e5d94a: [avalanche] Cleanup circular dependency between peer manager and contender cache (authored by roqqit).
[avalanche] Cleanup circular dependency between peer manager and contender cache
Mar 14 2025
Mar 14 2025
roqqit committed rABC821580a6ab15: [avalanche] Move contender cache to peer manager (authored by roqqit).
[avalanche] Move contender cache to peer manager
Mar 13 2025
Mar 13 2025
roqqit updated the diff for D17788: [avalanche] Move contender cache to peer manager.
- add lint exception for circular dependency
- fix lock annotations
Mar 12 2025
Mar 12 2025
roqqit planned changes to D17788: [avalanche] Move contender cache to peer manager.
roqqit requested review of D17788: [avalanche] Move contender cache to peer manager.
Mar 4 2025
Mar 4 2025
Mar 3 2025
Mar 3 2025
[avalanche] Down-rank rejected contenders
roqqit committed rABC489c9cfea31a: [avalanche] Fix a race in contenders voting test (again) (authored by roqqit).
[avalanche] Fix a race in contenders voting test (again)
roqqit updated the diff for D17728: [avalanche] Down-rank rejected contenders.
feedback
roqqit committed rABCd25b7436bca1: [avalanche] Process responses to stake contender polls (authored by roqqit).
[avalanche] Process responses to stake contender polls
roqqit requested review of D17737: [avalanche] Fix a race in contenders voting test (again).
Feb 28 2025
Feb 28 2025
roqqit committed rABCd029363ecc33: [avalanche] Fix a race in contenders voting test (authored by roqqit).
[avalanche] Fix a race in contenders voting test
roqqit requested review of D17729: [avalanche] Fix a race in contenders voting test.
reviewer note: the call to CustomInit() got added to BaseIndex::Start() in one commit and removed in the other. it makes comparing each to this patch a bit weird.
re: the note at the end of the summary: when you write this other diff, be sure to include test coverage that attempt to poll finalized ancestors. there is risk of regression here otherwise.
roqqit added inline comments to D17700: [avalanche] Process responses to stake contender polls.
roqqit requested review of D17728: [avalanche] Down-rank rejected contenders.
roqqit added inline comments to D17700: [avalanche] Process responses to stake contender polls.
roqqit updated the diff for D17700: [avalanche] Process responses to stake contender polls.
- use proofid as int in more places
- check stake winner order
- fix comment
Feb 27 2025
Feb 27 2025
no longer needed since D17710