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

IPA / Verkle tree cleanups #367

Closed
7 of 8 tasks
mratsim opened this issue Mar 9, 2024 · 1 comment
Closed
7 of 8 tasks

IPA / Verkle tree cleanups #367

mratsim opened this issue Mar 9, 2024 · 1 comment

Comments

@mratsim
Copy link
Owner

mratsim commented Mar 9, 2024

From #364 (review)

Easy

  • there is a newSeq introduced that needs to be replaced to avoid dependency on Nim runtime, especially if we want to expose IPA through C/Go/Rust
  • some proc like absIntChecker are exported but might be local-only
  • scalarMul needs an overload for field element and not just big ints
  • views need a splitMiddle functions and StridedView can then be replaced

Medium

Complex

@mratsim
Copy link
Owner Author

mratsim commented Jun 27, 2024

Closed by #397

@mratsim mratsim closed this as completed Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant