Conversation
cityofships
commented
Feb 4, 2026
03:00.0 3D controller [0302]: NVIDIA Corporation GH100 [H200 NVL] [10de:233b] (rev a1)
There was a problem hiding this comment.
Code Review
This pull request adds support for NVIDIA H200 NVL GPUs by defining it in stackhpc-compute.yml and adding a corresponding release note. The changes are straightforward and correct. I've provided a couple of minor suggestions to improve consistency and fix a grammatical error.
| h200: | ||
| resource_name: "{{ h200_resource_name | default('h200') }}" |
There was a problem hiding this comment.
For better clarity and consistency with other GPU definitions in this file (e.g., h100_80_sxm, a100_80_sxm), it would be more descriptive to name this entry h200_nvl instead of just h200. This makes it clear that this definition is for the NVL variant of the H200 and avoids potential ambiguity if other H200 variants are added in the future.
h200_nvl:
resource_name: "{{ h200_nvl_resource_name | default('h200_nvl') }}"There was a problem hiding this comment.
I actually agree with gemini here
There was a problem hiding this comment.
Looking at the file, the trend seemed to be just SXM versions having their suffix
There was a problem hiding this comment.
I think that's just because we don't have any NVL cards in the list yet
| --- | ||
| features: | ||
| - | | ||
| Adds a definitions for ``NVIDIA H200 NVL`` to the |