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

Simulate position with coordinator #973

Closed
Tracked by #972
holzeis opened this issue Jul 21, 2023 · 1 comment · Fixed by #1216
Closed
Tracked by #972

Simulate position with coordinator #973

holzeis opened this issue Jul 21, 2023 · 1 comment · Fixed by #1216
Assignees
Labels
Milestone

Comments

@holzeis
Copy link
Contributor

holzeis commented Jul 21, 2023

The maker does not actually setup a dlc with the coordinator as it is a trusted setup. Accordingly the maker will simply create a simulated position in his database once the coordinator setup the dlc with the taker.

This position will always be updated contrary to how the coordinator manages its positions which are always inserted newly on open trade.

Additionally, PnL should be exchanged between the two players regularly, i.e. if the simulated position resulted in a loss for the maker, the maker should pay the coordinator (and vice-versa).

@bonomat bonomat mentioned this issue Jul 21, 2023
@holzeis holzeis transferred this issue from another repository Jul 21, 2023
@holzeis holzeis transferred this issue from another repository Jul 21, 2023
@bonomat bonomat added the maker label Jul 30, 2023
@luckysori
Copy link
Contributor

Additionally, PnL should be exchanged between the two players regularly, i.e. if the simulated position resulted in a loss for the maker, the maker should pay the coordinator (and vice-versa).

I suppose we could leave this out of the scope of this ticket. The maker can keep track of the PNL (and the coordinator too?), but we don't have to apply the changes until we have a channel in place (added with #926).

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

Successfully merging a pull request may close this issue.

3 participants