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

Add split int hints #274

Merged
merged 6 commits into from
Sep 20, 2023
Merged

Add split int hints #274

merged 6 commits into from
Sep 20, 2023

Conversation

entropidelic
Copy link
Contributor

Closes #183

@codecov-commenter
Copy link

codecov-commenter commented Sep 20, 2023

Codecov Report

Merging #274 (53228cb) into main (061c955) will decrease coverage by 16.72%.
Report is 45 commits behind head on main.
The diff coverage is 48.11%.

@@             Coverage Diff             @@
##             main     #274       +/-   ##
===========================================
- Coverage   75.29%   58.58%   -16.72%     
===========================================
  Files          20       40       +20     
  Lines        2040     5179     +3139     
===========================================
+ Hits         1536     3034     +1498     
- Misses        453     1895     +1442     
- Partials       51      250      +199     
Files Changed Coverage Δ
pkg/builtins/builtin_runner.go 0.00% <0.00%> (ø)
pkg/hints/hint_utils/hint_reference.go 99.37% <ø> (ø)
pkg/parser/parser.go 57.89% <0.00%> (ø)
pkg/types/exec_scope_utils.go 0.00% <0.00%> (ø)
pkg/utils/math_utils.go 0.00% <0.00%> (ø)
pkg/builtins/poseidon.go 30.28% <4.00%> (-58.36%) ⬇️
pkg/builtins/signature.go 5.44% <5.44%> (ø)
pkg/vm/program.go 17.94% <14.28%> (-3.27%) ⬇️
pkg/builtins/pedersen.go 19.72% <19.72%> (ø)
pkg/builtins/output.go 24.69% <22.07%> (-62.81%) ⬇️
... and 27 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@entropidelic entropidelic marked this pull request as ready for review September 20, 2023 15:52
@fmoletta fmoletta added this pull request to the merge queue Sep 20, 2023
Merged via the queue into main with commit edaabf3 Sep 20, 2023
@fmoletta fmoletta deleted the split-int-hints branch September 20, 2023 18:55
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.

Add Div_rem and Split_int hints
4 participants