Skip to content

Commit 45e98d8

Browse files
committed
fix namespace format
1 parent e61de90 commit 45e98d8

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

gkr_iop/src/circuit_builder.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -880,14 +880,14 @@ impl<'a, E: ExtensionField> CircuitBuilder<'a, E> {
880880
NR: Into<String>,
881881
N: Fn() -> NR,
882882
{
883-
assert!(size <= 16);
883+
assert!(size <= 16, "{size} > 16");
884884
self.assert_u16(
885-
|| format!("assert_ux_in_u16_1/{}", name_fn().into()),
885+
|| format!("assert_ux_in_u16_{}_check1", name_fn().into()),
886886
expr.clone(),
887887
)?;
888888
if size < 16 {
889889
self.assert_u16(
890-
|| format!("assert_ux_in_u16_2/{}", name_fn().into()),
890+
|| format!("assert_ux_in_u16_{}_check2", name_fn().into()),
891891
expr * E::BaseField::from_canonical_u64(1 << (16 - size)).expr(),
892892
)?;
893893
}
@@ -1126,7 +1126,7 @@ impl<'a, E: ExtensionField> CircuitBuilder<'a, E> {
11261126

11271127
// Lookup ranges
11281128
for (i, (size, elem)) in split_rep.iter().enumerate() {
1129-
self.assert_ux_in_u16(|| format!("{}/{}", name().into(), i), *size, elem.clone())?;
1129+
self.assert_ux_in_u16(|| format!("{}_{}", name().into(), i), *size, elem.clone())?;
11301130
}
11311131

11321132
// constrain the fact that rep8 and repX.rotate_left(chunks_rotation) are

gkr_iop/src/precompiles/lookup_keccakf.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -173,12 +173,12 @@ impl<E: ExtensionField> KeccakLayout<E> {
173173
): (KeccakWitCols<WitIn>, KeccakFixedCols<Fixed>, [WitIn; 6]) = unsafe {
174174
(
175175
transmute::<[WitIn; KECCAK_WIT_SIZE], KeccakWitCols<WitIn>>(array::from_fn(|id| {
176-
cb.create_witin(|| format!("keccak/witin_{}", id))
176+
cb.create_witin(|| format!("keccak_witin_{}", id))
177177
})),
178178
transmute::<[Fixed; 8], KeccakFixedCols<Fixed>>(array::from_fn(|id| {
179-
cb.create_fixed(|| format!("keccak/fixed_{}", id))
179+
cb.create_fixed(|| format!("keccak_fixed_{}", id))
180180
})),
181-
array::from_fn(|id| cb.create_witin(|| format!("keccak/eq_{}", id))),
181+
array::from_fn(|id| cb.create_witin(|| format!("keccak_eq_{}", id))),
182182
)
183183
};
184184

@@ -276,7 +276,7 @@ impl<E: ExtensionField> ProtocolBuilder<E> for KeccakLayout<E> {
276276
assert_eq!(c_temp.slice(s![i, ..]).iter().len(), sizes.iter().len());
277277

278278
system.require_left_rotation64(
279-
|| format!("theta rotation/{i}"),
279+
|| format!("theta rotation_{i}"),
280280
&c_aux
281281
.slice(s![i, 4, ..])
282282
.iter()

0 commit comments

Comments
 (0)