-
Notifications
You must be signed in to change notification settings - Fork 875
[NO REVIEW TEST CI] Fix/docs2 #7683
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
…n signatures - Remove `no_need_convert_file_path` parameter from `generate_category1_table` and `generate_category2_table` - Update `extract_no_need_convert_list` to read from `api_mapping.json` instead of parsing a Python file - Remove unused file paths in `main` function - Improve error handling in `load_mapping_json` to return empty dict on failure
|
感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-7683.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html |
📚 本次 PR 文档预览链接(点击展开)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This is a test PR explicitly marked as "NO REVIEW TEST CI" that includes refactoring of API mapping utilities and documentation updates.
Changes:
- Refactored
extract_no_need_convert_list()to read fromapi_mapping.jsoninstead of parsing Python AST - Fixed bug in
load_mapping_jsonerror handling (returning{}instead of[]) - Updated API documentation for
transformers.StoppingCriteriaListandtransformers.LogitsProcessor - Moved
torch.allclosedocumentation fromargs_name_difftoinvok_only_diffcategory with updated Paddle API mapping
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| docs/guides/model_convert/convert_from_pytorch/tools/validate_api_difference_consistency.py | Updated function call to use simplified signature |
| docs/guides/model_convert/convert_from_pytorch/tools/utils.py | Refactored extract_no_need_convert_list() to use JSON parsing; removed unused ast import; fixed error return type in load_mapping_json |
| docs/guides/model_convert/convert_from_pytorch/tools/generate_pytorch_api_mapping.py | Updated function signatures and calls to remove unused parameters |
| docs/guides/model_convert/convert_from_pytorch/api_difference/torch_more_args/torch.Tensor.log_normal_.md | Deleted file (entire file removed) |
| docs/guides/model_convert/convert_from_pytorch/api_difference/invok_only_diff/transformers.StoppingCriteriaList.md | Updated API signatures to include *args parameter |
| docs/guides/model_convert/convert_from_pytorch/api_difference/invok_only_diff/torch.allclose.md | Added file with mapping from torch.allclose to paddle.compat.allclose |
| docs/guides/model_convert/convert_from_pytorch/api_difference/args_name_diff/transformers.LogitsProcessor.md | Corrected Paddle parameter name from scores to logits |
| docs/guides/model_convert/convert_from_pytorch/api_difference/args_name_diff/torch.allclose.md | Deleted file (moved to invok_only_diff category) |
| transformers.LogitsProcessor(input_ids: torch.LongTensor, scores: torch.FloatTensor) | ||
| ``` | ||
|
|
||
| ### [paddleformers.generation.LogitsProcessor](https://github.com/PaddlePaddle/PaddleNLP/blob/e336e78c338d2514ee6c937982ce5d8c960b85ff/paddlenlp/generation/logits_process.py#L26) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NO REVIEW