search

netdevopsbr / netbox-proxbox

Netbox Plugin for integration between Proxmox and Netbox
Apache License 2.0
341 stars 50 forks source link

remove numpy and matplotlib - they do not seem to be in use #57

Closed klaernie closed 2 years ago

klaernie commented 2 years ago

This removes the requirements of numpy and matplotlib. Both libraries are not used directly in this module, and also searching the issues only returns #39, which simply lists the installed packages.

Why remove them? (besides the obvious "keep the dependencies lean") While no current package is available on PyPI, I started to setup the package in my netbox-docker container the long way, which means installing it from git. But installing in the container fails at installing matplotlib, as there is no gcc available in the base image (which is the official netbox image). So I wondered, considering the purpose of the project, what numpy and matplotlib could be used for, and I searched for the usage of the libraries.

emersonfelipesp commented 2 years ago

I really will not use it by now. Thank you!