Fix: Pythonドキュメント周りの色々を修正 #570
Merged
Conversation
This file contains 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
qryxip
reviewed
Aug 8, 2023
Comment on lines
212
to
217
| """単語の優先度。 | ||
|
|
||
| 0から10までの整数。 | ||
| 数字が大きいほど優先度が高くなる。 | ||
| 1から9までの値を指定することを推奨する。 | ||
| """ |
There was a problem hiding this comment.
細かいのですが、「高くなる。」と「1から9」の間だけ0x20が入るようになっているみたいです。
おそらく行頭がASCII文字?だとスペースが入るようになっています。
あああああ
いいいいい
ううううう
aえええええ
bおおおおお
まあ気になる人がどれだけいるかという話になりますが...
(追記) 試したらGitHubもRustdocも同様でした。
| """ | ||
| @property | ||
| def words(self) -> Dict[UUID, UserDictWord]: | ||
| """単語のDict。""" |
There was a problem hiding this comment.
:class:`dict`みたいな表現にした方がよいかなと思いました。
私が表現するならこうでしょうか。
Suggested change
| """単語のDict。""" | |
| """このオプジェクトの :class:`dict` としての表現。""" |
There was a problem hiding this comment.
今気付いたのですが、更新を意図してこのwordsに__setitem__するユーザーが現れる気がしました。
to_dictみたいな名前にするか、あるいはこの@propertyはやめて__iter__を付けてdict(userdict)とできるようにするかした方がよいかもしれません。
There was a problem hiding this comment.
collections.abc.Mappingを継承するのが良さそうに思えました。
Co-authored-by: Ryo Yamashita <[email protected]>
|
そういえば：左上、結構カスタマイズできそうでした。こんな感じにしても良いかも？ |
qryxip
added a commit
to qryxip/voicevox_core
that referenced
this pull request
Sep 22, 2023
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.
内容
__init__がおかしかったので修正しました。関連 Issue
その他
逆にreSTに統一しても良かったかも？