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

docs: add bitwise operation gadgets to o1js #737

Merged
merged 43 commits into from
Dec 11, 2023
Merged

docs: add bitwise operation gadgets to o1js #737

merged 43 commits into from
Dec 11, 2023

Conversation

barriebyron
Copy link
Contributor

@barriebyron barriebyron commented Nov 17, 2023

This PR closes https://app.zenhub.com/workspaces/zkapps-product-eng-6130fedb3b0fc600123d8796/issues/gh/o1-labs/docs2/733

Doc previews:

This PR includes a short overview of what gadgets are in general
A separate bitwise operations doc includes the descriptions and examples from https://github.com/o1-labs/o1js/blob/main/src/lib/gadgets/gadgets.ts

see questions in-line; thank yoU!

Includes the updated gadgets and new gadgets from o1-labs/o1js#1259

@barriebyron barriebyron added docs Improvements or additions to documentation o1js labels Nov 17, 2023
@barriebyron barriebyron self-assigned this Nov 17, 2023
Copy link

vercel bot commented Nov 17, 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 Dec 11, 2023 11:38am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
07-oracles ⬜️ Ignored (Inspect) Visit Preview Dec 11, 2023 11:38am

docs/zkapps/o1js/gadgets.mdx Outdated Show resolved Hide resolved
docs/zkapps/o1js/gadgets.mdx Outdated Show resolved Hide resolved
docs/zkapps/o1js/gadgets.mdx Outdated Show resolved Hide resolved
docs/zkapps/o1js/gadgets.mdx Outdated Show resolved Hide resolved
…Would suggest eng. review as this involved some interpretation
mitschabaude and others added 3 commits December 11, 2023 11:13
Co-authored-by: Barrie Byron <barrie@o1labs.org>
Co-authored-by: garwalsh <gwalsh@gmail.com>
Co-authored-by: Barrie Byron <barrie@o1labs.org>
Co-authored-by: Barrie Byron <barrie@o1labs.org>
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.

I applied @barriebyron's suggestions and one more. Good to go from my side!

@barriebyron barriebyron merged commit 0593b09 into main Dec 11, 2023
4 checks passed
@barriebyron barriebyron deleted the gadgets branch December 11, 2023 14:41
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 o1js
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants