Conversation
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Greptile SummaryThis PR adds comprehensive documentation for all environment variables used in Transformer Engine. The new Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant Docs as Documentation Site
participant Index as docs/index.rst
participant EnvVars as docs/envvars.rst
User->>Docs: Navigate to documentation
Docs->>Index: Load index page
Index->>Index: Display "Advanced" section
Note over Index: Added envvars link
User->>Index: Click "Environment Variables"
Index->>EnvVars: Load envvars.rst
EnvVars->>EnvVars: Render RST documentation
Note over EnvVars: Build-Time Variables<br/>Runtime Variables<br/>Examples
EnvVars->>User: Display formatted documentation
Note over User: Can reference 60+ environment<br/>variables with types,<br/>defaults, and examples
|
There was a problem hiding this comment.
Additional Comments (1)
-
docs/envvars.rst, line 380-399 (link)style: Missing
NVTE_DPA_FP8_RECIPE_DPAandNVTE_DPA_FP8_RECIPE_MHA(used intransformer_engine/pytorch/attention/multi_head_attention.py:42-43). Also missing:NVTE_AG_P2P_MULTI_ATOMIC,NVTE_APPLY_QK_LAYER_SCALING,NVTE_BATCH_MHA_P2P_COMM,NVTE_CUDA_ARCHS, several internal/experimental variables
2 files reviewed, 1 comment
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Greptile's behavior is changing!From now on, if a review finishes with no comments, we will not post an additional "statistics" comment to confirm that our review found nothing to comment on. However, you can confirm that we reviewed your changes in the status check section. This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR". |
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
There was a problem hiding this comment.
Additional Comments (1)
-
docs/envvars.rst, line 2 (link)syntax: Copyright year should be 2026 to match other documentation files
2 files reviewed, 1 comment
pggPL
left a comment
There was a problem hiding this comment.
Looks really nice in HTML, I added some small suggestion to cpu coffloading envvar.
Co-authored-by: Paweł Gadziński <62263673+pggPL@users.noreply.github.com> Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Description
Document all environment variables used in TE.
Type of change
Changes
Document all environment variables used in TE.
Checklist: