Skip to content

Update README for Windows and OpenBLAS details #11

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.

Sign up for GitHub

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

Open
SanaeProject wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update README for Windows and OpenBLAS details #11

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
- C++17 以上（C++20 推奨）
- CMake 3.15 以上（CMake を使う例を示します）
- （任意）OpenBLAS（性能を出したい場合）
CMakeListsでは初期値でwindows環境かつvcpkgフォルダがCドライブ直下にあり、OPENBLASがインストール済みであることを想定しています。
Copy link

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after 'CMakeLists'. Should be 'CMakeLists では' instead of 'CMakeListsでは' to maintain proper spacing between the filename and the Japanese particle.

Copilot uses AI. Check for mistakes.

## OpenBLAS の利用ポリシー

Expand All @@ -60,7 +61,8 @@ OpenBLAS を利用する場合、本リポジトリでは次の条件を満た
## ビルド手順

- 以下は CMake を使った一例です。`USE_OPENBLAS` オプションを用意し、OpenBLAS のヘッダとライブラリが見つかった場合にのみ `USE_OPENBLAS` を定義してリンクします。
- Windows環境を想定しているため、vcpkgでOpenBLASをインストールした場合のパスを例示しています。
- Windows環境を想定しているため、Cドライブ直下のvcpkgでOpenBLASをインストールした場合のパスを例示しています。
- Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください
Copy link

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing punctuation. The sentence should end with a period (。) following Japanese writing conventions.

Suggested change
- Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください
- Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください。

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing comma or reading marker. Should be '場合、適宜' or '場合は適宜' instead of '場合適宜' for proper Japanese grammar.

Suggested change
- Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください
- Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合、適宜[プリセット](CMakePresets.json)に変更を加えてください

Copilot uses AI. Check for mistakes.

```cmake
# CMakeLists.txt 抜粋
Expand Down Expand Up @@ -204,4 +206,4 @@ int main() {
- GPU サポート（CUDA / ROCm）
- 追加の最適化アルゴリズム（Adam 等）

もしこのREADMEの内容で追加・修正したい点があれば教えてください。ビルド設定が別のシステム（Makefile, Bazel 等）なら、そのビルド手順に合わせて文面を調整します。
もしこのREADMEの内容で追加・修正したい点があれば教えてください。ビルド設定が別のシステム（Makefile, Bazel 等）なら、そのビルド手順に合わせて文面を調整します。