Repository: conbench/conbench (language: python)

Issue:
Enhancement: Units of data Bytes not available, only Bytes/second 
Benchmarks library asv uses seconds or Bytes as units. Conbench does not give the option to use "Bytes". We have to use Bytes/second. Can just "Bytes" be added?

![Screenshot 2023-11-27 at 09 53 46](https://github.com/conbench/conbench/assets/11835246/cc126737-bb11-4e1b-94be-4ce65a10d0c3)

In case this helps (this is temporary): 
http://15.235.45.255:5000/


Discussion / hints:
Yes! I think this would be a rather simple change to https://github.com/conbench/conbench/blob/main/conbench/units.py.

Fix patch:
diff --git a/conbench/units.py b/conbench/units.py
index 03c75802..04e02409 100644
--- a/conbench/units.py
+++ b/conbench/units.py
@@ -30,7 +30,7 @@
 # These are the symbols. We could call this type UnitSymbol, but we can
 # have the convention that whenever "unit" pops up in the code base it means
 # the short version, the symbol.
-TUnit = Literal["B/s", "s", "ns", "i/s"]
+TUnit = Literal["B", "B/s", "s", "ns", "i/s"]
 
 
 class TUnitDef(TypedDict):
@@ -40,6 +40,10 @@ class TUnitDef(TypedDict):
 
 # slash ("/"") or "per"? We will see.
 KNOWN_UNITS: Dict[TUnit, TUnitDef] = {
+    "B": {
+        "long": "bytes",
+        "less_is_better": True,
+    },
     "B/s": {
         "long": "bytes per second",
         "less_is_better": False,


Test patch:
diff --git a/conbench/tests/api/_expected_docs.py b/conbench/tests/api/_expected_docs.py
index 22673208..562dfb1d 100644
--- a/conbench/tests/api/_expected_docs.py
+++ b/conbench/tests/api/_expected_docs.py
@@ -1527,7 +1527,7 @@
                         "type": "array",
                     },
                     "unit": {
-                        "description": "Unit of the numbers in `data`. Allowed values: B/s, s, ns, i/s",
+                        "description": "Unit of the numbers in `data`. Allowed values: B, B/s, s, ns, i/s",
                         "type": "string",
                     },
                 },