Skip to content

Exclude packages that are newer than 7 days for installation#771

Open
AVHopp wants to merge 1 commit intomainfrom
repo/exclude_newer_packages
Open

Exclude packages that are newer than 7 days for installation#771
AVHopp wants to merge 1 commit intomainfrom
repo/exclude_newer_packages

Conversation

@AVHopp
Copy link
Copy Markdown
Collaborator

@AVHopp AVHopp commented Mar 30, 2026

This PR improves the configuration of BayBE such that only package version older than 7 days can be installed.

I have tested this locally and can confirm that it works: I've attempted to raise the version of uv itself in this PR, which was not possible as the latest version of uv is only 4 days old. I've also done a check using a value of 100 days, which results in an error that the necessary version 0.14.11. of ruff cannot be installed. Since 100 days ago was December 20, 2025, this also checks out:
image

@AVHopp AVHopp self-assigned this Mar 30, 2026
Copilot AI review requested due to automatic review settings March 30, 2026 12:04
@AVHopp AVHopp added the repo Requires changes to the project configuration label Mar 30, 2026
@AVHopp AVHopp force-pushed the repo/exclude_newer_packages branch from ea78bd1 to 695cb14 Compare March 30, 2026 12:05
Copy link
Copy Markdown
Contributor

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 updates the repository’s uv configuration to prevent installing package versions that are newer than 7 days, helping stabilize dependency resolution and avoid very recent releases.

Changes:

  • Add a uv configuration block to exclude packages newer than 7 days.
  • Minor formatting adjustment in the dev optional dependency list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AVHopp
Copy link
Copy Markdown
Collaborator Author

AVHopp commented Mar 30, 2026

This is currently blocked by #772

@AVHopp AVHopp added the on hold PR progress is awaiting for something else to continue label Mar 30, 2026
AVHopp added a commit that referenced this pull request Apr 1, 2026
Release `1.24.1` of `onnxruntime` dropped support for Python 3.10, see
https://github.com/microsoft/onnxruntime/releases/tag/v1.24.1.
Unfortunately, they did not properly update their metadata, which causes
issues when trying to install the package as `uv` *thinks* that python
3.10 is still supported for several versions that actually do not
support this anymore. The metadata was only fixed recently in
https://github.com/microsoft/onnxruntime/releases/tag/v1.24.4 (see
microsoft/onnxruntime#27354).

Although this does not cause issues when *updating* the lock file, it
creates issues when it has to be fully regenerated which e.g. happens in
#771 .

It is thus necessary to pin the used version of `onnxruntime` for python
3.10.
@AVHopp AVHopp force-pushed the repo/exclude_newer_packages branch from da56572 to 8f105f7 Compare April 1, 2026 07:33
@AVHopp AVHopp marked this pull request as ready for review April 1, 2026 07:33
@AVHopp
Copy link
Copy Markdown
Collaborator Author

AVHopp commented Apr 1, 2026

@Scienfitz @AdrianSosic Note: Since this PR changes one of the settings of uv, the lock file had to be regenerated completely and was not just updated. This seems to be just how uv is configured and is what made me discover #772 . Probably has no consequences, but I wanted to make you aware of this.

@AVHopp AVHopp force-pushed the repo/exclude_newer_packages branch from 8f105f7 to 76d2195 Compare April 1, 2026 07:35
@AVHopp AVHopp marked this pull request as draft April 1, 2026 07:40
@AVHopp AVHopp force-pushed the repo/exclude_newer_packages branch from 76d2195 to 77aea7e Compare April 1, 2026 07:43
@AVHopp
Copy link
Copy Markdown
Collaborator Author

AVHopp commented Apr 1, 2026

Note that there is a fixed timestamp in the lock file which is static and will only be updated when a completely new resolution is triggered - just FYI.

@AVHopp AVHopp marked this pull request as ready for review April 1, 2026 07:47
@AdrianSosic AdrianSosic removed the on hold PR progress is awaiting for something else to continue label Apr 1, 2026
@Scienfitz Scienfitz added this to the 0.15.0 milestone Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

repo Requires changes to the project configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants