Skip to content

Refactor: Extract shared constants, populate __init__.py, fix Python version constraint#63

Merged
jhauret merged 2 commits intomainfrom
copilot/refactor-repo-structure
Nov 29, 2025
Merged

Refactor: Extract shared constants, populate __init__.py, fix Python version constraint#63
jhauret merged 2 commits intomainfrom
copilot/refactor-repo-structure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 28, 2025

Repository cleanup to improve code organization and maintainability.

Changes

  • Python version constraint: Relaxed requires-python = "==3.12.0"">=3.12.0" to allow installation on any 3.12.x
  • Duplicate import: Removed duplicate from lightning import LightningDataModule in noisybwe.py
  • Shared constants: Extracted LIST_OF_VIBRAVOX from bwe.py and stp.py into vibravox/constants.py
  • Public API exports: Populated all __init__.py files with proper exports

New imports enabled

from vibravox import LIST_OF_VIBRAVOX, AVAILABLE_SENSORS
from vibravox.lightning_datamodules import BWELightningDataModule
from vibravox.lightning_modules import EBENLightningModule
from vibravox.metrics import NoresqaMOS
from vibravox.torch_modules.dnn import EBENGenerator
Original prompt

refactor the repo


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ract shared constants, populate __init__.py files

Co-authored-by: jhauret <53187038+jhauret@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor the repository structure for better organization Refactor: Extract shared constants, populate __init__.py, fix Python version constraint Nov 28, 2025
Copilot AI requested a review from jhauret November 28, 2025 14:56
@jhauret jhauret marked this pull request as ready for review November 29, 2025 12:50
@jhauret jhauret merged commit 452276a into main Nov 29, 2025
1 check passed
@jhauret jhauret deleted the copilot/refactor-repo-structure branch December 1, 2025 17:29
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