Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: build_util ディレクトリを scripts へリネーム #1380

Closed
tarepan opened this issue Jun 1, 2024 · 2 comments · Fixed by #1388
Closed

refactor: build_util ディレクトリを scripts へリネーム #1380

tarepan opened this issue Jun 1, 2024 · 2 comments · Fixed by #1388
Assignees

Comments

@tarepan
Copy link
Contributor

tarepan commented Jun 1, 2024

内容

概要: build_util ディレクトリを scripts へリネームしてリファクタリング

現在の build_util は様々なスクリプトを含んでおり、その目的はビルドに限らない(例: get_cost_candidates.py)。
Python のメジャーなレポジトリではビルドスクリプトその他をまとめて scripts ディレクトリに入れるのがデファクトである(例: PyTorch - /scripts, FastAPI - /scripts)。

このような背景から、build_util ディレクトリを scripts へリネームするリファクタリングを提案します。

ref #688

Pros 良くなる点

  • Python デファクトに沿った命名による見通し改善

Cons 悪くなる点

無し

実現方法

  • リネーム

VOICEVOXのバージョン

0.19.0

@tarepan tarepan added 機能向上 要議論 実行する前に議論が必要そうなもの 状態:必要性議論 必要性を議論している状態 labels Jun 1, 2024
@Hiroshiba
Copy link
Member

とりあえずscriptsを目指すのが良さそうに思いました!
ビルド時にしか使わない系のは更にbuildとかで分けるとわかりやすそうですね!
まあ・・・ライセンスファイルの作成とか、buildなのかそうじゃないのかどっちなんだってなりますが・・・。

pytorchはtoolsscriptsがありますね。何が違うんだろう。

@tarepan tarepan added 状態:設計 設計をおこなっている状態 and removed 状態:必要性議論 必要性を議論している状態 labels Jun 2, 2024
@tarepan
Copy link
Contributor Author

tarepan commented Jun 3, 2024

scriptsを目指すのが良さそう

👍️
実装します。


pytorchはtoolsscripts ... 何が違う

両方ともビルド系含めて色々入っています。
scripts は(不思議ですが)Python パッケージになってます。pip install したい事情とかがあったんですかね?

@tarepan tarepan added 状態:実装 実装をおこなっている状態 and removed 要議論 実行する前に議論が必要そうなもの 状態:設計 設計をおこなっている状態 labels Jun 3, 2024
@tarepan tarepan self-assigned this Jun 3, 2024
@tarepan tarepan removed the 状態:実装 実装をおこなっている状態 label Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants