Update manipulation.py #46380
Closed
Update manipulation.py #46380
Conversation
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
fix bugs
|
你的PR提交成功，感谢你对开源项目的贡献! |
|
❌ The PR is not created using PR's template. You can refer to this Demo. |
SigureMo
requested changes
Sep 22, 2022
| If True, use the overwrite mode to update the output of the same index, | ||
| if False, use the accumulate mode to update the output of the same index.Default value is True. | ||
| overwrite (bool): The mode that updating the output when there are same indices. If True, use the overwrite mode to update the output of the same index, if False, use the accumulate mode to update the output of the same index.Default value is True. |
Member
There was a problem hiding this comment.
这里折一下行吧，其实把那个空行删掉应该就可以了
rst 会在多行之间自动加一个空格，就像这样：
xxxxxxxxxxxxxx
yyyyyyyyyyyyyy
zzzzzzzzzzzzzz会渲染成
<p>xxxxxxxxxxxxxx yyyyyyyyyyyyyy zzzzzzzzzzzzzz</p>这种效果在英文中是恰到好处的，而在中文却是有问题的，英文这边由于是写在 docstring 里的，是需要便于开发者在编辑器阅读的，因此行宽不能过长，这也是为什么英文建议折行，而中文不建议折行。
flake8 有一个错误码就是针对行宽的，不过目前没有启用，而且优先级也不高，但还是尽可能地不要让行宽太宽吧～
以及 2674 行因为有一个 tab，应该会与 #46288 冲突，所以可以等等再 merge
Contributor
|
这个issue已经被 #46765 解决 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
fix bugs
PR types
PR changes
Describe