graphics icon indicating copy to clipboard operation
graphics copied to clipboard

Remove OpenEXR from requirements and warning

Open jonas-eschle opened this issue 5 years ago • 2 comments

The docs and code say that OpenEXR is an optional dependency (https://github.com/tensorflow/graphics/blob/6b905c68e802b73c04857c21cca2e026cc37c900/tensorflow_graphics/util/data_formats/init.py#L25 or here but is seemingly is not. This means I cannot install the TFG package, since the OpenEXR does not get installed properly. Furthermore, the warning is rather annoying, as we would like to use some simple functionality from the package (bsplines) in a completely different context.

I would propose to a) remove OpenEXR from the dependency list and make it optional (as documented in docs and code) b) remove the warning and instead make OpenEXR a local import inside the function. If the function is called and it cannot be imported, it fails saying that it has to install OpenEXR first.

What do you think? Glad also to make this changes and contribute

jonas-eschle avatar Jul 11 '20 20:07 jonas-eschle

+1 on this, having to build an unwanted dependency from source is a burden

atyshka avatar Sep 22 '20 03:09 atyshka

Is there any news on this?

jonas-eschle avatar Nov 10 '20 13:11 jonas-eschle