Skip to content

Request: include the "dev" dependencies in the Docker container #162

@jsta

Description

@jsta

Without these you can't run pytest (even with a -k downselect) due to the following chunk in pyproject.toml:

[tool.pytest.ini_options]
asyncio_mode = "auto"
addopts = "--numprocesses auto --dist worksteal"

adding the dev dependencies "on-the-fly" can be challenging due to the root install of the uv environment. Because ursa-ai is being installed from wheel, I don't see a way to do this as part of the Dockerfile recipe alone. My proposal is to duplicate the following:

[dependency-groups]
dev = [
    "fastmcp>=2.13.3",
    "notebook>=7.3.3",
    "pre-commit>=4.3.0",
    "pytest>=8.4.2",
    "pytest-asyncio>=1.3.0",
    "pytest-xdist",
    "scikit-optimize>=0.10.2",
]

into:

[project.optional-dependencies]
dev = [
    "fastmcp>=2.13.3",
    "notebook>=7.3.3",
    "pre-commit>=4.3.0",
    "pytest>=8.4.2",
    "pytest-asyncio>=1.3.0",
    "pytest-xdist",
    "scikit-optimize>=0.10.2",
]

and

# in Dockerfile
RUN uv add "ursa-ai[dev] @ file:///`ls /wheelhouse/*.whl`"

unless you know of a way to avoid this duplication. PR incoming for your consideration.

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