Skip to content

[Bug]: Move enum explanations and health checks from cuda_core to cuda_bindings #1712

@rwgk

Description

@rwgk

Related: nvbug 5932944

Situation

Currently (main @ eabaf4b) we have these enum explanations and related health checks :

Problem

The tests succeed only for the matching cuda-bindings release.

If cuda_core is tests are run against an older or newer version of cuda-bindings, they will fail (that's nvbug 5932944).

Solution

The explanations dicts are fundamentally directly tied to the bindings, therefore they should be moved to cuda_bindings (maybe a new cuda_bindings/cuda/bindings/_utils/ directory). Accordingly, the tests should be moved, too.

Backward compatibility

When the explanations dicts are moved, the cuda_core code using them needs to be made tolerant to not finding them in existing releases of cuda-bindings. I.e. the explanations will be used if available in cuda_bindings, or otherwise simply not be added when generating error messages at runtime. This is considered to be a minor loss that will become a non-issue over time as old cuda-bindings releases fall out of use.

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingcuda.bindingsEverything related to the cuda.bindings modulecuda.coreEverything related to the cuda.core module

Type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions