Update README for Windows and OpenBLAS details#11
Conversation
Clarified assumptions for Windows environment and OpenBLAS installation paths.
There was a problem hiding this comment.
Pull Request Overview
This PR updates the README documentation to provide clearer instructions about OpenBLAS installation paths and CMake configuration for Windows environments using vcpkg.
- Added clarification about default CMake assumptions for Windows/vcpkg/OpenBLAS setup
- Updated build instructions to specify the vcpkg installation location (C drive root)
- Added guidance for users with non-Windows environments or alternative OpenBLAS installation paths
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - C++17 以上（C++20 推奨） | ||
| - CMake 3.15 以上（CMake を使う例を示します） | ||
| - （任意）OpenBLAS（性能を出したい場合） | ||
| CMakeListsでは初期値でwindows環境かつvcpkgフォルダがCドライブ直下にあり、OPENBLASがインストール済みであることを想定しています。 |
There was a problem hiding this comment.
Missing space after 'CMakeLists'. Should be 'CMakeLists では' instead of 'CMakeListsでは' to maintain proper spacing between the filename and the Japanese particle.
| - 以下は CMake を使った一例です。`USE_OPENBLAS` オプションを用意し、OpenBLAS のヘッダとライブラリが見つかった場合にのみ `USE_OPENBLAS` を定義してリンクします。 | ||
| - Windows環境を想定しているため、vcpkgでOpenBLASをインストールした場合のパスを例示しています。 | ||
| - Windows環境を想定しているため、Cドライブ直下のvcpkgでOpenBLASをインストールした場合のパスを例示しています。 | ||
| - Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください |
There was a problem hiding this comment.
Missing punctuation. The sentence should end with a period (。) following Japanese writing conventions.
| - Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください | |
| - Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください。 |
| - 以下は CMake を使った一例です。`USE_OPENBLAS` オプションを用意し、OpenBLAS のヘッダとライブラリが見つかった場合にのみ `USE_OPENBLAS` を定義してリンクします。 | ||
| - Windows環境を想定しているため、vcpkgでOpenBLASをインストールした場合のパスを例示しています。 | ||
| - Windows環境を想定しているため、Cドライブ直下のvcpkgでOpenBLASをインストールした場合のパスを例示しています。 | ||
| - Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください |
There was a problem hiding this comment.
Missing comma or reading marker. Should be '場合、適宜' or '場合は適宜' instead of '場合適宜' for proper Japanese grammar.
| - Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合適宜[プリセット](CMakePresets.json)に変更を加えてください | |
| - Windows環境でない場合や別ディレクトリでOpenBLASライブラリをインストールしている場合、適宜[プリセット](CMakePresets.json)に変更を加えてください |
1 participant
Clarified assumptions for Windows environment and OpenBLAS installation paths.