bulletproofs Example on Readme is outdated | should use curve25519_dalek

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

Oct 17 '23 00:10 tapiocaboy

I can make a PR for this

Oct 17 '23 00:10 tapiocaboy

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.

Dec 08 '23 18:12 potuz

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: https://github.com/zkcrypto/bulletproofs/issues/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

Dec 10 '23 12:12 Stentonian

Can we close this ?

Nov 23 '25 23:11 tapiocaboy

Example on Readme is outdated | should use curve25519_dalek_ng