nflows icon indicating copy to clipboard operation
nflows copied to clipboard

Published 20 hours ago • verified publisher icon bayesiains

Unnecessary dependencies

Open francois-rozet opened this issue 3 years ago • 0 comments

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

francois-rozet avatar Dec 17 '21 21:12 francois-rozet