Skip to content

Commit 18c78ba

Browse files
ddevilledavedittrich
authored andcommitted
Fix invalid pyproject.toml file and move to setup.cfg (openai#201)
The `[project]` section of a `pyproject.toml` file *has* to include both a `name` and a `version`, that we were missing. However, rather than adding more stuff to `pyproject.toml`, I've moved the license info to the setup file and pointed `pyproject.toml` to use `setuptools` for building (the `pyproject.toml` file is really only needed to integrated with `black`, that doesn't support setup files...) While I was at it I decided to migrate to a `setup.cfg` file rather than `setup.py` since its declaritive config is usally preferred these days (and it's a bit easier to load from version/license files than with a `setup.py` file).
1 parent 48fcfa1 commit 18c78ba

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

setup.cfg

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,9 @@ embeddings =
4949
scikit-learn >= 1.0.2 # Needed for embedding utils, versions >= 1.1 require python 3.8
5050
tenacity >= 8.0.1
5151
matplotlib
52-
sklearn
5352
plotly
5453
numpy
54+
scipy
5555
pandas >= 1.2.3 # Needed for CLI fine-tuning data preparation tool
5656
pandas-stubs >= 1.1.0.11 # Needed for type hints for mypy
5757
openpyxl >= 3.0.7 # Needed for CLI fine-tuning data preparation tool xlsx format

0 commit comments

Comments
 (0)