Lunderberg commented on a change in pull request #9203:
URL: https://github.com/apache/tvm/pull/9203#discussion_r725255992
##########
File path: docs/conf.py
##########
@@ -51,10 +51,11 @@
else:
tvm_path = Path(os.pardir)
-
sys.path.insert(0, str(tvm_path / "python"))
sys.path.insert(0, str(tvm_path / "vta" / "python"))
+os.environ["PYTHONPATH"] = os.pathsep.join(sys.path)
Review comment:
This one was an earlier attempt to fix the issue
(https://github.com/apache/tvm/pull/9203#issuecomment-936676542). The changes
to `sys.path` were to make sure that `$TVM_HOME/python` is locatable, even if
the cwd is within the new `_staging` directory. The updates to `PYTHONPATH`
were to mirror that to subprocesses, since that appeared to be where the issue
was. After testing, that ended up not being the issue after all.
--
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]
