Skip to content

Conversation

@Ricardo-shuo-liu
Copy link
Contributor

PR Category

User Experience
Description

Fix:
修复如下 typo 问题:
treshold',
'trian',
'traing ,
trainning , traning','transfered',
'trasformed'

目前通过codespell验证已经不存在t单词的问题
截图 2025-10-28 22-20-02
其中存在typos误判将单词部分标注为错误,选择保留
截图 2025-10-28 22-12-40
截图 2025-10-28 22-11-30

Related links
✏️ Typos 工具引入 #7543

@github-actions
Copy link

github-actions bot commented Oct 28, 2025

📚 本次 PR 文档预览链接(点击展开)
ℹ️ 预览提醒
请等待 Docs-NEW 流水线运行完成后再点击预览链接,否则可能会看到旧版本内容或遇到链接无法访问的情况。

@paddle-bot
Copy link

paddle-bot bot commented Oct 28, 2025

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-7588.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:飞桨文档预览工具

@Ricardo-shuo-liu Ricardo-shuo-liu changed the title [CodeStyle][Typos][T-[6-10]] Fix typo('treshold','trian','traing , trainning , traning','transfered','trasformed') [CodeStyle][Typos][T-[11-15]] Fix typo('treshold','trian','traing , trainning , traning','transfered','trasformed') Oct 28, 2025
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Oct 29, 2025
@ooooo-create
Copy link
Collaborator

其中存在typos误判将单词部分标注为错误,选择保留

可以说一下原因和位置吗

@Ricardo-shuo-liu
Copy link
Contributor Author

其中存在typos误判将单词部分标注为错误,选择保留

可以说一下原因和位置吗

我直接使用grep搜索trian和traing定位到的都是单词一部分或者文件名,然后我搜索train 和traing 后面加上空格分词没有查询到。使用codespell检验错误然后使用grep对检验重复上面两种搜索也没找到,然后我把其他改完毕后用codespell搜索不存在t or T开头错误,所以我觉得可能是错误识别。
如果是我检验失误,请在联系我。
还有一个问题是在修改issue的a1过程中,Accuray是在ipynb也就是jupyter的代码部分的问题,他在代码的print里面错误使用Accuray导致输出格里面全部出错,按道理只要修改代码就好,但是我不可能本地跑一遍,然后实现修改,因为jupyter输出格子里面输出数据不可修改。
截图 2025-10-29 23-26-14

@ooooo-create
Copy link
Collaborator

我直接使用grep搜索trian和traing定位到的都是单词一部分或者文件名,然后我搜索train 和traing 后面加上空格分词没有查询到。使用codespell检验错误然后使用grep对检验重复上面两种搜索也没找到,然后我把其他改完毕后用codespell搜索不存在t or T开头错误,所以我觉得可能是错误识别。

正确的修复方式是先注释掉你要修复的单词,然后执行 typos .(需要先 pip install typos) 来查看问题信息并修复吧,修复完再把已经注释的单词删掉吧

@Ricardo-shuo-liu
Copy link
Contributor Author

我直接使用grep搜索trian和traing定位到的都是单词一部分或者文件名,然后我搜索train 和traing 后面加上空格分词没有查询到。使用codespell检验错误然后使用grep对检验重复上面两种搜索也没找到,然后我把其他改完毕后用codespell搜索不存在t or T开头错误,所以我觉得可能是错误识别。

正确的修复方式是先注释掉你要修复的单词,然后执行 typos .(需要先 pip install typos) 来查看问题信息并修复吧,修复完再把已经注释的单词删掉吧

sorry,昨天没有说清楚,就是我是先用 typos,然后没找到,之后使用grep和codespell的。
截图 2025-10-30 07-17-01
然后呢,typos检验到的是文件名字,和一个代码,但是被注释掉了,所以我就没改,因为我不太敢,然后上面说的a,因为没有被注释掉,需要运行,所以来问一下你怎么办??如果上面的代码和文件名部分需要修改的话,我再修改一次。

@Ricardo-shuo-liu
Copy link
Contributor Author

