feat: catchain overview #1002
feat: catchain overview #1002
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Co-authored-by: Novus Nota <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
# Conflicts: # docs.json
skywardboundd
left a comment
skywardboundd
left a comment
There was a problem hiding this comment.
best article I've ever seen
|
Diagram is not perfect. The only idea to improve that is to do #1075 with Aigerim |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This comment was marked as outdated.
This comment was marked as outdated.
verytactical
left a comment
verytactical
left a comment
There was a problem hiding this comment.
The flow of the article is unclear. (TL/DR: didn't understand a thing.)
We've pushed some more comments directly into the branch.
| import { Stub } from '/snippets/stub.jsx'; | ||
|
|
||
| <Stub issue="171" /> | ||
| <Stub |
There was a problem hiding this comment.
Why is this article called "adnl", but its title is "Consensus"?
| import { Stub } from '/snippets/stub.jsx'; | ||
|
|
||
| <Stub issue="171" /> | ||
| <Stub |
There was a problem hiding this comment.
I don't think it's important enough to be listed between "Messages" and "Account status".
| ### Example message flow | ||
|
|
||
| ```mermaid | ||
| flowchart LR |
There was a problem hiding this comment.
"Which part of this image is unclear?" :)
There was a problem hiding this comment.
All message-related schemes should be in the same formatting, and every additional scheme adds context to what kind of design should be defined. So thanks to @Shvandre, he created another one.
For me, the unclear part starts from:
- Appearing crossing lines make this more complicated.
- Hard to get a timeline of events.
| - how to fetch dependencies; | ||
| - how to detect and penalize nodes that try to disrupt the system. | ||
|
|
||
| ## Choosing neighbor nodes |
There was a problem hiding this comment.
"Choosing neighbor" is part of overlay network protocol, not catchain
There was a problem hiding this comment.
Why? In sources it is in catchain-receiver.cpp
https://github.com/ton-blockchain/ton/blob/34823b1ea378edbe3bc59f3bcc48126480a0b768/catchain/catchain-receiver.cpp#L888
|
To fix the formatting issues:
npx remark -o --silent --silently-ignore foundations/consensus/bcp-deep-dive.mdx foundations/consensus/bcp-overview.mdx foundations/consensus/catchain-overview.mdx foundations/network/adnl.mdx |
|
|
||
| <Aside type="tip"> | ||
| In the current implementation, [Mersenne Twister](https://en.wikipedia.org/wiki/Mersenne_Twister) is used to pick the random interval for refreshing the neighbor list. It is seeded with values from [std::random_device](https://en.cppreference.com/w/cpp/numeric/random/random_device.html), whose behavior is platform-dependent. | ||
| In the current implementation, [Mersenne Twister](https://en.wikipedia.org/wiki/Mersenne_Twister) is used to pick the random interval for refreshing the neighbor list. It is seeded with values from [std::random\_device](https://en.cppreference.com/w/cpp/numeric/random/random_device.html), whose behavior is platform-dependent. |
There was a problem hiding this comment.
| In the current implementation, [Mersenne Twister](https://en.wikipedia.org/wiki/Mersenne_Twister) is used to pick the random interval for refreshing the neighbor list. It is seeded with values from [std::random\_device](https://en.cppreference.com/w/cpp/numeric/random/random_device.html), whose behavior is platform-dependent. | |
| In the current implementation, [Mersenne Twister](https://en.wikipedia.org/wiki/Mersenne_Twister) is used to pick the random interval for refreshing the neighbor list. It is seeded with values from [`std::random_device`](https://en.cppreference.com/w/cpp/numeric/random/random_device.html), whose behavior is platform-dependent. |
Since its a code ref, let's use backticks
Closes #986