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

invariant explanation made more precise; some extra clarifications #714

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

ivan-gavran
Copy link
Contributor

Description

In this commit I tried to make some notions more precise, correct some typos, and add extra explanations where I thought they were necessary.

In particular, I replaced the assumption of the "final jailed provider smaller than F" with the assumption of "every provider smaller than F". The reason is that it is up to adversary to choose the last slashed provider and thus we need to have this stronger assumption.

Other changes are mostly about formatting and adding gluing explanations.

Copy link
Contributor

@shaspitz shaspitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks great, thanks!

@shaspitz shaspitz merged commit b837e74 into cosmos:throttle-adr Feb 7, 2023
shaspitz added a commit that referenced this pull request Feb 9, 2023
* wip

* Update adr-002-throttle.md

* fix typos

* rm todo

* link #685

* note on unjailing

* mathz

* Update adr-002-throttle.md

* invariant explanation made more precise; some extra clarifications (#714)

* Update docs/architecture/adr-002-throttle.md

* title update

* invariant -> property

* math

* smol

* latex

---------

Co-authored-by: Ivan Gavran <ivan.gavran7@gmail.com>
Co-authored-by: Jehan <jehan.tremback@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants