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

Tutorial 05: Add hints for console output #165

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

Conversation

SebastienGllmt
Copy link

The console log outputs aren't explained. We should at least have some hints about why this is the output

Ideally we would have links to learn more, but I've been told the signature description documentation is incorrect because "snarkyjs is not performing the algorithm from that doc, it's much simpler because it uses a random scalar for the nonce instead of deriving it"

@vercel
Copy link

vercel bot commented Nov 25, 2022

@SebastienGllmt is attempting to deploy a commit to the o1labs Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Member

@mitschabaude mitschabaude left a comment

Choose a reason for hiding this comment

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

thanks, but I'm not sure we should add JS-styled comments to console output. Maybe better to add those comments as normal text, after the console output?

@SebastienGllmt
Copy link
Author

Yeah, I have no strong preference on where the information is, but I think it would be useful for the section that explains the PrivateKey and PublicKey type to mention how these types are actually encoded. It could be a comment, or a paragraph that explains the output

@barriebyron barriebyron added docs Improvements or additions to documentation Tutorials How to, step-by-step developer walk throughs labels Aug 15, 2023
@barriebyron barriebyron requested a review from a team as a code owner August 15, 2023 20:37
@barriebyron
Copy link
Contributor

take a look now @mitschabaude wdyt?

Copy link
Member

@mitschabaude mitschabaude left a comment

Choose a reason for hiding this comment

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

Maybe this could be turned into a useful footnote, but as is I think it just hurts the text

@@ -161,6 +161,12 @@ signature verified for data2: true
Fields in signature: 256
```

where:

- `Fields in private key: 255` is a bit field element of the scalar field of the curve
Copy link
Member

Choose a reason for hiding this comment

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

I don't think "bit field" is a good term, to me it's confusing. So, to me, this doesn't clearly explain why a private key has 255 field elements, and I'm also questioning whether this is a good place to explain that in the first place.

Copy link
Contributor

Choose a reason for hiding this comment

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

@mitschabaude shall we forgo comments altogether? Or do you have a better way to describe?

@vercel
Copy link

vercel bot commented Aug 15, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 3, 2023 8:32pm

@barriebyron
Copy link
Contributor

gist: document the PrivateKey and PublicKey type to mention how these types are actually encoded, useful for console log output description

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation Tutorials How to, step-by-step developer walk throughs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants