crypto-trading-bot icon indicating copy to clipboard operation
crypto-trading-bot copied to clipboard

Minimum revenue before selling

Open Fluorz opened this issue 4 years ago • 10 comments

In this file : https://github.com/TeamWertarbyte/crypto-trading-bot/blob/development/src/modules/bot/index.ts

You are checking if revenue is more than 0 before selling. We can config a minimum revenue before selling. Sometimes the bot is selling when he makes 10 cents, but with the bittrex commission we are losing money.

I think it would be wise to sell a share only if the revenue including the commission fees are superior to the configurated minimum_revenue.

What do you think about adding a minimum revenue variable?

Fluorz avatar Apr 07 '21 11:04 Fluorz

It will also be a good idea to set in the config the bittrex commission. Like this we are sure to sell when revenue - bittrex_commission > minimun_revenue

Fluorz avatar Apr 07 '21 11:04 Fluorz

Good idea! Thanks for the suggestion. @Fluorz Will include that in the next release. https://github.com/TeamWertarbyte/crypto-trading-bot/blob/235120abbe806537c5d68a1d05107134e5d57801/src/modules/bot/index.ts#L151

How should we interpret minimun_revenue ? As an absolute value like 5.50 -> 5$ and 50 cents or percent like 5.50 -> 5.5% of amountToInvest -> 5.5% from 50$ = 2.75$

saschb2b avatar Apr 07 '21 12:04 saschb2b

In percent I think : https://bittrexglobal.zendesk.com/hc/en-us/articles/360009625260-What-fees-does-Bittrex-Global-charge-

Fluorz avatar Apr 07 '21 13:04 Fluorz

I meant the minimun_revenue parameter not bittrex_commission. bittrex_commission seems to be just a percentage, yes.

Edit: How to calculate comission:

The fee (commission) charged by Bittrex on each executed trade is calculated by taking the (amount * purchase price * rate) for the given trade

saschb2b avatar Apr 07 '21 14:04 saschb2b

minimun_revenue in percent would be great 👍

Fluorz avatar Apr 07 '21 14:04 Fluorz

A parameter to never reject and just hold would also be nice.

https://github.com/TeamWertarbyte/crypto-trading-bot/blob/development/src/modules/evaluation/EMAEvaluation.ts

Fluorz avatar Apr 07 '21 14:04 Fluorz

A parameter to never reject and just hold would also be nice.

https://github.com/TeamWertarbyte/crypto-trading-bot/blob/development/src/modules/evaluation/EMAEvaluation.ts

Already implemented. See example for BTC and ETH here https://github.com/TeamWertarbyte/crypto-trading-bot/blob/235120abbe806537c5d68a1d05107134e5d57801/config.json#L2

Just add your coins to hodl in that array

Edit: Or do you mean globally?

saschb2b avatar Apr 07 '21 14:04 saschb2b

I meant globally

Fluorz avatar Apr 07 '21 14:04 Fluorz

Would you mind creating a separate issue for that?

saschb2b avatar Apr 07 '21 14:04 saschb2b

#49

Fluorz avatar Apr 07 '21 14:04 Fluorz