bayesiains / nflows

Normalizing flows in PyTorch
MIT License
849 stars 118 forks source link

Unnecessary dependencies #53

Open francois-rozet opened 2 years ago

francois-rozet commented 2 years ago

Hello,

The dependencies matplotlib, tensorboard and tqdm are installed by setup.py (and environment.yml) even though they are not used by nflows. They should be removed.

https://github.com/bayesiains/nflows/blob/ac0bf432fc7904458a933ed14180f0ac26e3f93d/setup.py#L22-L26

It should be noted that matplotlib is imported (but not used) here

https://github.com/bayesiains/nflows/blob/639c3a771d57c29a27c307140cc94a1008ee9f55/nflows/nn/nde/made.py#L6