DickJC123 commented on pull request #20940: URL: https://github.com/apache/incubator-mxnet/pull/20940#issuecomment-1063164058
FYI, issue https://github.com/apache/incubator-mxnet/issues/20869 discussing the numpy upgrade motivation had the comment from @huubvh95: ``` Is it also possible to update Graphviz in the PR? The required version for Graphviz is rather old. See issue https://github.com/apache/incubator-mxnet/issues/20897 ``` My feeling on this: ``` I'm not able to do a reasonable job of testing an updated version of graphviz. If we put the graphviz update in this PR, then a required revert based on either a numpy or graphviz problem would revert both modules- an unfortunate coupling. Could you suggest a graphviz version to move to, based on some testing, and make a separate PR for that? ``` -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
