Skip to content

Commit

Permalink
congestion: show 90th tx delay percentile in summary (near#11008)
Browse files Browse the repository at this point in the history
For quickly comparing many different parameters in the fine-tuning
phase, knowing the median latency is not good enoungh.
  • Loading branch information
jakmeier authored Apr 10, 2024
1 parent 246f7cc commit 4c10f94
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions tools/congestion-model/src/evaluation/summary_table.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,13 @@ use crate::PGAS;

pub fn print_summary_header() {
println!(
"{:<25}{:<25}{:>25}{:>25}{:>16}{:>16}{:>16}{:>16}",
"{:<25}{:<25}{:>25}{:>25}{:>16}{:>16}{:>16}{:>16}{:>16}",
"WORKLOAD",
"STRATEGY",
"BURNT GAS",
"TRANSACTIONS FINISHED",
"MEDIAN TX DELAY",
"90p TX DELAY",
"MAX QUEUE LEN",
"MAX QUEUE SIZE",
"MAX QUEUE PGAS",
Expand All @@ -24,10 +25,11 @@ pub fn print_summary_row(
user_experience: &UserExperience,
) {
println!(
"{workload:<25}{strategy:<25}{:>20} PGas{:>25}{:>16}{:>16}{:>16}{:>16}",
"{workload:<25}{strategy:<25}{:>20} PGas{:>25}{:>16}{:>16}{:>16}{:>16}{:>16}",
throughput.total / PGAS,
progress.finished_transactions,
user_experience.successful_tx_delay_median,
user_experience.successful_tx_delay_90th_percentile,
max_queues.queued_receipts.num,
bytesize::ByteSize::b(max_queues.queued_receipts.size),
max_queues.queued_receipts.gas / PGAS,
Expand Down

0 comments on commit 4c10f94

Please sign in to comment.