Codaone / DEXBot

Trading Bot for the BitShares Decentralized Exchange

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Relative orders multiple workers on same account balance logic clarification

bitphage opened this issue · comments

#434 adds two new worker params "operational percent BASE" and "operational percent QUOTE" to define which part of balance the worker should use.

For Relative Orders, I don't think the worker should obey this setting. This would allow for the following setup:

  • 1 SO worker with 49% BASE
  • 1 SO worker with 50% BASE
  • 1 RO worker with 1% BASE and relative order size say 10%

Despite the RO worker formally should be limited by 1% of total BASE, it could use all unallocated BASE balance (not in orders).

This is implicit logic which I don't like, but it's probably expected logic too.

So you are proposing to change this logic to to be ignored by RO?

Woudl this be the only change needed? To add an exception such that RO doesn't obey that logic?

This is a current behavior of RO, no changes needed. RO uses free balance (not in orders).