[GitHub] marcoabreu edited a comment on issue #11636: [MXNET-769] set MXNET_HOME as base for downloaded models through base.data_dir()

[GitBox <gi...@apache.org>]

marcoabreu edited a comment on issue #11636: [MXNET-769] set MXNET_HOME as base for downloaded models through base.data_dir()
URL: https://github.com/apache/incubator-mxnet/pull/11636#issuecomment-409744523
 
 
   Yes. I am concerned that we are breaking backwards compatibility if we merge it as-is.
   
   I personally don't think that mxnet home is a good variable if our convention is that it points to the source (I didn't check, just judging from the conversation here). We dont want the models to be stored in the git managed repository or pollute the workspace.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services