已合并
[test] Add torch.autograd.profiler_util.StringTable.values Api Coverage #42025
lihaokun-2026创建于 7月18日
[test] Add torch.autograd.profiler_util.StringTable.values Api Coverage #42025
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 lihaokun-2026 的贡献）
7月18日 创建了 pull request，commit 531eff44
7月18日 关联了issue：【社区任务】7月社区任务第二期-Ascend for PyTorch API 一致性开发（68）
atomgit-bot
7月18日 评论：
7月18日 评论：
变更摘要
本 PR 新增了 torch.autograd.profiler_util.StringTable.values 接口的独立测试用例文件，旨在覆盖该 API 在空表、显式写入、__missing__ 自动写入、短字符串 key、动态视图更新以及 NPU 环境下的功能正确性验证。该测试文件在 torch_npu 环境中运行，确认 StringTable.values() 返回的 ValuesView 行为与预期一致。
主要改动
- 新增测试文件
test/test_profiler_util_string_table_values.py：创建独立的测试模块，包含TestStringTableValues类，对StringTable.values()进行最小化功能验证。 - 空表场景覆盖：
test_values_empty_string_table验证空StringTable调用values()返回空ValuesView实例。 - 显式写入与
__missing__自动写入覆盖：test_values_contains_explicit_items验证显式添加的 key/value 可通过values()获取；test_values_reflects_missing_key_insertion验证访问缺失 key 触发__missing__后，自动生成的 value 出现在values()结果中。 - 短字符串 key 与动态视图覆盖：
test_values_keeps_short_key_unchanged验证单字符 key 不触发 demangle 变更；test_values_view_updates_after_mutation验证values()返回动态视图，后续新增元素可被已有视图感知。 - NPU 环境验证：
test_values_runs_in_torch_npu_environment确认当前 privateuse1 backend 名称为"npu"，保证测试在正确的硬件环境下执行。
ascend-robot
7月18日 评论：
7月18日 评论：
atomgit-bot
7月18日 评论：
7月18日 评论：
代码审查
审查总结
对变更文件 test/test_profiler_util_string_table_values.py 的审查已完成，共发现 1 个 P3 级问题。
审查结果
| 文件 | 审查结论 |
|---|---|
test/test_profiler_util_string_table_values.py |
发现 1 个 P3 问题 |
发现汇总
- P3 (1)：
test_values_runs_in_torch_npu_environment方法名暗示验证StringTable.values()在 NPU 环境运行，但方法体内仅检查后端名称，未实际调用values()，构成测试名与实现不一致的覆盖盲区。
整体评估
该变更为新增的 API 覆盖测试文件，测试用例覆盖了空表、显式写入、__missing__ 自动写入、短字符串保持、动态视图更新等核心场景，逻辑正确，结构清晰。唯一的问题是最后一个测试方法未实际触及被测 API，风险较低（P3），不影响合入但建议修正。
⚠️ 已识别出整体风险，但无法提取行内评论，请参考整体评估。
此处折叠了113条消息 查看更多
梁松伟
18 天前 评论：
18 天前 评论：
/approve
18 天前 添加了label：approvedlgtm
18 天前 合入了pull request
ascend-robot
18 天前 评论：
18 天前 评论：
Pull Request 已合并或已关闭。
If you want to solve this problem, you can click here to do it in the FAQs.
ascend-robot
18 天前 评论：
18 天前 评论：
流水线 pytorch_gitcode_PR_multiVersion#13549 [ commitID：735b58e3 ] 已完成
【合入来源】
#2928
torch.autograd.profiler_util.StringTable.values: 【社区任务】7月社区任务第二期-Ascend for PyTorch API 一致性开发（68）【修改方案】
API 功能说明：
torch.autograd.profiler_util.StringTable.values：StringTable继承自defaultdict，values()返回当前 string table 中所有 value 的动态视图，用于获取 profiler 字符串映射表中的值集合。上游用例检查情况：
检索 PyTorch / Torch-NPU 现有测试后，未找到面向
torch.autograd.profiler_util.StringTable.values的独立、直接、最小验证用例，因此按任务要求新增 NPU API 验证用例。新增用例文件：
test_profiler_util_string_table_values.py新增用例覆盖场景：
StringTable.values()在空表场景下返回空ValuesViewvalues()返回全部 valueStringTable.__missing__后，values()可读取自动写入的 valuevalues()返回动态视图，后续新增元素可被已有 view 感知torch_npu环境下验证 privateuse1 backend 为npu新增文件头部已按要求补充注释，说明该文件验证的 API 功能。
【资料变更】
不涉及。未修改产品资料、接口说明文档或用户可见文档。
【接口变更】
不涉及。未修改跨代码仓接口，也未修改用户可见接口行为。
【功能验证】
测试环境：
测试方法：
source /usr/local/Ascend/ascend-toolkit/set_env.sh TORCH_DEVICE_BACKEND_AUTOLOAD=1 python3 test/test_profiler_util_string_table_values.py -v测试结果：
【CheckList】