tevador opened PR #507 (base branch fcmp++-stage is correct). Lock time increased to 3 days for better UX (e.g., Friday force-close allows until Monday). jberman will review and include in upstream PRs; concept has had sufficient time for objections. Prior conflicting issues no longer apply after recent changes.
rucknium: Since last time this was discussed, tevador made the PR:
tevador: I'm not sure if the base branch for the PR is correct. It should be whichever branch gets the consensus changes.
tevador: It would be nice to have it reviewed and merged for the audit.
jeffro256: Base branch of fcmp++-stage is good
jberman: Ya I made some changes over in in the upstream PR's that would conflict with this, and the issue UkoeHB pointed out in #445 shouldn't apply after those changes
tevador: Btw, I decided to increase the lock time to 3 days, which is more user friendly (channel force close on Friday gives the user until Monday to react).
jberman: I can review it and get it included in the upstream PR's as well. At this point this concept has had quite a bit of time for people to voice objection to payment channels imo
tevador: Thanks
View quoted note →
GitHub
Relative lock - consensus code by tevador · Pull Request #507 · seraphis-migration/monero
This PR adds consensus support for relative locks (replaces #445).
Relative locks are activated from HF 18 and the relative lock time is 2160 block...
#monero-research-lab