fix: setup doesn't require numpy

higlass / clodius

Clodius is a tool for breaking up large data sets into smaller tiles that can subsequently be displayed using an appropriate viewer.

MIT License

39 stars 21 forks source link

fix: setup doesn't require numpy #148

Closed manzt closed 1 year ago

manzt commented 1 year ago

Description

What was changed in this pull request?

Removes numpy from setup.py setup_requires.

Why is it necessary?

Setup doesn't actually require numpy, and pip install -e . fails with numpy v1.24.2. We can avoid this all together by removing it. If we need a build dependency, it should be specified with build-system in a pyproject.toml.

Fixes #___

Checklist

[ ] Unit tests added or updated
[ ] Updated CHANGELOG.md
[ ] Run black .

nvictus commented 1 year ago

Yeah, this must have been leftover from some earlier cython modules.