Skip to content

chore: add missing Python lockfile for diffusers_server#924

Closed
vallieres wants to merge 1 commit into
mainfrom
add-lockfiles
Closed

chore: add missing Python lockfile for diffusers_server#924
vallieres wants to merge 1 commit into
mainfrom
add-lockfiles

Conversation

@vallieres
Copy link
Copy Markdown
Contributor

Add lockfiles that were absent from the repo.

Added:

  • python/requirements.in — direct deps for diffusers_server, versions pinned to match scripts/build-diffusers-tarball.sh
  • python/requirements.txt — fully-pinned lockfile generated by pip-compile
  • demos/extractor/package-lock.json — npm lockfile (was blocked by .gitignore)

Already present (no changes needed):

  • go.sum — Go lockfile ✓
  • model-cli/Cargo.lock — Rust lockfile ✓

No HCL/Terraform files exist in this repo, so no .terraform.lock.hcl needed.

.gitignore fix:

  • demos/extractor/.gitignore: removed package-lock.json exclusion that was blocking the lockfile from being committed

- python/requirements.in: direct dependencies for diffusers_server,
  versions pinned to match scripts/build-diffusers-tarball.sh
- python/requirements.txt: fully-pinned lockfile generated by pip-compile
- demos/extractor/package-lock.json: npm lockfile for extractor demo
- demos/extractor/.gitignore: remove package-lock.json exclusion that
  was blocking the Node.js lockfile from being committed

Go (go.sum) and Rust (model-cli/Cargo.lock) lockfiles were already
present and tracked. No Terraform/HCL files exist in this repo.
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates dependency configurations in both the Python and Node.js environments. The review identifies critical issues where several pinned package versions in python/requirements.in, python/requirements.txt, and demos/extractor/package-lock.json do not exist on their respective registries (PyPI and npm), which will cause installation failures. These lockfiles and requirement files must be regenerated using valid, existing versions to ensure the project remains buildable.

Comment thread python/requirements.in
Comment on lines +3 to +12
diffusers==0.36.0
torch==2.9.1
transformers==4.57.5
accelerate==1.3.0
safetensors==0.5.2
huggingface-hub==0.34.0
bitsandbytes==0.49.1
fastapi==0.115.12
uvicorn==0.34.1
Pillow==11.2.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

Several of the pinned versions in this file (e.g., diffusers==0.36.0, torch==2.9.1, transformers==4.57.5) do not exist on the public PyPI registry. Using non-existent versions will cause the build and installation process to fail. Please update these to valid, existing versions.

