Skip to content

Commit

Permalink
Document with doc comments
Browse files Browse the repository at this point in the history
  • Loading branch information
EmilLuta committed Jul 30, 2024
1 parent 77328ce commit 61ee109
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions core/lib/config/src/configs/fri_witness_generator.rs
Original file line number Diff line number Diff line change
Expand Up @@ -97,11 +97,11 @@ impl FriWitnessGeneratorConfig {
self.last_l1_batch_to_process.unwrap_or(u32::MAX)
}

// 500 was picked as a mid-ground between allowing enough circuits in flight to speed up circuit generation,
// whilst keeping memory as low as possible. At the moment, max size of a circuit is ~50MB.
// This number is important when there are issues with saving circuits (network issues, service unavailability, etc.)
// Maximum theoretic extra memory consumed is up to 25GB (50MB * 500 circuits), but in reality, worse case scenarios are closer to 5GB (the average space distribution).
// During normal operations (> P95), this will incur an overhead of ~100MB.
/// 500 was picked as a mid-ground between allowing enough circuits in flight to speed up circuit generation,
/// whilst keeping memory as low as possible. At the moment, max size of a circuit is ~50MB.
/// This number is important when there are issues with saving circuits (network issues, service unavailability, etc.)
/// Maximum theoretic extra memory consumed is up to 25GB (50MB * 500 circuits), but in reality, worse case scenarios are closer to 5GB (the average space distribution).
/// During normal operations (> P95), this will incur an overhead of ~100MB.
const fn default_max_circuits_in_flight() -> usize {
500
}
Expand Down

0 comments on commit 61ee109

Please sign in to comment.