Skip to content

Conversation

@DanielSchiavini
Copy link
Contributor

@DanielSchiavini DanielSchiavini commented May 21, 2025

  • when the wallet is empty, scenarioWithLowestBalancesBN ends up with index -1 and that throws an error
  • instead, when all balances are zero, we can just stop the calculation

mimaklas
mimaklas previously approved these changes May 21, 2025
fedorovdg
fedorovdg previously approved these changes May 21, 2025
@DanielSchiavini DanielSchiavini marked this pull request as draft May 21, 2025 18:54
@DanielSchiavini DanielSchiavini dismissed stale reviews from fedorovdg and mimaklas via 9ff3b92 May 22, 2025 14:10
@DanielSchiavini DanielSchiavini marked this pull request as ready for review May 22, 2025 14:39
@DanielSchiavini DanielSchiavini merged commit a975af8 into master May 22, 2025
1 check passed
@DanielSchiavini DanielSchiavini deleted the fix/balanced branch November 14, 2025 08:58
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