Skip to content

Conversation

@mrdaybird
Copy link
Contributor

This PR fixes/closes issue #184

  • bigint, bigfield
    • cleanup, add comments
  • update existing primitives to use this
  • remove crypto_bigint as dependency

@Autoparallel
Copy link
Contributor

Great start!

@Autoparallel
Copy link
Contributor

Anything I can do to help move this along @mrdaybird ?

@mahmudsudo
Copy link
Contributor

I will like to contribute to this , looking at the last bullet point

@Autoparallel
Copy link
Contributor

Can we make this a tracking issue and have sub issues?

Also, I think we really only need a bigint -- we can just wrap PrimeField around our bigint once we have it.

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.

3 participants