Skip to content

Conversation

kanze1
Copy link
Contributor

@kanze1 kanze1 commented Nov 23, 2024

PR Category

User Experience

PR Types

Bug fixes

Description

修复代码中的拼写错误并更新 _typos.toml:

修复 round_robin.h 中的拼写错误:

将注释中的 "AVALIABLE" 改为 "available"
更新 _typos.toml:

删除了已修复的 "AVALIABLE= 'AVALIABLE'" 条目

@kanze1 kanze1 requested a review from SigureMo as a code owner November 23, 2024 15:38
Copy link

paddle-bot bot commented Nov 23, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Nov 23, 2024
@SigureMo
Copy link
Member

学一下分支管理,别从上一个 PR 的分支继续提 PR

@kanze1
Copy link
Contributor Author

kanze1 commented Nov 23, 2024

学一下分支管理,别从上一个 PR 的分支继续提 PR

那这次的提交还要改吗

@SigureMo
Copy link
Member

那这次的提交还要改吗

这次可以不改分支,但你得解决冲突啊

@SigureMo SigureMo changed the title [CodeStyle][Typos][A-45] Fix typo('available') [CodeStyle][Typos][A-45] Fix typo('AVALIABLE') Nov 23, 2024
@SigureMo SigureMo changed the title [CodeStyle][Typos][A-45] Fix typo('AVALIABLE') [CodeStyle][Typos][A-45] Fix typo(AVALIABLE) Nov 23, 2024
@SigureMo SigureMo changed the title [CodeStyle][Typos][A-45] Fix typo(AVALIABLE) [CodeStyle][Typos][A-45] Fix typo (AVALIABLE) Nov 23, 2024
_typos.toml Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

自己看看 diff,冲突解决的不对

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好的,我研究下

@kanze1 kanze1 force-pushed the fix_typo_available branch from c98fc33 to 9434d7a Compare November 23, 2024 16:31
cmake/cuda.cmake Outdated
@@ -287,7 +287,7 @@ elseif(${CMAKE_CUDA_COMPILER_VERSION} LESS 13.0) # CUDA 12.0+
endif()

if(NOT ${CMAKE_CUDA_COMPILER_VERSION} LESS 10.0)
add_definitions("-DTRT_PLUGIN_FP16_AVALIABLE")
add_definitions("-DTRT_PLUGIN_FP16_available")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

人家原来是全大写的,你改成小写的不觉得有问题吗?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

那怎么改呀?=-=

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

改成全大写啊

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

他原本就是全大写,我应该是只修复原本的那一个错误,然后关于大写的就不动,那之前合并的那次pr应该就可以了对吗?我修改描述说明这个宏定义的大写不需要修改是不是这个意思?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AVALIABLE -> AVAILABLE 怎么能是不动呢?这里是拼写错误又不是大小写错误

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

啊,我明白了,抱歉,犯蠢了

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMeow 🐾

@SigureMo SigureMo merged commit 31efc0a into PaddlePaddle:develop Nov 23, 2024
28 of 29 checks passed
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants