Uh oh!
There was an error while loading. Please reload this page.
A first draft of a document describing Matrix state and state resolution (second attempt) - #8
A first draft of a document describing Matrix state and state resolution (second attempt)#8Magnap wants to merge 1 commit into
Conversation
non-Jedi
left a comment
There was a problem hiding this comment.
If the new state resolution doc in matrix-doc is fairly clear, no need to put
more work into this one. Actually, adding a link to it in here probably wouldn't
be a bad idea.
| ** A total order over states (as by "Erik's draft" and current Synapse) | ||
| For two events =A= and =B=, | ||
| =A < B= if =A='s depth is less than =B='s, | ||
| or, if their depths are equal, |
There was a problem hiding this comment.
This could be more clear. Do you mean something like:
if depth(A) < depth(B):
A < B
elseif depth(A) == depth(B) & hash(A) < hash(B):
A < B
else:
A > B
The phrasing of the sentence for the second condition is just a bit awkward and
convoluted.
| if the hash value of =A= is less than | ||
| the hash value of =B=. | ||
| Otherwise, =B > A=. | ||
There was a problem hiding this comment.
A practical example would be very helpful here. Especially interested in an
example with branches that have more than one event in them. To me what you
describe here sounds like those events would be interleaved, but that doesn't
seem to match what I've seen synapse do in practice.
Same content as was approved by @maxidor in #5, but that was merged before @non-Jedi had reviewed, leading to #7. So here's the second attempt.