Skip to content

Switched to uv #32

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
clstaudt wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Switched to uv #32

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
2 changes: 1 addition & 1 deletion .python-version
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3.9.12
3.11
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

The Python version has been updated from 3.9.12 to 3.11. Consider specifying a more precise version (e.g., 3.11.0 or a specific patch version) to ensure consistency across development environments, especially if you're using features or fixes from a particular Python 3.11 release.

Suggested change
3.11
3.11.0

Copilot uses AI. Check for mistakes.
26 changes: 13 additions & 13 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,33 +26,33 @@ Contributions welcome! Have a look at [CONTRIBUTING.md](CONTRIBUTING.md)

## Development

We use [`pyenv`](https://github.com/pyenv/pyenv) and [`poetry`](https://github.com/python-poetry/poetry) to setup the python development environment.
We use [`uv`](https://docs.astral.sh/uv/) to manage the Python environment and dependencies.

After installing those two, run the following lines to be ready to develop:
After [installing uv](https://docs.astral.sh/uv/getting-started/installation/), run the following to set up the project:

```
pyenv install -s "$(cat .python-version)"
poetry env use "$(cat .python-version)"
poetry install
uv sync
```

Now you can start your `jupyter-lab` UI with
This will automatically install the required Python version and all dependencies.

To make the environment available as a Jupyter kernel, run:

```
poetry run jupyter-lab
uv run python -m ipykernel install --user --name data-science-learning-paths --display-name "Data Science Learning Paths"
```
Comment on lines +39 to 43
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

The ipykernel package is not explicitly listed in the project dependencies, but the README instructs users to run uv run python -m ipykernel install. While ipykernel is likely installed as a transitive dependency of jupyter or jupyterlab, it would be more explicit and clear to add it as a direct dependency to ensure it's available and to make the dependency on this feature explicit.

Copilot uses AI. Check for mistakes.

or a classic `notebook` view with
Now you can start your `jupyter-lab` UI with

```
poetry run jupyter notebook
uv run jupyter lab
```

### Working with on Apple Silicon M1 hardware

As there might occur some problems using the preferred setup on M1 hardware, you could also use [`conda`](https://github.com/conda-forge/miniforge).
or a classic `notebook` view with

As the leading system is poetry, please update the requirements file according to the dependencies in the `pyproject.toml`.
```
uv run jupyter notebook
```

## Work with us

Expand Down
13 changes: 13 additions & 0 deletions library/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
[project]
name = "data-science-learning-paths-library"
version = "0.2.0"
description = "Helper library for Data Science Learning Paths"
requires-python = ">=3.11"
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

The Python version requirement in the main pyproject.toml is >=3.11,<3.14, but the library's pyproject.toml only specifies >=3.11 without an upper bound. These should be consistent to avoid potential compatibility issues. Consider updating the library's requires-python to match the main project: >=3.11,<3.14.

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

Copilot uses AI. Check for mistakes.
dependencies = []

[build-system]
requires = ["hatchling"]
build-backend = "hatchling.build"

[tool.hatch.build.targets.wheel]
packages = ["data_science_learning_paths"]
11 changes: 0 additions & 11 deletions library/setup.py
View file
Open in desktop

This file was deleted.

Loading