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.

Sign up for GitHub

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

Jump to bottom

feat: add WideMul128 hint #124

Open
Tracked by #90
zmalatrax opened this issue Aug 6, 2024 · 8 comments · May be fixed by #137
Open
Tracked by #90

feat: add WideMul128 hint #124

zmalatrax opened this issue Aug 6, 2024 · 8 comments · May be fixed by #137
Assignees
@coxmars
Labels
good first issue Good for newcomers

Comments

@zmalatrax
Copy link
Collaborator

Estimated time: 0.5d
Lifespan: 1d

  • Implement the WideMul128 hint.
  • Add unit tests, testing the parsing and the logic of the hint.
  • Add a Cairo program using the hint, used as an integration test .

How-to implement a hint on the Cairo VM TS available here

Rust implementation reference
@zmalatrax zmalatrax mentioned this issue Aug 6, 2024
Open
@zmalatrax zmalatrax added the good first issue Good for newcomers label Aug 6, 2024
@D240021
Copy link

D240021 commented Aug 6, 2024

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

I am Daniel Barrantes, a fullstack software developer passionate about learning new technologies. I have recently entered the world of web3 and I would like to complete my first issue.

How I plan on tackling this issue

-I'll read the code carefully to understand the task
-I'll work hard to make the code work properly
-I'll implement unit tests and integration tests to verify their functionality

@CristopherAguilar10
Copy link

Hello @zmalatrax, can I work on this issue? I would like to help

@onlydustapp OnlyDustApp
Copy link

onlydustapp bot commented Aug 6, 2024

Hey @CristopherAguilar10!
Thanks for showing interest.
We've created an application for you to contribute to Kakarot zkEVM.
Go check it out on OnlyDust!

@zmalatrax
Copy link
Collaborator Author

Sure, assigned @D240021

@zmalatrax zmalatrax linked a pull request Aug 7, 2024 that will close this issue
feat: add WideMul128 hint #126
Closed
@zmalatrax zmalatrax linked a pull request Aug 27, 2024 that will close this issue
feat(hint): widemul128 implementation #137
Open
@zmalatrax
Copy link
Collaborator Author

@coxmars re-assigned

@coxmars
Copy link
Contributor

coxmars commented Aug 27, 2024

@coxmars re-assigned

Thanks sir, I will be working on it asap 🫡

@onlydustapp OnlyDustApp
Copy link

onlydustapp bot commented Aug 27, 2024

Hey @coxmars!
Thanks for showing interest.
We've created an application for you to contribute to Kakarot zkEVM.
Go check it out on OnlyDust!

@od-hunter
Copy link

Hi @zmalatrax , please is this open for contribution?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@coxmars coxmars

Labels
good first issue Good for newcomers
Projects
Cairo VM ts
Status: 🏗️ In progress
Milestone
No milestone
5 participants
@zmalatrax @coxmars @D240021 @CristopherAguilar10 @od-hunter