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

Discuss time till profitability #533

Closed
3 tasks done
Tracked by #522
LukSteib opened this issue Nov 11, 2022 · 4 comments · Fixed by #537
Closed
3 tasks done
Tracked by #522

Discuss time till profitability #533

LukSteib opened this issue Nov 11, 2022 · 4 comments · Fixed by #537

Comments

@LukSteib
Copy link
Contributor

LukSteib commented Nov 11, 2022

Goal

Callee selection affects time till profitability

Context

We have introduced multi callee support that allows a user to select between different exchange callees. These come with different market price.
So ultimately one can expect that the "Time till profitability" indicator would need to change based on the callee selection.
Currently unsure whether we take this into account.
If the market prices of different callees only slightly differ chances are high that auction price will drop below all with the same price drop step.
However if prices of different callees differ quite a lot the time till profitability might also be different since the thresholds won't be reached with the same price drop

Let's use this issue to discuss whether the assumption above is correct and if so how to address

Tasks

  • Discuss effect of callee selection on "Time till profitability"
  • (if needed)Outline approach for adjustment
  • (if needed) Implement adjustment
@valiafetisov
Copy link
Contributor

As far as I remember Time till profitability is already computed per callee. But not sure it the correct one is displayed. @aomafarag can you please confirm that with links to the source code (lines where it's computed per callee and lines where it is changed based on the selected callee) or outline what needs to be changed?

@aomafarag
Copy link
Contributor

As far as I remember Time till profitability is already computed per callee.

Yes, it is. When enriching the auction, we calculate transactionGrossProfitDate for each callee.

@valiafetisov
Copy link
Contributor

and lines where it is changed based on the selected callee

What about this question? It is currently calculated, but not changed on the frontend?

@aomafarag
Copy link
Contributor

and lines where it is changed based on the selected callee

What about this question? It is currently calculated, but not changed on the frontend?

OK, I just checked. It is not updated on the frontend because the TimeTillProfitable component takes the whole auctionTransaction object as a prop, not just the transactionGrossProfitDate (check this).

What we need to do is just pass the transactionGrossProfitDate of the selected callee instead of the whole object and the issue would be resolved.

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 a pull request may close this issue.

3 participants