Skip to content

Conversation

@Maru-mee
Copy link

改行を入れて可読性を上げてみました。
他のオプティマイザーでの書き方慣例を参考にしています(例:AdamWなど)。

この変更によるメリットは、
・このコードを使用するユーザーが、どんな引数指定が可能なのか、
一目で判別しやすくなる。
・今後スクリプトの変更があったときに、
どの引数が変化したのか、in-out関係がわかりやすくなる。

とはいえ結局のところ、
オーナー様のご都合や好みによるところも大きいと思いますので、
変更するかどうかのご判断はお任せ致します。

@muooon
Copy link
Owner

muooon commented Nov 18, 2025

お気遣いに感謝します。可読性は向上するかも。参考になりました。今後の公開では公開分について改行をいれるかどうか検討します。自分で読む分には縦長は読みづらく、でもユーザー視点ではおっしゃるとおりかもと思います。ご提案に感謝します、ありがとうございました。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants