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

Add limit for total number of active coins #147

Closed
redhelix opened this issue May 4, 2021 · 3 comments · Fixed by #311
Closed

Add limit for total number of active coins #147

redhelix opened this issue May 4, 2021 · 3 comments · Fixed by #311
Labels
enhancement New feature or request

Comments

@redhelix
Copy link

redhelix commented May 4, 2021

Is your proposal related to a problem?

When looking for the best deals, I like to add many different coin options. When many coins reach possible trade conditions, there is a risk that the bot will draw down the available balance very low or end up with a lot of trades in play.

Describe the solution you'd like

Add a parameter to limit total number of active deals

Describe alternatives you've considered

reducing the number of coins to monitor in the list - means foregoing potential profits

Additional context

@redhelix redhelix added the enhancement New feature or request label May 4, 2021
@chrisleekr
Copy link
Owner

chrisleekr commented May 4, 2021

Hmm, so like if I limit to 5 deals and there are 5 open orders, you don't want more buy?

@redhelix
Copy link
Author

redhelix commented May 5, 2021

Correct - Could limit by bases - 5 USDT, 5 BTC etc. Wait till coins cycle out could also sort by closest to buy price

@chrisleekr
Copy link
Owner

chrisleekr commented May 6, 2021

I actually found this feature needed when all coins were going down and placing a lot of orders for many coins at the same time.

I will add it to the TODO list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants