fixes from copilot review

Signed-off-by: Mark Kurtz <[email protected]>

Author: markurtz

src/guidellm/benchmark/progress.py

@@ -253,7 +253,7 @@ def format_progress_display(
         decimal_places: Optional[int] = None,
     ) -> str:
         if decimal_places is None and digits_places is None:
-            formatted_number = f"{value}:.0f"
+            formatted_number = f"{value:.0f}"
         elif digits_places is None:
             formatted_number = f"{value:.{decimal_places}f}"
         elif decimal_places is None:

src/guidellm/utils/__init__.py

@@ -58,7 +58,7 @@
     clean_text,
     filter_text,
     format_value_display,
-    is_puncutation,
+    is_punctuation,
     load_text,
     split_text,
     split_text_list_by_length,
@@ -109,7 +109,7 @@
     "filter_text",
     "format_value_display",
     "get_literal_vals",
-    "is_puncutation",
+    "is_punctuation",
     "load_text",
     "safe_add",
     "safe_divide",

src/guidellm/utils/synchronous.py

@@ -11,7 +11,7 @@
 
 import asyncio
 import contextlib
-from datetime import time
+import time
 from multiprocessing.synchronize import Barrier as ProcessingBarrier
 from multiprocessing.synchronize import Event as ProcessingEvent
 from threading import Barrier as ThreadingBarrier

src/guidellm/utils/text.py

@@ -31,7 +31,7 @@
     "clean_text",
     "filter_text",
     "format_value_display",
-    "is_puncutation",
+    "is_punctuation",
     "load_text",
     "split_text",
     "split_text_list_by_length",
@@ -64,7 +64,7 @@ def format_value_display(
     :return: Formatted string with value, units, and colored label
     """
     if decimal_places is None and digits_places is None:
-        formatted_number = f"{value}:.0f"
+        formatted_number = f"{value:.0f}"
     elif digits_places is None:
         formatted_number = f"{value:.{decimal_places}f}"
     elif decimal_places is None:
@@ -268,7 +268,7 @@ def load_text(data: str | Path, encoding: str | None = None) -> str:
     return data.read_text(encoding=encoding)
 
 
-def is_puncutation(text: str) -> bool:
+def is_punctuation(text: str) -> bool:
     """
     Check if a single character is a punctuation mark.
 
@@ -332,7 +332,7 @@ def create_text(self, start: int, length: int) -> str:
             index = (start + counter) % len(self.words)
             add_word = self.words[index]
 
-            if counter != 0 and not is_puncutation(add_word):
+            if counter != 0 and not is_punctuation(add_word):
                 text += " "
 
             text += add_word

tests/unit/utils/test_text.py

@@ -14,7 +14,7 @@
     clean_text,
     filter_text,
     format_value_display,
-    is_puncutation,
+    is_punctuation,
     load_text,
     split_text,
     split_text_list_by_length,
@@ -372,8 +372,8 @@ def test_url_error(self, mock_client):
             load_text("http://example.com/error.txt")
 
 
-class TestIsPuncutation:
-    """Test suite for is_puncutation."""
+class TestIsPunctuation:
+    """Test suite for is_punctuation."""
 
     @pytest.mark.smoke
     @pytest.mark.parametrize(
@@ -392,8 +392,8 @@ class TestIsPuncutation:
         ],
     )
     def test_invocation(self, text, expected):
-        """Test is_puncutation with various characters."""
-        result = is_puncutation(text)
+        """Test is_punctuation with various characters."""
+        result = is_punctuation(text)
         assert result == expected
 
     @pytest.mark.sanity
@@ -405,9 +405,9 @@ def test_invocation(self, text, expected):
         ],
     )
     def test_invalid_invocation(self, text):
-        """Test is_puncutation with invalid inputs."""
+        """Test is_punctuation with invalid inputs."""
         with pytest.raises((TypeError, AttributeError)):
-            is_puncutation(text)
+            is_punctuation(text)
 
 
 class TestEndlessTextCreator: