From b98cde15f68a13fe0caec07d83d1a89830193dab Mon Sep 17 00:00:00 2001
From: Nico Wagner <n.wagner@dnb.de>
Date: Sun, 23 Jul 2023 13:00:03 +0200
Subject: [PATCH] Change `--threshold` behavior of `frequency` command (#674)

---
 CHANGELOG.md                                  | 11 ++++++-----
 docs/book/src/referenz/kommandos/frequency.md |  9 +++++----
 pica-toolkit/src/commands/frequency.rs        |  6 +++---
 3 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 05b67c52e..8e0480f38 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -9,13 +9,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
-* #637 Stabilize `print` command
-* #641 Stabilize `sample` command
-* #642 Add `--squash` and `--merge` option
-* #644 Add `!^` and `!$` operator
+* #674 Change `--threshold` behavior (`frequency` command)
 * #658 Add unique-strategy config option (`cat` command)
-* #672 Stabilize `select` command
 * #673 Add contains relation matcher (`=?`)
+* #644 Add `!^` and `!$` operator
+* #642 Add `--squash` and `--merge` option
+* #641 Stabilize `sample` command
+* #637 Stabilize `print` command
+* #672 Stabilize `select` command
 
 ### Changed
 
diff --git a/docs/book/src/referenz/kommandos/frequency.md b/docs/book/src/referenz/kommandos/frequency.md
index deac65e73..9f65335b9 100644
--- a/docs/book/src/referenz/kommandos/frequency.md
+++ b/docs/book/src/referenz/kommandos/frequency.md
@@ -39,7 +39,7 @@ Ts1,1
   ausgegeben.
 * `-l`, `--limit` `<n>` — Eingrenzung der Ausgabe auf die häufigsten _n_
   Unterfeldwerte.
-* `--threshold` `<t>` — Zeilen mit einer Häufigkeit ≤ _t_ ignorieren.
+* `--threshold` `<n>` — Zeilen mit einer Häufigkeit < `<n>` ignorieren.
 * `-H`, `--header` `<header>` — Kopfzeile, die den Ergebnissen
   vorangestellt wird.
 * `-t`, `--tsv` — Ausgabe erfolgt im TSV-Format.
@@ -127,14 +127,15 @@ vbal,4
 ### Eingrenzen der Treffermenge (Schwellenwert)
 
 Die Treffermenge kann auch mittels der Angabe eines Schwellenwerts
-eingeschänkt werden. Sollen nur die Werte angezeigt werden, die häufiger
-als ein Schwellenwert _n_ vorkommen, dann kann dies mit der Option
-`--threshold` bzw. `-t` erzielt werden:
+eingeschänkt werden. Sollen nur die Werte angezeigt werden, die ab einem
+Schwellenwert vorkommen, dann kann dies mit der Option
+`--threshold`/`-t` erzielt werden:
 
 ```console
 $ pica frequency -s --threshold 4 "041R.4" DUMP.dat.gz
 beru,12
 obal,5
+vbal,4
 
 ```
 
diff --git a/pica-toolkit/src/commands/frequency.rs b/pica-toolkit/src/commands/frequency.rs
index 850ba896f..109f0e624 100644
--- a/pica-toolkit/src/commands/frequency.rs
+++ b/pica-toolkit/src/commands/frequency.rs
@@ -66,10 +66,10 @@ pub(crate) struct Frequency {
     )]
     limit: usize,
 
-    /// Ignore rows with a frequency ≤ <t>.
+    /// Ignore rows with a frequency < <value>.
     #[arg(
         long,
-        value_name = "t",
+        value_name = "value",
         default_value = "0",
         hide_default_value = true
     )]
@@ -185,7 +185,7 @@ impl Frequency {
                 break;
             }
 
-            if **frequency <= self.threshold {
+            if **frequency < self.threshold {
                 break;
             }