-
Notifications
You must be signed in to change notification settings - Fork 20
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
Consider removing checkpoint from the contracts #1597
Comments
If we do this, let's not forget to update the docs also. |
@NiloCK keen for your feedback on this. |
So far, the Yellow team has been considering using
That being said, the Yellow team thinks removing |
Thanks for this input @nksazonov! Case 1 is absolutely the intended application of It may be that we can keep If we revert to a "delegated funding" model, we could run virtual channels with turn numbers, along with many other benefits. |
Regarding cross-channel checks: In our use case we have a state channel (let's call it clearing channel) that operates with collateral, but does not have funds deposited (so it is non-custodial). After some time a need for settlement arises, and parties have to exchange collateralized funds. One of the options we are implementing is state channel (escrow channel), where parties deposit funds. Before executing a settlement, parties have to agree on a post-settlement margin (collateral distribution on a clearing channel). Therefore, settlement and applying post-settlement margin (PSM) must be an atomic operation. Atomicity is achieved by introducing a check in an escrow channel regarding the PSM. Thus, the steps are the following:
It would be great to hear your opinion on this! P.S. For now, we are not using virtual channels due to their complexity, but this will change in future. |
Based on discussion over the
virtual payment app
andcheckpoint
it's now clear thatcheckpoint
-ing while using theVirtualPaymentApp
doesn't make sense.That prompted us to consider whether we should have a
checkpoint
function at all. It's not clear that there any strong use cases for it and removing it would simplifyForceMove
and reduce the attack service.We should consider if
checkpoint
will be useful in use cases we're interested in and if not consider removing it.The text was updated successfully, but these errors were encountered: