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

Spec: IPC Actors #906

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Spec: IPC Actors #906

wants to merge 2 commits into from

Conversation

snissn
Copy link
Contributor

@snissn snissn commented May 8, 2024

Adds Ipc Actors Spec

@snissn snissn requested a review from cryptoAtwill May 8, 2024 21:04
@snissn snissn self-assigned this May 8, 2024
@cryptoAtwill
Copy link
Contributor

This seems more like a IPC tech overview than IPC Actor, is this the goal of the doc?

@maciejwitowski maciejwitowski changed the title Add IPC spec - Ipc Actors Spec: IPC Actors May 9, 2024
@snissn
Copy link
Contributor Author

snissn commented May 13, 2024

This seems more like a IPC tech overview than IPC Actor, is this the goal of the doc?

I agree there may be something a little unpolished about this spec, but overall it is helpful. It is a combination of this notion doc https://www.notion.so/pl-strflt/IPC-Actors-3ca52db23a114eb193573e88f901ba98?pm=c and notes that I wrote about the ipc actors. I also noticed that the images from the notion doc were missing and I added them to this PR. I think the images give context to make this document more palatable and worth including as is.

Copy link
Contributor

@raulk raulk left a comment

Choose a reason for hiding this comment

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

There is useful information here, but not in a clean/structured enough way. Keeping this PR open so we can potentially use this content as reference if needed while we engage in a major docs overhaul.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants