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

Example on Readme is outdated | should use curve25519_dalek_ng #373

Open
tapiocaboy opened this issue Oct 17, 2023 · 3 comments
Open

Example on Readme is outdated | should use curve25519_dalek_ng #373

tapiocaboy opened this issue Oct 17, 2023 · 3 comments

Comments

@tapiocaboy
Copy link

Seems the example on README uses use curve25519_dalek::scalar::Scalar; but it expects use curve25519_dalek_ng::scalar::Scalar;

The function

    #[cfg(feature = "std")]
    pub fn prove_single(
        bp_gens: &BulletproofGens,
        pc_gens: &PedersenGens,
        transcript: &mut Transcript,
        v: u64,
        v_blinding: &Scalar,
      

accepts v_blinding accepts type

curve25519_dalek_ng::scalar
pub struct Scalar
@tapiocaboy tapiocaboy changed the title Example on Readme is outdated Example on Readme is outdated | should use curve25519_dalek_ng Oct 17, 2023
@tapiocaboy
Copy link
Author

I can make a PR for this

@potuz
Copy link

potuz commented Dec 8, 2023

The last two issues on the repo are about this. In addition the ng package is outdated with respect to the original curve25519-dalek package and so it's missing some methods like the basic from() to construct scalars.

@Stentonian
Copy link

I've made a PR on zkcrypto/bulletproofs (which is bulletproofs version 4 on crates.io, where as this repo is version 2) to sort out the dependency issues.

See here for more details: zkcrypto#15

I'm not sure what the best way forward is:

  • keep zkcrypto/bulletproofs as the latest bulletproofs version on crates.io
  • make this repo the latest version

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

No branches or pull requests

3 participants