PR #551 (3e47d15) renamed requirement_check → requirement-check in the catalog and moved the repo from rag-intrinsics-lib to granitelib-core-r1.0. This breaks test_huggingface.py (18/19 ERROR).
cc @ink-pad @mnagired @jakelorocco
Reproduced on LSF with:
uv sync --all-extras --group dev --group notebook --group docs && uv run --all-extras --group dev pytest test/backends/test_huggingface.py --durations=0 -v
Investigation
The immediate error is a ValueError — the test uses IntrinsicAdapter("requirement_check", ...) but the catalog now only has requirement-check. We attempted fixing the underscore→hyphen naming in the test, which got past the catalog lookup, but then hit a deeper problem:
The R1 repo (granitelib-core-r1.0) only has:
requirement-check/granite-4.0-micro/lora/
It does not have:
- Any Granite 3.3 adapters (test uses
granite-3.3-8b-instruct)
- Any alora adapters (test fixture defaults to alora)
So even switching to Granite 4 in the test wouldn't work — there's no alora variant in the R1 repo.
The old repo (rag-intrinsics-lib) still has both lora and alora for granite-3.3-{2b,8b}-instruct under the requirement_check (underscore) name.
Additionally, stale underscore references remain in huggingface.py:385 and requirement.py:76.
Suggested fix
Add the old catalog entry back alongside the new one so both repos are accessible, or migrate the test to Granite 4 once alora adapters are published (#359).
PR #551 (
3e47d15) renamedrequirement_check→requirement-checkin the catalog and moved the repo fromrag-intrinsics-libtogranitelib-core-r1.0. This breakstest_huggingface.py(18/19 ERROR).cc @ink-pad @mnagired @jakelorocco
Reproduced on LSF with:
Investigation
The immediate error is a
ValueError— the test usesIntrinsicAdapter("requirement_check", ...)but the catalog now only hasrequirement-check. We attempted fixing the underscore→hyphen naming in the test, which got past the catalog lookup, but then hit a deeper problem:The R1 repo (
granitelib-core-r1.0) only has:requirement-check/granite-4.0-micro/lora/It does not have:
granite-3.3-8b-instruct)So even switching to Granite 4 in the test wouldn't work — there's no alora variant in the R1 repo.
The old repo (
rag-intrinsics-lib) still has both lora and alora forgranite-3.3-{2b,8b}-instructunder therequirement_check(underscore) name.Additionally, stale underscore references remain in
huggingface.py:385andrequirement.py:76.Suggested fix
Add the old catalog entry back alongside the new one so both repos are accessible, or migrate the test to Granite 4 once alora adapters are published (#359).