search

bandframework / surmise

A python package for surrogate models that interface with calibration and other tools
https://surmise.readthedocs.io/en/latest/
MIT License
10 stars 7 forks source link

`setup.py` vs `pyproject.toml` #137

Open jared321 opened 2 weeks ago

jared321 commented 2 weeks ago

The package currently has both a pyproject.toml file and a setup.py file in place. The former contains the majority of the package specification; the latter, content for converting a single .pyx file to a Python module in the package.

According to the Cython docs, using setup.py is the recommended method. Evaluate the possibility of using just or the other but not both.

Note that the source distribution and binary wheels presently contain files such as documentation, example code, GH actions, etc. Is this as intended or can we remove those?

mosesyhc commented 2 weeks ago

I believe we should remove non-codebase files from the source distribution and binary wheels.

FWIW, at one point __init__.py is included in tests/ and will take up the tests namespace. Those were removed.