Skip to content

Conversation

@tschm
Copy link
Member

@tschm tschm commented Aug 4, 2025

No description provided.

@tschm tschm linked an issue Aug 4, 2025 that may be closed by this pull request
@tschm tschm requested a review from Copilot August 4, 2025 15:26

This comment was marked as outdated.

@tschm tschm requested a review from Copilot August 4, 2025 15:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the dependency management approach by migrating from traditional requirements.txt files to uv's inline script dependencies feature. The change enables running experiments without manually setting up a virtual environment or installing dependencies from requirements files.

  • Removes frozen requirements file and replaces pip-based dependency management with uv's script metadata
  • Updates the experiments script to include inline dependency declarations using PEP 723 format
  • Simplifies the Makefile by removing installation steps and updating experiment execution

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
requirements_frozen.txt Completely removes the frozen dependencies file
experiments.py Adds script metadata header with dependency declarations and usage instructions
README.md Updates documentation to reflect the new uv-based workflow
Makefile Removes install and freeze targets, simplifies experiments target
.python-version Removes Python version specification file

@tschm tschm requested review from kasperjo and phschiele August 4, 2025 15:52
@tschm tschm force-pushed the 286-run-without-requirementstxt-install-dependencies-in-a-header-for-uv branch from 77a1402 to 7f5dea7 Compare August 11, 2025 19:42
@tschm tschm merged commit ffaeb0d into main Aug 11, 2025
2 checks passed
@tschm tschm deleted the 286-run-without-requirementstxt-install-dependencies-in-a-header-for-uv branch August 11, 2025 19:43
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.

Run without requirements.txt, install dependencies in a header for uv

2 participants