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

#PR 1733 #1902

Closed
wants to merge 37 commits into from
Closed

#PR 1733 #1902

wants to merge 37 commits into from

Conversation

manikey123
Copy link
Contributor

PR for bts issue #1733

pmconrad and others added 30 commits August 11, 2019 17:47
Some bots place limit orders then try to cancel some of them before a
new block arrives. However, the order's id (in pending state) may be
different in individual nodes, thus caused the exceptions.

A potential solution to the order_id inconsistent issue:
bitshares#556
Some misconfigured bots were trying to place limit orders in
unauthorized markets, or place orders when don't have sufficient
balance (this could also be caused by inconsistent pending state).
pmconrad and others added 7 commits August 11, 2019 17:47
The multi-stage build makes builds more reliable by building only
the protocol in the first stage, which reliably passes in time,
caching those binaries, and then building the rest and running
tests in the second stage, which uses the cache to complete in
time as well.
@manikey123 manikey123 closed this Aug 12, 2019
@manikey123 manikey123 deleted the bts1733_f1 branch August 12, 2019 01:15
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.

7 participants