Skip to content

Fix oracle_cute_cpp to skip instead of error when nvidia is absent#10

Merged
jduprat merged 1 commit intofacebookresearch:mainfrom
jduprat:dev
Apr 9, 2026
Merged

Fix oracle_cute_cpp to skip instead of error when nvidia is absent#10
jduprat merged 1 commit intofacebookresearch:mainfrom
jduprat:dev

Conversation

@jduprat
Copy link
Copy Markdown
Contributor

@jduprat jduprat commented Apr 9, 2026

Fixes an issue found in CI when trying to release 0.2.1

Wrap the importlib.util.find_spec call in _module_root with a
try/except catching (ModuleNotFoundError, ValueError).  Previously,
find_spec("nvidia.cuda_runtime") raised ModuleNotFoundError when the
nvidia parent package was not installed, crashing before the fixture's
skip logic could run.  Now _module_root returns None, the candidate
include dirs list omits it, and the fixture skips gracefully.
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Apr 9, 2026
@jduprat jduprat merged commit 68dfea5 into facebookresearch:main Apr 9, 2026
3 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant