consensus-shipyard / consensuslab

The go-to place for scalable decentralised consensus research
66 stars 3 forks source link

Add hierarchical consensus spec #113

Closed adlrocha closed 2 years ago

adlrocha commented 2 years ago

Updated version of hierarchical consensus spec: Rendered version

adlrocha commented 2 years ago

Question to all involved. Do you think that the spec would be easier to read and review if we split it in different files? Should we consider this for a follow-up PR once we get this merged?

I want to address the last few comments from @aakoshh regarding cross-net fees (and move the description earlier in the text), and if no one opposes I was planning to get the current version merged. Let me know if someone needs more time to review :pray:

jsoares commented 2 years ago

Question to all involved. Do you think that the spec would be easier to read and review if we split it in different files? Should we consider this for a follow-up PR once we get this merged?

I don't think so. Change tracking is also easier in a single file. If things get really out of hand (e.g. Filecoin spec), we could have a sidebar + collapsible sections, but I don't think this is big enough to justify that.

I want to address the last few comments from @aakoshh regarding cross-net fees (and move the description earlier in the text), and if no one opposes I was planning to get the current version merged. Let me know if someone needs more time to review 🙏

No objections from my end.

adlrocha commented 2 years ago

Thank you all for the amazing job reviewing the spec :pray: :tada:. I am merging this PR and opening follow-ups to discuss the parts of the spec that need more work. Feel free to do the same if you feel there's anything that needs further discussion.