[None][fix] AutoDeploy: Move hf_id_to_local_model_dir to function for GLM4.7 Flash test

[bmarimuthu-nv]

Summary by CodeRabbit

• Tests
  • Updated test infrastructure to use Hugging Face model identifiers instead of precomputed model paths.
  • Modified test configurations for BF16 and NVFP4 dtype handling to derive model paths at runtime.
  • Preserved existing quantization settings while streamlining test setup.

Fixes: https://nvbugspro.nvidia.com/bug/6270692

Description

Test Coverage

PR Checklist

Please review the following before submitting your PR:

• PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

• PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

• Test cases are provided for new code paths (see test instructions)

• If PR introduces API changes, an appropriate PR label is added - either api-compatible or api-breaking. For api-breaking, include BREAKING in the PR title.

• Any new dependencies have been scanned for license and vulnerabilities

• CODEOWNERS updated if ownership changes

• Documentation updated as needed

• Update tava architecture diagram if there is a significant design change in PR.

• The reviewers assigned automatically/manually are appropriate for the PR.

• Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

[bmarimuthu-nv]

@CodeRabbit summary

[coderabbitai]

✅ Action performed

Summary regeneration triggered.

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR updates TestGLM4Flash to derive model directory paths at runtime from new Hugging Face ID constants and passes those derived paths into AutoDeployLLM in the BF16 and NVFP4 tests.

Changes

GLM4Flash HF ID-based path derivation

Layer / File(s)	Summary
GLM4Flash test model path computation tests/integration/defs/accuracy/test_llm_api_autodeploy.py	TestGLM4Flash introduces MODEL_HF_ID_BF16 and MODEL_HF_ID_NVFP4, and updates test_auto_dtype and test_nvfp4 to call hf_id_to_local_model_dir() to compute model_path at runtime and pass it as both model and tokenizer to AutoDeployLLM.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

• suyoggupta
• marinayanov
• dpitman-nvda

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	⚠️ Warning	The pull request description is incomplete with no content in the Description, Test Coverage, or PR Checklist sections; only the template structure and a bug reference are provided.	Provide a clear description of what was changed and why (the issue and solution), list the relevant tests that safeguard the changes, and complete the PR checklist items with actual answers or explanations.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly indicates the main change: refactoring hf_id_to_local_model_dir into a function for the GLM4.7 Flash test, which aligns with the raw summary describing runtime derivation of model_path.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[bmarimuthu-nv]

/bot run

[fredricz-20070104]

Approved

[tensorrt-cicd]

PR_Github #52291 [ run ] triggered by Bot. Commit: 81b59ef Link to invocation

[tensorrt-cicd]

PR_Github #52291 [ run ] completed with state SUCCESS. Commit: 81b59ef
/LLM/main/L0_MergeRequest_PR pipeline #41600 completed with status: 'SUCCESS'

CI Report

Link to invocation