Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mm: Epoch reports follow up #3050

Merged
merged 2 commits into from
Oct 31, 2024
Merged

Conversation

martonp
Copy link
Contributor

@martonp martonp commented Oct 31, 2024

Some code clean up and fixes.

  • Removed unused code related to the simple arb
  • Simple arb bot was not reporting if the CEX orderbook was not synced
  • Fixes a divide by 0 panic happening in libxc.orderbook.vwap

Some code clean up and fixes.
- Removed unused code related to the simple arb
- Simple arb bot was not reporting if the CEX orderbook was not synced
- Fixes a divide by 0 panic happening in libxc.orderbook.vwap
Copy link
Contributor

@dev-warrior777 dev-warrior777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@buck54321 buck54321 merged commit 3506117 into decred:master Oct 31, 2024
5 checks passed
buck54321 pushed a commit that referenced this pull request Oct 31, 2024
* mm: Epoch reports follow up

Some code clean up and fixes.
- Removed unused code related to the simple arb
- Simple arb bot was not reporting if the CEX orderbook was not synced
- Fixes a divide by 0 panic happening in libxc.orderbook.vwap

* Fix UI for multiple bots and basic mm
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.

3 participants