You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/07/17 00:01:35 UTC

[GitHub] [incubator-tvm] tkonolige opened a new pull request #6083: [WIP] Fix LocalBuilder on macOS with python 3.8.

tkonolige opened a new pull request #6083:
URL: https://github.com/apache/incubator-tvm/pull/6083


   Python 3.8 changes the default way multiprocessing creates new processes on macOS from forking to spawing. Spawning requires all objects to be picklable. Nested functions and lambdas are not picklable, so this commit fixes the one instance of nested functions in the codebase that was causing issues.
   
   Although I've fixed the error from objects not being picklable, I'm getting an error in `tests/python/unittest/test_autotvm_measure.py` where registered tasks aren't found. Looking in `python/tvm/autotvm/measure/local_executor.py`, the global `TASK_TABLE` is empty in some child processes. The error only occurs with python 3.8 and pytest. Running  `tests/python/unittest/test_autotvm_measure.py` directly (without pytest) gives no errors. I'd assume this issue is related to multiprocessing.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen commented on pull request #6083: [WIP] Fix LocalBuilder on macOS with python 3.8.

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6083:
URL: https://github.com/apache/incubator-tvm/pull/6083#issuecomment-660150229


   Thanks @tkonolige !


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] tkonolige commented on pull request #6083: [WIP] Fix LocalBuilder on macOS with python 3.8.

Posted by GitBox <gi...@apache.org>.
tkonolige commented on pull request #6083:
URL: https://github.com/apache/incubator-tvm/pull/6083#issuecomment-659743769


   @tqchen @junrushao1994 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen merged pull request #6083: [WIP] Fix LocalBuilder on macOS with python 3.8.

Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6083:
URL: https://github.com/apache/incubator-tvm/pull/6083


   


----------------------------------------------------------------
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:
users@infra.apache.org