Skip to content

Update project metadata and UV index configuration #1047

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
MH0386 wants to merge 9 commits into
base: main
Choose a base branch
from
+547 −419
Open

Update project metadata and UV index configuration #1047

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
f0e0eaf
Remove explicit flag from UV index configuration in pyproject.toml
MH0386 Jan 12, 2026
b5c113e
Remove Python 3.12 classifier from project metadata in pyproject.toml
MH0386 Jan 12, 2026
5ed47d9
Update Python version requirement to 3.13 in pyproject.toml
MH0386 Jan 12, 2026
35fd0cc
Update uv.lock
MH0386 Jan 12, 2026
7044ce5
Merge branch 'main' into uv-sources
MH0386 Jan 12, 2026
d00536b
Add .vscode/settings.json to .gitignore
MH0386 Jan 12, 2026
a5b9a63
Merge branch 'uv-sources' of https://github.com/AlphaSphereDotAI/voca…
MH0386 Jan 12, 2026
a2025c3
Merge branch 'main' into uv-sources
mergify[bot] Jan 12, 2026
49a6724
Merge branch 'main' into uv-sources
mergify[bot] Jan 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,5 @@
.venv/
logs/
results/
pycrucible_payload/
pycrucible_payload/
.vscode/settings.json
8 changes: 1 addition & 7 deletions pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,13 @@ name = "vocalizr"
version = "0.1.0"
description = "Voice Generator part of the Chatacter Backend"
readme = "README.md"
requires-python = ">=3.12, <3.13"
requires-python = ">=3.13,<3.14"
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

cat pyproject.toml | head -50

Repository: AlphaSphereDotAI/vocalizr

Length of output: 1304

🏁 Script executed:

#!/bin/bash
# Check Python 3.13 compatibility for key dependencies from PyPI

echo "=== Checking kokoro Python compatibility ==="
curl -s https://pypi.org/pypi/kokoro/json | jq -r '.info.requires_python // "Not specified"'
curl -s https://pypi.org/pypi/kokoro/json | jq -r '.info.classifiers[] | select(contains("Python"))' 2>/dev/null | head -10

echo -e "\n=== Checking gradio Python compatibility ==="
curl -s https://pypi.org/pypi/gradio/json | jq -r '.info.requires_python // "Not specified"'
curl -s https://pypi.org/pypi/gradio/json | jq -r '.info.classifiers[] | select(contains("Python"))' 2>/dev/null | head -10

echo -e "\n=== Checking soundfile Python compatibility ==="
curl -s https://pypi.org/pypi/soundfile/json | jq -r '.info.requires_python // "Not specified"'
curl -s https://pypi.org/pypi/soundfile/json | jq -r '.info.classifiers[] | select(contains("Python"))' 2>/dev/null | head -10

Repository: AlphaSphereDotAI/vocalizr

Length of output: 734

kokoro does not support Python 3.13 and will fail with the current requirement.

The Python 3.13 requirement is incompatible with the kokoro dependency. PyPI metadata shows kokoro requires <3.13,>=3.10, which explicitly excludes Python 3.13. Either downgrade the Python requirement to >=3.10,<3.13 or wait for kokoro to release a version that supports Python 3.13. gradio and soundfile are compatible with Python 3.13.

🤖 Prompt for AI Agents 
In @pyproject.toml at line 6, Update the project's Python requirement to exclude
3.13 so kokoro is satisfied: change the requires-python spec in pyproject.toml
from ">=3.13,<3.14" to ">=3.10,<3.13" (or an equivalent range that includes
3.10–3.12) so the kokoro dependency constraint (<3.13,>=3.10) is honored.

Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spacing around the version constraint has been changed from ">=3.12, <3.13" to ">=3.13,<3.14" (removed space after comma). For consistency with common Python packaging conventions and PEP 440, it's recommended to include a space after the comma in version specifiers.

Suggested change
requires-python = ">=3.13,<3.14"
requires-python = ">=3.13, <3.14"

Copilot uses AI. Check for mistakes.
authors = [
{ name = "Mohamed Hisham Abdelzaher", email = "mohamed.hisham.abdelzaher@gmail.com" },
]
keywords = ["audio", "kokoro", "voice"]
classifiers = [
"Environment :: GPU :: NVIDIA CUDA :: 12 :: 12.4",
"Programming Language :: Python :: 3.12",
"Programming Language :: Python :: 3.13",
Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Python version classifier is inconsistent with the project's actual Python version requirements. The project specifies requires-python = ">=3.12, <3.13" on line 6, which means it only supports Python 3.12. However, the classifier now lists Python 3.13 without listing Python 3.12. Either the Python 3.12 classifier should be kept and the 3.13 classifier removed, or the requires-python constraint should be updated to support Python 3.13.

Suggested change
"Programming Language :: Python :: 3.13",
"Programming Language :: Python :: 3.12",

Copilot uses AI. Check for mistakes.
"Topic :: Scientific/Engineering :: Artificial Intelligence",
]
Comment on lines 11 to 15
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

There appears to be a mismatch between the requires-python setting and the classifiers. The requires-python is set to >=3.12, <3.13, which indicates the project supports Python 3.12 but not 3.13. The uv.lock file also specifies requires-python = "==3.12.*", reinforcing that Python 3.12 is the target version.

However, this change removes the Programming Language :: Python :: 3.12 classifier while keeping the incorrect Programming Language :: Python :: 3.13 classifier. To ensure the project metadata is accurate, I recommend restoring the Python 3.12 classifier and removing the one for Python 3.13.

Suggested change
classifiers = [
"Environment :: GPU :: NVIDIA CUDA :: 12 :: 12.4",
"Programming Language :: Python :: 3.12",
"Programming Language :: Python :: 3.13",
"Topic :: Scientific/Engineering :: Artificial Intelligence",
]
classifiers = [
"Environment :: GPU :: NVIDIA CUDA :: 12 :: 12.4",
"Programming Language :: Python :: 3.12",
"Topic :: Scientific/Engineering :: Artificial Intelligence",
]

Expand Down Expand Up @@ -49,12 +48,7 @@ dev = [
"uv-build>=0.9.11",
]

[tool.uv.sources]
torch = [{ index = "pytorch-cu124" }]
torchaudio = [{ index = "pytorch-cu124" }]

[[tool.uv.index]]
explicit = true
name = "pytorch-cu124"
url = "https://download.pytorch.org/whl/cu124"

Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the [tool.uv.sources] section and the explicit = true flag may cause PyTorch to be installed from the default PyPI index instead of the CUDA-specific pytorch-cu124 index. The project has a classifier indicating it requires "NVIDIA CUDA :: 12 :: 12.4", but without the sources configuration, uv may not install the correct CUDA-enabled PyTorch builds. The lockfile currently shows torch being installed from PyPI, which may not include CUDA support. Consider whether this change is intentional, and if CUDA support is required, the sources configuration should be retained.

Suggested change
[tool.uv.sources]
torch = { index = "pytorch-cu124", explicit = true }

Copilot uses AI. Check for mistakes.
Expand Down
Loading
Loading