docs: update time-related documentation (#1222)

* docs: update time-related documentation

* fix retention too

2 files changed, 29 insertions, 19 deletions

diff --git a/src/args.rs b/src/args.rs
index b384acc0..88cfe3fb 100644
--- a/src/args.rs
+++ b/src/args.rs
@@ -149,9 +149,9 @@ pub fn build_app() -> Command {
         .short('n')
         .long("unnormalized_cpu")
         .action(ArgAction::SetTrue)
-        .help("Show process CPU% without normalizing over the number of cores.")
+        .help("Show process CPU% usage without normalizing over the number of cores.")
         .long_help(
-            "Shows process CPU usage without averaging over the number of CPU cores in the system.",
+            "Shows all process CPU% usage without averaging over the number of CPU cores in the system.",
         );
 
     let disable_click = Arg::new("disable_click")
@@ -304,7 +304,7 @@ Defaults to \"default\".
         .value_name("TIME")
         .help("Default time value for graphs.")
         .long_help(
-            "Default time value for graphs. The minimum time is 30s, and the default is 60s.",
+            "Default time value for graphs. Takes a number in milliseconds or a human duration (e.g. 60s). The minimum time is 30s, and the default is 60s.",
         );
 
     // TODO: Charts are broken in the manpage
@@ -352,9 +352,9 @@ use CPU (3) as the default instead.
         .short('r')
         .long("rate")
         .action(ArgAction::Set)
-        .value_name("MS")
+        .value_name("TIME")
         .help("Sets the data refresh rate.")
-        .long_help("Sets the data refresh rate. The minimum is 250ms, and defaults to 1000ms. Smaller values may take more computer resources.");
+        .long_help("Sets the data refresh rate. Takes a number in milliseconds or a human duration (e.g. 5s). The minimum is 250ms, and defaults to 1000ms. Smaller values may take more computer resources.");
 
     let time_delta = Arg::new("time_delta")
         .short('d')
@@ -362,7 +362,7 @@ use CPU (3) as the default instead.
         .action(ArgAction::Set)
         .value_name("TIME")
         .help("The amount of time changed upon zooming.")
-        .long_help("The amount of time changed when zooming in/out. The minimum is 1s, and defaults to 15s.");
+        .long_help("The amount of time changed when zooming in/out. Takes a number in milliseconds or a human duration (e.g. 30s). The minimum is 1s, and defaults to 15s.");
 
     let tree = Arg::new("tree")
         .short('T')
@@ -395,8 +395,8 @@ use CPU (3) as the default instead.
         .long("retention")
         .action(ArgAction::Set)
         .value_name("TIME")
-        .help("The timespan of data kept.")
-        .long_help("How much data is stored at once in terms of time. Takes in human-readable time spans (e.g. 10m, 1h), with a minimum of 1 minute. Note higher values will take up more memory. Defaults to 10 minutes.");
+        .help("The timespan of data stored.")
+        .long_help("How much data is stored at once in terms of time. Takes a number in milliseconds or a human duration (e.g. 20m), with a minimum of 1 minute. Note higher values will take up more memory. Defaults to 10 minutes.");
 
     let version = Arg::new("version")
         .short('V')
diff --git a/src/options.rs b/src/options.rs
index 412748a6..26b21d7f 100644
--- a/src/options.rs
+++ b/src/options.rs
@@ -102,9 +102,7 @@ pub struct ConfigFlags {
     network_use_binary_prefix: Option<bool>,
     enable_gpu_memory: Option<bool>,
     enable_cache_memory: Option<bool>,
-    #[serde(with = "humantime_serde")]
-    #[serde(default)]
-    retention: Option<Duration>,
+    retention: Option<StringOrNum>,
 }
 
 #[derive(Clone, Debug, Default, Deserialize, Serialize)]
@@ -200,7 +198,7 @@ pub fn build_app(
     let config = &config;
 
     let retention_ms =
-        get_retention_ms(matches, config).context("Update `retention` in your config file.")?;
+        get_retention(matches, config).context("Update `retention` in your config file.")?;
     let autohide_time = is_flag_enabled!(autohide_time, matches, config);
     let default_time_value = get_default_time_value(matches, config, retention_ms)
         .context("Update 'default_time_value' in your config file.")?;
@@ -887,16 +885,17 @@ fn get_network_scale_type(matches: &ArgMatches, config: &Config) -> AxisScaling
     AxisScaling::Linear
 }
 
-fn get_retention_ms(matches: &ArgMatches, config: &Config) -> error::Result<u64> {
+fn get_retention(matches: &ArgMatches, config: &Config) -> error::Result<u64> {
     const DEFAULT_RETENTION_MS: u64 = 600 * 1000; // Keep 10 minutes of data.
 
     if let Some(retention) = matches.get_one::<String>("retention") {
-        humantime::parse_duration(retention)
-            .map(|dur| dur.as_millis() as u64)
-            .map_err(|err| BottomError::ConfigError(format!("invalid retention duration: {err:?}")))
+        try_parse_ms(retention)
     } else if let Some(flags) = &config.flags {
-        if let Some(retention) = flags.retention {
-            Ok(retention.as_millis() as u64)
+        if let Some(retention) = &flags.retention {
+            Ok(match retention {
+                StringOrNum::String(s) => try_parse_ms(s)?,
+                StringOrNum::Num(n) => *n,
+            })
         } else {
             Ok(DEFAULT_RETENTION_MS)
         }
@@ -913,7 +912,9 @@ mod test {
     use crate::{
         app::App,
         canvas::canvas_styling::CanvasStyling,
-        options::{get_default_time_value, get_update_rate, try_parse_ms, ConfigFlags},
+        options::{
+            get_default_time_value, get_retention, get_update_rate, try_parse_ms, ConfigFlags,
+        },
     };
 
     #[test]
@@ -999,6 +1000,7 @@ mod test {
             time_delta: Some("2 min".to_string().into()),
             default_time_value: Some("300s".to_string().into()),
             rate: Some("1s".to_string().into()),
+            retention: Some("10m".to_string().into()),
             ..Default::default()
         };
 
@@ -1015,6 +1017,8 @@ mod test {
         );
 
         assert_eq!(get_update_rate(&matches, &config), Ok(1000));
+
+        assert_eq!(get_retention(&matches, &config), Ok(600000));
     }
 
     #[test]
@@ -1027,6 +1031,7 @@ mod test {
             time_delta: Some("120000".to_string().into()),
             default_time_value: Some("300000".to_string().into()),
             rate: Some("1000".to_string().into()),
+            retention: Some("600000".to_string().into()),
             ..Default::default()
         };
 
@@ -1043,6 +1048,8 @@ mod test {
         );
 
         assert_eq!(get_update_rate(&matches, &config), Ok(1000));
+
+        assert_eq!(get_retention(&matches, &config), Ok(600000));
     }
 
     #[test]
@@ -1055,6 +1062,7 @@ mod test {
             time_delta: Some(120000.into()),
             default_time_value: Some(300000.into()),
             rate: Some(1000.into()),
+            retention: Some(600000.into()),
             ..Default::default()
         };
 
@@ -1071,6 +1079,8 @@ mod test {
         );
 
         assert_eq!(get_update_rate(&matches, &config), Ok(1000));
+
+        assert_eq!(get_retention(&matches, &config), Ok(600000));
     }
 
     fn create_app(config: Config, matches: ArgMatches) -> App {