search

openairplay / airplay2-receiver

AirPlay 2 Receiver - Python implementation
2.12k stars 131 forks source link

Remove numpy from requirements #69

Closed m3brown closed 2 years ago

m3brown commented 2 years ago

numpy is currently unused in the source code. Since numpy is a rather large package to build and bundle, my opinion is that it doesn't make sense to include if it's not in use.

If there's a reason for its presence in the requirements.txt (e.g. for debugging?), perhaps it makes sense to provide a comment in requirements.txt explaining the rationale.

Note: it also looks like requests is unused, should we remove that from requirements.txt too?

systemcrash commented 2 years ago

I did my best to avoid using it for this reason, just never got around to doing this bit :)