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

Min volume rounding can result in error #2181

Closed
smk762 opened this issue Dec 12, 2022 · 0 comments
Closed

Min volume rounding can result in error #2181

smk762 opened this issue Dec 12, 2022 · 0 comments

Comments

@smk762
Copy link
Collaborator

smk762 commented Dec 12, 2022

Generally, it works. You can't start trade with Volume < maker min_volume Yet, there's an issue:

image

If you use exact trade volume == maker min_volume, where maker's min volume has more than 6 decimals, order will likely not match due to rounding:

image

@smk762 this PR seems to fix initial issue/report, we can also fix my problem as separate issue. What do you think?

Yeah, def needs to be fixed - will create issue for it and tackle at same time as #2180

Originally posted by @smk762 in #2170 (comment)

@smk762 smk762 closed this as completed Dec 12, 2022
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

No branches or pull requests

1 participant