-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Conversation
a2f6f52
to
a516d80
Compare
You mentio
Graph Service/GraphSync isn't a thing yet, so I think it shouldn't be in the diagram. |
@vmx the internal pieces in boxes with dashed pattern mean that they are in progress/missing but that is where they fit on the picture today. My bad for not explaining that right away, updated the description. |
@diasdavid K, cool. It would be nice if the diagram would have a legend what dashed boxes mean. |
@diasdavid can we replace the diagram here with it please? |
@alanshaw absolutely. I was expecting a bit more question/discussion, but if everyone simply agrees with this and likes it, then all good to me! :) |
Ok cool! I think it's great and there's not a lot to say about it since that IS the architecture. It's much better than the existing diagram already. Some feedback tho:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems awesome to me!
I am just not sure if we should add IPNS in this diagram, as well!
ping @daviddias |
Added to my TODO list for this week |
cfde4c4
to
ad8bac6
Compare
Applied CR :) |
I took some time to design the current IPFS Architecture so that we could have more informed discussions.
You can see it as png, txt or a monodraw file.
This diagram presents what the architecture is today and not what is next (the dashed boxes for the internal pieces also don't exist but that is where they fit the puzzle).