larroy commented on a change in pull request #15715: Improve diagnose.py to display environment variables URL: https://github.com/apache/incubator-mxnet/pull/15715#discussion_r309871961
########## File path: tools/diagnose.py ########## @@ -171,6 +161,31 @@ def check_network(args): for name, url in URLS.items(): test_connection(name, url, args.timeout) + +def check_environment(): + print('----------Environment----------') + for k,v in os.environ.items(): + print('{}="{}"'.format(k,v)) + + +def parse_args(): Review comment: usually you want it next to main, instead of scrolling all the way up. ---------------------------------------------------------------- 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 With regards, Apache Git Services