Skip to content

Commit 6c11449

Browse files
committed
Fix some benchmarks
Signed-off-by: adir <adir@il.ibm.com>
1 parent c167f63 commit 6c11449

File tree

3 files changed

+19
-13
lines changed

3 files changed

+19
-13
lines changed

token/core/zkatdlog/nogh/v1/issue/issuer_test.go

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,10 +67,11 @@ func BenchmarkIssuer(b *testing.B) {
6767

6868
i := 0
6969
for b.Loop() {
70-
issuer := issue2.NewIssuer("ABC", &mock.SigningIdentity{}, env.IssuerEnvs[i].pp)
70+
e := env.IssuerEnvs[i%len(env.IssuerEnvs)]
71+
issuer := issue2.NewIssuer("ABC", &mock.SigningIdentity{}, e.pp)
7172
action, _, err := issuer.GenerateZKIssue(
72-
env.IssuerEnvs[i].outputValues,
73-
env.IssuerEnvs[i].outputOwners,
73+
e.outputValues,
74+
e.outputOwners,
7475
)
7576
require.NoError(b, err)
7677
_, err = action.Serialize()
@@ -100,16 +101,17 @@ func BenchmarkProofVerificationIssuer(b *testing.B) {
100101

101102
i := 0
102103
for b.Loop() {
104+
e := env.IssuerEnvs[i%len(env.IssuerEnvs)]
103105
// deserialize action
104106
action := &issue2.Action{}
105-
require.NoError(b, action.Deserialize(env.IssuerEnvs[i].actionRaw))
107+
require.NoError(b, action.Deserialize(e.actionRaw))
106108

107109
// verify
108110
coms := make([]*math.G1, len(action.Outputs))
109111
for i := range len(action.Outputs) {
110112
coms[i] = action.Outputs[i].Data
111113
}
112-
require.NoError(b, issue2.NewVerifier(coms, env.IssuerEnvs[i].pp).Verify(action.GetProof()))
114+
require.NoError(b, issue2.NewVerifier(coms, e.pp).Verify(action.GetProof()))
113115
i++
114116
}
115117
})

token/core/zkatdlog/nogh/v1/transfer/transfer_test.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,11 +81,12 @@ func BenchmarkTransferProofGeneration(b *testing.B) {
8181

8282
i := 0
8383
for b.Loop() {
84+
e := env.ProverEnvs[i%len(env.ProverEnvs)]
8485
prover, err := transfer.NewProver(
85-
env.ProverEnvs[i].a,
86-
env.ProverEnvs[i].b,
87-
env.ProverEnvs[i].c,
88-
env.ProverEnvs[i].d,
86+
e.a,
87+
e.b,
88+
e.c,
89+
e.d,
8990
env.pp,
9091
)
9192
require.NoError(b, err)

token/core/zkatdlog/nogh/v1/transfer_test.go

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,23 +73,26 @@ func BenchmarkTransferServiceTransfer(b *testing.B) {
7373
outputs, err := benchmark2.NumOutputs(1, 2, 3)
7474
require.NoError(b, err)
7575
testCases := benchmark2.GenerateCases(bits, curves, inputs, outputs, []int{1})
76+
configurations, err := benchmark.NewSetupConfigurations("./testdata", bits, curves)
77+
require.NoError(b, err)
7678

7779
for _, tc := range testCases {
7880
b.Run(tc.Name, func(b *testing.B) {
79-
env, err := newBenchmarkTransferEnv(b.N, tc.BenchmarkCase, nil)
81+
env, err := newBenchmarkTransferEnv(b.N, tc.BenchmarkCase, configurations)
8082
require.NoError(b, err)
8183

8284
// Optional: Reset timer if you had expensive setup code above
8385
b.ResetTimer()
8486

8587
i := 0
8688
for b.Loop() {
87-
action, _, err := env.Envs[i].ts.Transfer(
89+
e := env.Envs[i%len(env.Envs)]
90+
action, _, err := e.ts.Transfer(
8891
b.Context(),
8992
"an_anchor",
9093
nil,
91-
env.Envs[i].ids,
92-
env.Envs[i].outputs,
94+
e.ids,
95+
e.outputs,
9396
nil,
9497
)
9598
require.NoError(b, err)

0 commit comments

Comments
 (0)