Skip to content

Commit

Permalink
chore: copy edit docs
Browse files Browse the repository at this point in the history
Thanks to Aurel for expanding the docs.  That helps my understanding a
lot, and is sure to be useful for people who come later, too.

This PR does some minor polishing of the comments for ease of
readability.
  • Loading branch information
matthiasgoergens committed Nov 21, 2024
1 parent e426fa7 commit e5f10ed
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions ceno_zkvm/src/tables/ram/ram_circuit.rs
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ impl<E: ExtensionField, NVRAM: NonVolatileTable + Send + Sync + Clone> TableCirc
num_fixed: usize,
io_addrs: &[Addr],
) -> RowMajorMatrix<E::BaseField> {
// assume returned table is well-formed include padding
// assume returned table is well-formed including padding
config.gen_init_state(num_fixed, io_addrs)
}

Expand All @@ -127,7 +127,7 @@ impl<E: ExtensionField, NVRAM: NonVolatileTable + Send + Sync + Clone> TableCirc
_multiplicity: &[HashMap<u64, usize>],
final_cycles: &[Cycle],
) -> Result<RowMajorMatrix<E::BaseField>, ZKVMError> {
// assume returned table is well-formed include padding
// assume returned table is well-formed including padding
config.assign_instances(num_witin, final_cycles)
}
}
Expand All @@ -153,8 +153,8 @@ pub trait DynVolatileRamTable {
}
}

/// DynVolatileRamCircuit initializes and finalizes memory with
/// - at witnessed addresses, in a contiguous range chosen by the prover.
/// DynVolatileRamCircuit initializes and finalizes memory
/// - at witnessed addresses, in a contiguous range chosen by the prover,
/// - with zeros as initial content,
/// - with witnessed final content that the program wrote.
pub struct DynVolatileRamCircuit<E, R>(PhantomData<(E, R)>);
Expand Down

0 comments on commit e5f10ed

Please sign in to comment.