Reamer commented on a change in pull request #4078:
URL: https://github.com/apache/zeppelin/pull/4078#discussion_r592996201



##########
File path: scripts/docker/zeppelin/bin/env_python_3_with_R.yml
##########
@@ -17,7 +17,7 @@ dependencies:
   - jupyter_client=5
   - bokeh=1.3.4
   - panel
-  - holoviews
+  - holoviz=0.11.6

Review comment:
       Very interesting package. But maybe we should give users the possibility 
to build the Docker image with the latest version of holoviz without changing 
the version in the Dockerfile.
   Since holoviz is a meta package, it might make sense to remove the 
[dependencies](https://github.com/holoviz/holoviz/blob/master/setup.cfg#L33-L50).
 e.g. numpy, bokeh, matplotlib
   A small comment for this particular dependency should also be placed.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to