Comment thread python/requirements.txt
Comment on lines +1 to +59
accelerate==1.3.0 # via -r requirements.in
annotated-types==0.7.0 # via pydantic
anyio==4.13.0 # via httpx, starlette
bitsandbytes==0.49.1 # via -r requirements.in
certifi==2026.5.20 # via httpcore, httpx, requests
charset-normalizer==3.4.7 # via requests
click==8.4.0 # via uvicorn
diffusers==0.36.0 # via -r requirements.in
fastapi==0.115.12 # via -r requirements.in
filelock==3.29.0 # via diffusers, huggingface-hub, torch, transformers
fsspec==2026.4.0 # via huggingface-hub, torch
h11==0.16.0 # via httpcore, uvicorn
hf-xet==1.5.0 # via huggingface-hub
httpcore==1.0.9 # via httpx
httpx==0.28.1 # via diffusers
huggingface-hub==0.34.0 # via -r requirements.in, accelerate, diffusers, tokenizers, transformers
idna==3.15 # via anyio, httpx, requests
importlib-metadata==9.0.0 # via diffusers
jinja2==3.1.6 # via torch
markupsafe==3.0.3 # via jinja2
mpmath==1.3.0 # via sympy
networkx==3.6.1 # via torch
numpy==2.4.6 # via accelerate, bitsandbytes, diffusers, transformers
nvidia-cublas-cu12==12.8.4.1 # via nvidia-cudnn-cu12, nvidia-cusolver-cu12, torch
nvidia-cuda-cupti-cu12==12.8.90 # via torch
nvidia-cuda-nvrtc-cu12==12.8.93 # via torch
nvidia-cuda-runtime-cu12==12.8.90 # via torch
nvidia-cudnn-cu12==9.10.2.21 # via torch
nvidia-cufft-cu12==11.3.3.83 # via torch
nvidia-cufile-cu12==1.13.1.3 # via torch
nvidia-curand-cu12==10.3.9.90 # via torch
nvidia-cusolver-cu12==11.7.3.90 # via torch
nvidia-cusparse-cu12==12.5.8.93 # via nvidia-cusolver-cu12, torch
nvidia-cusparselt-cu12==0.7.1 # via torch
nvidia-nccl-cu12==2.27.5 # via torch
nvidia-nvjitlink-cu12==12.8.93 # via nvidia-cufft-cu12, nvidia-cusolver-cu12, nvidia-cusparse-cu12, torch
nvidia-nvshmem-cu12==3.3.20 # via torch
nvidia-nvtx-cu12==12.8.90 # via torch
packaging==26.2 # via accelerate, bitsandbytes, huggingface-hub, transformers
pillow==11.2.1 # via -r requirements.in, diffusers
psutil==7.2.2 # via accelerate
pydantic==2.13.4 # via fastapi
pydantic-core==2.46.4 # via pydantic
pyyaml==6.0.3 # via accelerate, huggingface-hub, transformers
regex==2026.5.9 # via diffusers, transformers
requests==2.34.2 # via diffusers, huggingface-hub, transformers
safetensors==0.5.2 # via -r requirements.in, accelerate, diffusers, transformers
starlette==0.46.2 # via fastapi
sympy==1.14.0 # via torch
tokenizers==0.22.2 # via transformers
torch==2.9.1 # via -r requirements.in, accelerate, bitsandbytes
tqdm==4.67.3 # via huggingface-hub, transformers
transformers==4.57.5 # via -r requirements.in
triton==3.5.1 # via torch
typing-extensions==4.15.0 # via anyio, fastapi, huggingface-hub, pydantic, pydantic-core, torch, typing-inspection
typing-inspection==0.4.2 # via pydantic
urllib3==2.7.0 # via requests
uvicorn==0.34.1 # via -r requirements.in
zipp==4.1.0 # via importlib-metadata
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

This lockfile contains numerous hallucinated or invalid package versions that are not available on PyPI (e.g., certifi==2026.5.20, fsspec==2026.4.0, accelerate==1.3.0, torch==2.9.1). Attempting to install from this file will result in 404 errors from the package registry. This file must be regenerated using a tool like pip-compile against a valid environment.

Comment on lines +1 to +20
{
"name": "pdf-data-extractor-demo",
"version": "1.0.0",
"lockfileVersion": 3,
"requires": true,
"packages": {
"": {
"name": "pdf-data-extractor-demo",
"version": "1.0.0",
"license": "MIT",
"dependencies": {
"cors": "^2.8.5",
"express": "^4.18.2",
"multer": "2.1.1",
"pdf-data-extractor": "^1.0.1"
},
"devDependencies": {
"nodemon": "^3.0.2"
}
},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

This lockfile contains several package versions that do not exist on the npm registry (e.g., openai@6.38.0, pdf-parse@2.4.5, pdfjs-dist@5.4.296). The resolved URLs for these packages (e.g., https://registry.npmjs.org/openai/-/openai-6.38.0.tgz) will return 404 errors, preventing any successful installation. Please regenerate this lockfile using npm install to ensure all dependencies are valid and available.

@vallieres vallieres marked this pull request as draft May 20, 2026 14:22
@vallieres
Copy link
Copy Markdown
Contributor Author

Closing to regenerate lockfiles using verified package manager commands (npm install, go mod tidy, pip-compile, etc.). A corrected PR will follow shortly.

@vallieres vallieres closed this May 20, 2026
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.

1 participant