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

Update README.md #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update README.md #4

wants to merge 1 commit into from

Conversation

A-sh2000
Copy link

@A-sh2000 A-sh2000 commented May 4, 2024

  1. In the first paragraph, "This is done by incorporating Zero-Knowledge technology with Microprocessors without Interlocked Pipeline Stages (MIPS) architecture" can be improved for clarity. Maybe break it into two sentences.
  2. In the second paragraph, the sentence "The process of ZKM’s zkVM looks like this:" is followed by a figure, which seems a bit redundant. You can simply say "The zkVM process consists of the following steps:"
  3. In the numbered list, it's better to avoid using a comma after "Execution Trace" in the first item.
  4. In the third paragraph, the sentence "The result of this process is a Zero-Knowledge Proof for the correct execution of a program that is public with an input that might or might not be public, depending on the use case." could be clarified. Perhaps split it into two sentences for better readability.
  5. In the last paragraph, the phrase "an execution trace is generated, capturing the detailed sequence of operations" can be enhanced by adding "which" before "capturing" to make it clearer.
  6. In the last paragraph, the phrase "Then the prover generates a ZKP proof based on the program’s execution trace." seems redundant. Maybe simplify it to "The prover then generates a ZKP based on the program's execution trace."

1. In the first paragraph, "This is done by incorporating Zero-Knowledge technology with Microprocessors without Interlocked Pipeline Stages (MIPS) architecture" can be improved for clarity. Maybe break it into two sentences.
2. In the second paragraph, the sentence "The process of ZKM’s zkVM looks like this:" is followed by a figure, which seems a bit redundant. You can simply say "The zkVM process consists of the following steps:"
3. In the numbered list, it's better to avoid using a comma after "Execution Trace" in the first item.
4. In the third paragraph, the sentence "The result of this process is a Zero-Knowledge Proof for the correct execution of a program that is public with an input that might or might not be public, depending on the use case." could be clarified. Perhaps split it into two sentences for better readability.
5. In the last paragraph, the phrase "an execution trace is generated, capturing the detailed sequence of operations" can be enhanced by adding "which" before "capturing" to make it clearer.
6. In the last paragraph, the phrase "Then the prover generates a ZKP proof based on the program’s execution trace." seems redundant. Maybe simplify it to "The prover then generates a ZKP based on the program's execution trace."
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.

1 participant