Skip to content

Commit d8db548

Browse files
authored
Fix TOML parsing errors in pyproject.toml for scikit-build configuration (#27)
* Update wheel configuration in pyproject.toml to include package tree * Added python dependency and shell globbing for metal command
1 parent a6d9d90 commit d8db548

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,7 @@ This repository provides a collection of reference implementations:
130130

131131
### Requirements
132132

133+
- python 3.12
133134
- On macOS: Install the Xcode CLI tools --> `xcode-select --install`
134135
- On Linux: These reference implementations require CUDA
135136
- On Windows: These reference implementations have not been tested on Windows. Try using solutions like Ollama if you are trying to run the model locally.
@@ -151,7 +152,7 @@ If you want to modify the code or try the metal implementation set the project u
151152

152153
```shell
153154
git clone https://github.com/openai/gpt-oss.git
154-
pip install -e .[metal]
155+
pip install -e ".[metal]"
155156
```
156157

157158
## Download the model

pyproject.toml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,10 @@ packages = ["gpt_oss"]
4242

4343
[tool.scikit-build]
4444
cmake.source-dir = "." # pick up the root CMakeLists.txt
45-
wheel.packages = ["gpt_oss"] # copy the whole Python package tree
4645
cmake.args = [
4746
"-DGPTOSS_BUILD_PYTHON=ON",
4847
"-DCMAKE_BUILD_TYPE=Release",
4948
"-DBUILD_SHARED_LIBS=OFF",
5049
]
5150
[tool.scikit-build.wheel]
52-
plat-name = "manylinux_2_17_x86_64"
51+
packages = ["gpt_oss"] # copy the whole Python package tree

0 commit comments

Comments
 (0)