Skip to content

Pin standard.def deps via requirements-direct.txt (fixes #19)#46

Closed
nevasini1 wants to merge 1 commit into
aisa-group:mainfrom
nevasini1:fix/issue-19-pin-inspect-ai-standard-image
Closed

Pin standard.def deps via requirements-direct.txt (fixes #19)#46
nevasini1 wants to merge 1 commit into
aisa-group:mainfrom
nevasini1:fix/issue-19-pin-inspect-ai-standard-image

Conversation

@nevasini1
Copy link
Copy Markdown

Summary

  • Switches containers/standard.def to load containers/requirements-direct.txt instead of installing an unpinned package list inline
  • Same pattern already used by vllm_debug.def and opus_4_6_codex_5_3.def, so standard.def now picks up the same pinned inspect-ai==0.3.150, transformers==4.57.3, and huggingface-hub==0.36.0 that resolve cleanly together
  • Pins flash-attn==2.8.3 (was flash_attn unpinned) to match the working images

Fixes #19. As @hrdkbhatnagar noted in that thread, the maintainer-recommended pin is inspect-ai==0.3.150, which is what requirements-direct.txt already provides.

Verification

All 20 packages from the previous inline install list are present in containers/requirements-direct.txt:

accelerate, boto3, bitsandbytes, datasets, evaluate, lm-eval, openai,
pandas, scikit-learn, shortuuid, tokenizers, transformers, trl, peft,
tiktoken, inspect-ai, matplotlib, certifi, ninja, packaging

Container behavior preserved otherwise:

  • AI CLI tool versions for standard.def left unchanged (claude-code@2.0.55, codex@0.79.0, gemini-cli@0.18.4, opencode-ai@1.1.59)
  • inspect_evals still cloned with --depth=1 (no pinned commit), matching prior behavior of this image
  • %runscript, %environment, %labels, %help blocks untouched

Test plan

  • bash containers/build_container.sh standard succeeds end-to-end
  • In the built image, python -c "import inspect_ai, transformers, huggingface_hub; print(inspect_ai.__version__, transformers.__version__, huggingface_hub.__version__)" reports 0.3.150 / 4.57.3 / 0.36.0
  • pip check inside the image reports no resolver conflicts

🤖 Generated with Claude Code

Fixes #19. The unpinned `inspect-ai` and `transformers` in standard.def
resolve to versions whose `huggingface-hub` constraints conflict at install
time. The working images (vllm_debug, opus_4_6_codex_5_3) already load
containers/requirements-direct.txt which pins inspect-ai==0.3.150 alongside
transformers==4.57.3 and huggingface-hub==0.36.0. Switch standard.def to the
same pattern and pin flash-attn==2.8.3 to match.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nevasini1 nevasini1 force-pushed the fix/issue-19-pin-inspect-ai-standard-image branch from d7990c0 to ac746d3 Compare May 27, 2026 03:18
@nevasini1
Copy link
Copy Markdown
Author

Closing to re-open from a clean fork — author email on the commit was wrong. Same one-file fix will be reopened shortly under the same branch name. Apologies for the noise.

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.

huggingface-hub Version Conflict with current transformers and inspect-ai dependency requirement

1 participant