Skip to content
This repository has been archived by the owner on Sep 5, 2023. It is now read-only.

Handling mix currencies USDT, BTC, ETH, USD all together. #90

Open
mrFingerlos opened this issue Nov 1, 2021 · 2 comments
Open

Handling mix currencies USDT, BTC, ETH, USD all together. #90

mrFingerlos opened this issue Nov 1, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@mrFingerlos
Copy link

Handling mix currencies USDT, BTC, ETH, USD all together.

To make the 3C portfolio manager consistent with the 3C interface.

@mrFingerlos mrFingerlos added the enhancement New feature or request label Nov 1, 2021
@coltoneshaw
Copy link
Owner

@mrFingerlos - Thanks! I've been trying to figure out how to handle this. The biggest issue is around properly showing / calculating your risk. I may end up just allowing multiple currencies to be selected and adding a risk warning that it may be misleading


So, right now the app is based on the currencies you select in the app itself only. This is primarily important for your risk factor because viewing your risk and combining multiple currencies will skew your "true risk" since your bankroll will be artificially higher.

Example: Your risk shows as 80% but your funds are split across USDT ($5000) and BTC ($5000). So you decide to increase your USDT bot from $5000 to $7000. Now your risk shows 100%. In reality, you're actually at 140% for USDT.

However, with that said. It does allow combining USD base currencies since they run the same value together and can be transitioned easily.


That's a long explanation to just say I don't want anyone to misunderstand their true risk. If you have a use case for why risk should be combined I'm happy to make adjustments if it makes sense! My way isn't the only way.

@coltoneshaw
Copy link
Owner

Proposed Solution

Allow selecting as many currencies as the end user wants within the stats filter but requires a few UI changes.

  • Risk metric should display with an * next to it if multiple currencies are mixed. This would have a tooltip that shows the warning
  • Look at added a risk table to show your risk based on the currencies
  • If too many currencies are selected the app filter would display something like 5 currencies and a hover would display them all
  • Need to decide what the "root currency" here is to handle the rounding across the app. Or just assume everything goes to USD and call it a day.

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

No branches or pull requests

2 participants