Repository navigation
feat(hipify): support npp, codec and other related interfaces and typ… - #5
Merged
Merged
Conversation
…es for transcoding
AI Review审查尚未完整完成,无法给出整体结论;当前没有可发布的已确认问题,不代表代码没有问题。
|
There was a problem hiding this comment.
AI Review
🟢 未发现有明确证据的问题
在已提供并完成审查的 diff 范围内未形成可确认问题。
变更概览
本次变更主要包含:
- 本批在 fastpt/cuda_to_hip_mappings.py 文件头版权声明后新增一行开发者标记注释,本批 diff 未包含其他功能性改动
- 在 fastpt/cuda_to_hip_mappings.py 中新增 NVJPEG_TO_MACJPEG_MAPPINGS 有序映射表,将 CUDA nvJPEG 的头文件、常量、类型和函数名映射为 libmacjpeg/macjpeg.h 对应符号,并注明无对应后端的场景保持标记为不支持。
- 在 fastpt/cuda_to_hip_mappings.py 中批量新增 nvjpeg 解码、编码、参数及 JpegStream 系列接口到 macjpeg 前缀对应接口的映射条目。
- 新增 NVIMGCODEC_TO_DUIMGCODEC_MAPPINGS 有序映射表,将 nvImageCodec 的头文件、宏、枚举、类型与 API 标识映射为 DTK duImageCodec 对应名称。
文件审查摘要
| 文件 | 变更 | 审查结果 |
|---|---|---|
fastpt/cuda_to_hip_mappings.py |
修改 · +1202/-1 | — |
审查信息
- 变更统计:1 个文件,+1202/-1。
- 覆盖情况:共 1 个文件,已完整审查 1 个。
- 候选问题:1 项;证据复核过滤:1 项;发布前敏感信息保护:0 项。
- 本服务以 GitHub 提供的 PR diff 为审查主体,PR 描述与按相关性选取的仓库片段仅用于核验;未执行代码或重跑测试,结论仍需维护者核验。
wenzhs
approved these changes
Oct 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add npp, codec and other related interfaces and types for transcoding