DickJC123 edited a comment on issue #14392: Debug bulking test on windows [WIP]
URL: https://github.com/apache/incubator-mxnet/pull/14392#issuecomment-472124999
 
 
   If this latest commit passes CI, it will be ready for merging.  The problem 
is that when one performs in Python:
   ```
   os.environ[<key>] = <value>
   ```
   this effects what the C++ backend sees only on Linux.  Under Windows, some 
additional (and possibly windows-version-specific) work would be required to 
push the environment change to the msvcrt, which otherwise gets a copy of the 
environment at process start-up.  My preferred fix here is to resort to the 
more bomb-proof approach of setting environment variables, then spawning a new 
process to perform the test (although admittedly this is bypassed on Python2 
due to lack of spawn support).

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to