Skip to content

馃悰 Bug: Fix misleading model mappings and incorrect comments#5

Description

@ManvithGopu13

Code Issues

Problems Found:

  1. Misleading Model Mappings

    • All model options map to the same actual model:
      • Image route: All map to stabilityai/sd-turbo
      • Audio route: All map to microsoft/speecht5_tts
    • Users expect different models but get the same one
  2. Incorrect Comments

    • routes/audio/audio.py:16: Says "Video generation request received" (should be "Audio")
    • routes/audio/audio.py:38: Comment says "Generate image" (should be "Generate audio")
    • routes/audio/audio.py:43: Comment says "Convert image to base64" (should be "Convert audio")
    • routes/audio/audio.py:50: Comment says "Return base64 image data" (should be "audio")
  3. Wrong Default Model

    • routes/audio/audio.py:13: Default model is 'stabilityai/stable-diffusion-2-1' (image model) instead of audio model
  4. Commented Out Code

    • routes/audio/audio.py:2: Unused import commented out
    • functions/audio/audio.py:31-43: Large commented code block should be removed

Recommended Fixes:

  • Implement actual different model mappings or remove misleading options
  • Fix all incorrect comments
  • Fix default model in audio route
  • Remove commented code or document why it's kept

Priority: Medium
Type: Bug

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions