Skip to content

Commit

Permalink
tools: Don't include extras in the requirements.txt
Browse files Browse the repository at this point in the history
This fixes a CI issue where we could not regenerate the container image
due to [1]

11 19.13 Collecting pydantic~=2.0
11 19.13 ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
11 19.13     pydantic~=2.0 from https://files.pythonhosted.org/packages/df/e4/ba44652d562cbf0bf320e0f3810206149c8a4e99cdbf66da82e97ab53a15/pydantic-2.9.2-py3-none-any.whl (from dandi==0.63.0->-r requirements.txt (line 364))
11 ERROR: process "/bin/sh -c python -m venv --system-site-packages /home/ci/.venv &&     . /home/ci/.venv/bin/activate                        &&     pip3 install -r requirements.txt" did not complete successfully: exit code: 1

The exact source of the problem is unkonwn but regenerating
tools/nwb-read-tests/requirements.txt on debian bookworm seems to fix the
issue.

As we also don't need the extras we can just not include them.

[1]: https://github.com/AllenInstitute/MIES/actions/runs/11076005552/job/30822809003
  • Loading branch information
t-b committed Sep 30, 2024
1 parent 6408628 commit 698f5fe
Show file tree
Hide file tree
Showing 2 changed files with 484 additions and 465 deletions.
1 change: 1 addition & 0 deletions tools/nwb-read-tests/requirements.in
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
# needs to be pip-compile'd on debian bookworm
pynwb==2.8.1
dandi==0.63.0
Loading

0 comments on commit 698f5fe

Please sign in to comment.