Skip to content

refactor(models): standardize outputs to structured dictionary format and added config validation.#32

Open
VedaSiddhartha010 wants to merge 2 commits intoruxailab:mainfrom
VedaSiddhartha010:refactor/model-output-dict
Open

refactor(models): standardize outputs to structured dictionary format and added config validation.#32
VedaSiddhartha010 wants to merge 2 commits intoruxailab:mainfrom
VedaSiddhartha010:refactor/model-output-dict

Conversation

@VedaSiddhartha010
Copy link
Copy Markdown

This PR standardizes model outputs across the repository by replacing tuple-based returns with structured dictionaries.

Changes : ---

  • Updated Whisper Transcript to return {"text", "chunks"} instead of tuple
  • Updated Bertweet sentiment model to return {"label", "score", "probabilities"}
  • Refactored dependent service layers to handle dict-based outputs
  • Added config validation checks to prevent runtime errors

Benefits: ---

  • Improved readability and maintainability
  • Consistent API design across models
  • Easier integration with downstream services and pipelines

Notes : ---

  • No functional changes to model predictions
  • Only output structure has been modified

@VedaSiddhartha010
Copy link
Copy Markdown
Author

@BasmaElhoseny01 Ma'am can you review this

@BasmaElhoseny01 BasmaElhoseny01 self-assigned this Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants