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 :Add project overview, key features, applications, a… #357

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

Conversation

cypriansakwa
Copy link

…nd detailed documentation

  • Added Project Overview and Introduction sections to provide a clear understanding of Nova.
  • Included Key Features and Applications to highlight Nova’s capabilities and use cases.
  • Expanded Library Details with information on elliptic curves, commitment schemes, and SNARK implementations.
  • Detailed the testing instructions and contributed to improving project accessibility.
  • Updated Acknowledgments section to recognize contributors and supporters.
  • Included table of contents and ensured links are clickable and functional. It improves navigation by allowing users to quickly jump to the sections they’re most interested in, enhancing the overall user experience.

…nd detailed documentation

- Added Project Overview and Introduction sections to provide a clear understanding of Nova.
- Included Key Features and Applications to highlight Nova’s capabilities and use cases.
- Expanded Library Details with information on elliptic curves, commitment schemes, and SNARK implementations.
- Detailed the testing instructions and contributed to improving project accessibility.
- Updated Acknowledgments section to recognize contributors and supporters.
- Included table of contents and ensured links are clickable and functional. It improves navigation by allowing users to quickly jump to the sections they’re most interested in, enhancing the overall user experience.
@cypriansakwa
Copy link
Author

Hi @srinathsetty . Hope you are fine. This update includes improvements to the README, such as enhanced documentation and structure. Please review the changes, especially the new sections for the project overview, features, and applications. Any feedback or suggestions would be greatly appreciated. Thank you!

@cypriansakwa
Copy link
Author

cypriansakwa commented Jan 18, 2025 via email

@cypriansakwa
Copy link
Author

@microsoft-github-policy-service agree

@cypriansakwa
Copy link
Author

Dear Team,

I hope this message finds you well. I have submitted a pull request titled [Update README.md :Add project overview, key features, applications], which addresses changes on the Readme.

I kindly request your review of the changes to ensure they align with the project’s requirements and standards. Please let me know if there are any areas that need further adjustment or improvement.

Your feedback is highly appreciated, and I’m happy to make any necessary changes promptly. Thank you for taking the time to review my submission.

Looking forward to your insights.

Best regards,
Cyprian Sakwa

@srinathsetty
Copy link
Collaborator

Hi @cypriansakwa thanks for the contribution!

Can you clarify what new documentation was added here (besides organizing existing README in a different way)?

@cypriansakwa
Copy link
Author

Thank you @srinathsetty for your question. While much of the content from the original README has been retained, the updated version introduces several key enhancements and new documentation, which we believe add significant value:

  1. Project Overview: A new section that provides a concise summary of Nova, highlighting its purpose and the innovative use of folding schemes.

  2. Table of Contents: Added for easier navigation through the document.

  3. Introduction: Expanded to explain the advantages of using folding schemes over traditional SNARKs and to provide a high-level overview of Nova's implementation and performance benefits.

  4. Key Features: A new section that lists and explains the main features of Nova, such as Incrementally Verifiable Computation (IVC), efficiency, no trusted setup, and the constant-sized verifier circuit.

  5. Acknowledgments: Added to recognize the contributions of researchers, the open-source community, and Microsoft.

  6. Running Tests: Simplified instructions for running tests in release mode.

These additions and reorganizations aim to make the README more user-friendly and informative, providing a clearer understanding of Nova's capabilities and usage.

Copy link
Author

@cypriansakwa cypriansakwa left a comment

Choose a reason for hiding this comment

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

Subject: Follow-Up on Pull Request Review
Hi @srinathsetty
I hope you're doing well.
I made the requested changes to my pull request based on your feedback. When you have a moment, could you please take another look and let me know if anything else needs adjustment?
I appreciate your time and guidance. Looking forward to your thoughts!
Best,
Cyprian Sakwa

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