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

Add optional dependencies hpo, torch #406

Merged
merged 9 commits into from
Dec 17, 2024
Merged

Conversation

KanaiYuma-aist
Copy link
Collaborator

@KanaiYuma-aist KanaiYuma-aist commented Dec 12, 2024

  • hpo torch の依存関係を個別にインストールできるように改修しました
    • 依存関係の記述を setup.py に移動させ、all ですべての依存関係をインストールできるように改修しました
    • おそらく setuptools の影響で、pip install . 実行時に build ディレクトリが生成されるようになったので、 mypy 実行時には build を無視するように改修しました
  • hpo torch の指定に合わせて、aiaccel/hpo aiaccel/toch を個別にインストールする機能については、探してもそれらしい機能や実装例がなかったので、未実装です

@KanaiYuma-aist
Copy link
Collaborator Author

  • setup.py に all_dev を追加し、all = torch+hpo に変更しました
  • docs/source/getting_started.md に hpo torch を個別インストールするコマンドを追加しました

@yoshipon
Copy link
Collaborator

ありがとうございます!

@yoshipon yoshipon merged commit 31dd240 into develop/v2 Dec 17, 2024
10 checks passed
@yoshipon yoshipon deleted the feature/v2-dependencies branch December 17, 2024 10:45
aramoto99 pushed a commit that referenced this pull request Dec 18, 2024
* Add optional-dependencies using setuptools.

* Fix for tests.

* Fix for tests.

* Add all_dev.

* Fix all to all_dev.

* Add dependencies description to the docs.

* Fix typo.

* Update setup.py

---------

Co-authored-by: Yoshiaki Bando <yoshipon@users.noreply.github.com>
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