Zg/improve logging (#8)

* reduce character description and text max length to 400 from 1000

* Reduce number of retries for tts calls to a single retry, and a 2 second wait before retrying

* Structure error logs for easier parsing

src/constants.py

@@ -24,10 +24,10 @@ HUME_TO_HUME: ComparisonType = "Hume AI - Hume AI"
 HUME_TO_ELEVENLABS: ComparisonType = "Hume AI - ElevenLabs"
 
 CHARACTER_DESCRIPTION_MIN_LENGTH: int = 20
-CHARACTER_DESCRIPTION_MAX_LENGTH: int = 1000
+CHARACTER_DESCRIPTION_MAX_LENGTH: int = 400
 
 TEXT_MIN_LENGTH: int = 100
-TEXT_MAX_LENGTH: int = 1000
+TEXT_MAX_LENGTH: int = 400
 
 OPTION_A_KEY: OptionKey = "option_a"
 OPTION_B_KEY: OptionKey = "option_b"

src/frontend.py

@@ -73,11 +73,11 @@ class Frontend:
             logger.info(f"Generated text ({len(generated_text)} characters).")
             return gr.update(value=generated_text), generated_text
         except AnthropicError as ae:
-            logger.error(f"AnthropicError while generating text: {ae!s}")
+            logger.error(f"Text Generation Failed: AnthropicError while generating text: {ae!s}")
             raise gr.Error(f'There was an issue communicating with the Anthropic API: "{ae.message}"')
         except Exception as e:
-            logger.error(f"Unexpected error while generating text: {e}")
-            raise gr.Error("Failed to generate text. Please try again later.")
+            logger.error(f"Text Generation Failed: Unexpected error while generating text: {e!s}")
+            raise gr.Error("Failed to generate text. Please try again shortly.")
 
     async def _synthesize_speech(
         self,
@@ -143,6 +143,7 @@ class Frontend:
 
             # Await both tasks concurrently using asyncio.gather()
             (generation_id_a, audio_a), (generation_id_b, audio_b) = await asyncio.gather(task_a, task_b)
+            logger.info(f"Synthesis Succeed for providers: {provider_a} and {provider_b}")
 
             option_a = Option(provider=provider_a, audio=audio_a, generation_id=generation_id_a)
             option_b = Option(provider=provider_b, audio=audio_b, generation_id=generation_id_b)
@@ -158,14 +159,14 @@ class Frontend:
                 True,
             )
         except ElevenLabsError as ee:
-            logger.error(f"ElevenLabsError while synthesizing speech from text: {ee!s}")
+            logger.error(f"Synthesis Failed with ElevenLabsError during TTS generation: {ee!s}")
             raise gr.Error(f'There was an issue communicating with the Elevenlabs API: "{ee.message}"')
         except HumeError as he:
-            logger.error(f"HumeError while synthesizing speech from text: {he!s}")
+            logger.error(f"Synthesis Failed with HumeError during TTS generation: {he!s}")
             raise gr.Error(f'There was an issue communicating with the Hume API: "{he.message}"')
         except Exception as e:
-            logger.error(f"Unexpected error during TTS generation: {e}")
-            raise gr.Error("An unexpected error occurred. Please try again later.")
+            logger.error(f"Synthesis Failed with an unexpected error during TTS generation: {e!s}")
+            raise gr.Error("An unexpected error occurred. Please try again shortly.")
 
     async def _vote(
         self,

src/integrations/elevenlabs_api.py

@@ -22,7 +22,7 @@ from typing import Optional, Tuple
 # Third-Party Library Imports
 from elevenlabs import AsyncElevenLabs, TextToVoiceCreatePreviewsRequestOutputFormat
 from elevenlabs.core import ApiError
-from tenacity import after_log, before_log, retry, retry_if_exception, stop_after_attempt, wait_exponential
+from tenacity import after_log, before_log, retry, retry_if_exception, stop_after_attempt, wait_fixed
 
 # Local Application Imports
 from src.config import Config, logger
@@ -76,8 +76,8 @@ class UnretryableElevenLabsError(ElevenLabsError):
 
 @retry(
     retry=retry_if_exception(lambda e: not isinstance(e, UnretryableElevenLabsError)),
-    stop=stop_after_attempt(3),
-    wait=wait_exponential(multiplier=1, min=2, max=5),
+    stop=stop_after_attempt(2),
+    wait=wait_fixed(2),
     before=before_log(logger, logging.DEBUG),
     after=after_log(logger, logging.DEBUG),
     reraise=True,

src/integrations/hume_api.py

@@ -21,7 +21,7 @@ from typing import Tuple, Union
 from hume import AsyncHumeClient
 from hume.core.api_error import ApiError
 from hume.tts.types import Format, FormatMp3, PostedUtterance, ReturnTts
-from tenacity import after_log, before_log, retry, retry_if_exception, stop_after_attempt, wait_exponential
+from tenacity import after_log, before_log, retry, retry_if_exception, stop_after_attempt, wait_fixed
 
 # Local Application Imports
 from src.config import Config, logger
@@ -79,8 +79,8 @@ class UnretryableHumeError(HumeError):
 
 @retry(
     retry=retry_if_exception(lambda e: not isinstance(e, UnretryableHumeError)),
-    stop=stop_after_attempt(3),
-    wait=wait_exponential(multiplier=1, min=2, max=5),
+    stop=stop_after_attempt(2),
+    wait=wait_fixed(2),
     before=before_log(logger, logging.DEBUG),
     after=after_log(logger, logging.DEBUG),
     reraise=True,