我直接使用grep搜索trian和traing定位到的都是单词一部分或者文件名,然后我搜索train 和traing 后面加上空格分词没有查询到。使用codespell检验错误然后使用grep对检验重复上面两种搜索也没找到,然后我把其他改完毕后用codespell搜索不存在t or T开头错误,所以我觉得可能是错误识别。

正确的修复方式是先注释掉你要修复的单词,然后执行 typos .(需要先 pip install typos) 来查看问题信息并修复吧,修复完再把已经注释的单词删掉吧

我觉得文件名不宜修改,可能会导致其他问题?而代码部分我觉得可以修改。对于这两个问题解决,希望你回复。然后我及时修改pr。

@ooooo-create
Copy link
Collaborator

我直接使用grep搜索trian和traing定位到的都是单词一部分或者文件名,然后我搜索train 和traing 后面加上空格分词没有查询到。使用codespell检验错误然后使用grep对检验重复上面两种搜索也没找到,然后我把其他改完毕后用codespell搜索不存在t or T开头错误,所以我觉得可能是错误识别。

正确的修复方式是先注释掉你要修复的单词,然后执行 typos .(需要先 pip install typos) 来查看问题信息并修复吧,修复完再把已经注释的单词删掉吧

我觉得文件名不宜修改,可能会导致其他问题?而代码部分我觉得可以修改。对于这两个问题解决,希望你回复。然后我及时修改pr。

文件名可以改呀,代码部分也可以改啊,只是你要看看它想 print 的什么就改成什么

@Ricardo-shuo-liu
Copy link
Contributor Author

我直接使用grep搜索trian和traing定位到的都是单词一部分或者文件名,然后我搜索train 和traing 后面加上空格分词没有查询到。使用codespell检验错误然后使用grep对检验重复上面两种搜索也没找到,然后我把其他改完毕后用codespell搜索不存在t or T开头错误,所以我觉得可能是错误识别。

正确的修复方式是先注释掉你要修复的单词,然后执行 typos .(需要先 pip install typos) 来查看问题信息并修复吧,修复完再把已经注释的单词删掉吧

我觉得文件名不宜修改,可能会导致其他问题?而代码部分我觉得可以修改。对于这两个问题解决,希望你回复。然后我及时修改pr。

文件名可以改呀,代码部分也可以改啊,只是你要看看它想 print 的什么就改成什么

文件名和ipynb代码部分已经完成修改。本pr应该没有问题了。
但是目前还有一个另一个pr的疑问就是关于A1的问题。上面提到了,也是一个代码错误,但是这个错误的部分被输出了。如果仅仅修改代码并且删除_typos文件这个Accuray会导致commit无法实现。因为jupyter代码块的输出无法直接更改。所以,我对这个问题改怎么处理????
截图 2025-10-30 21-56-29
Uploading 截图 2025-10-30 21-56-46.png…

@ooooo-create
Copy link
Collaborator

ooooo-create commented Oct 31, 2025

我记得_typos.toml 前面有排除文件的设置,我留空了,你可以添加进去试试

# The following files will be excluded from spell check during commits
extend-exclude = [
    # Skip `Accuray ` check in these files
    "文件路径",
]

@ooooo-create
Copy link
Collaborator

代码部分也可以改啊,只是你要看看它想 print 的什么就改成什么

虽然是修 typo,但是这个注释想打印的是 housing_features,既然发现了,建议最好也是一起修了吧,下面那个 traing_data 也是一样~
image

@Ricardo-shuo-liu
Copy link
Contributor Author

代码部分也可以改啊,只是你要看看它想 print 的什么就改成什么

虽然是修 typo,但是这个注释想打印的是 housing_features,既然发现了,建议最好也是一起修了吧,下面那个 traing_data 也是一样~ image

husing_features这个部分我也修改好了,然后A1的排除(也就是对于Jupyter的输出我直接就不管了,我在_typos.toml的给排除了),我在下个pr会修好把那A1个代码错误修正。这个pr的错误文件名也修改完毕,可以直接merge了。

@ooooo-create
Copy link
Collaborator

image 没改对吧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants