[GitHub] leleamol commented on a change in pull request #9124: Removing the PYTHONPATH inserts from the examples to avoid portabiliity issues.

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

leleamol commented on a change in pull request #9124: Removing the PYTHONPATH inserts from the examples to avoid portabiliity issues.
URL: https://github.com/apache/incubator-mxnet/pull/9124#discussion_r157590307
 
 

 ##########
 File path: example/ctc/ocr_predict.py
 ##########
 @@ -21,8 +21,6 @@
 from __future__ import print_function
 import sys, os
 curr_path = os.path.dirname(os.path.abspath(os.path.expanduser(__file__)))
-sys.path.append("../../amalgamation/python/")
-sys.path.append("../../python/")
 
 from mxnet_predict import Predictor
 
 Review comment:
   When I was trying out these tests, I had to manually comment out these lines.
   The ocr_predict.py used to proceed correctly and was able to find libmxnet_amalgmation.so  but it used to fail in loading the dependent libraries of libmxnet_amalagmation.so.  This might be due to the error in building amalgmation library on my machine.

----------------------------------------------------------------
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