echuraev commented on pull request #9032: URL: https://github.com/apache/tvm/pull/9032#issuecomment-922084902
> Did you try set `args=None` or set all elements to None (the predefined arrays are also converted to ndarray from numpy array here in each popen workers locally, which should also be freed) Otherwise the alternative fix looks good to me as it avoids deep copying Yes, I did it and it works fine. I wrote this code with indices just because I wasn't sure that I can set `None` to the args which were predefined. In this case, if you are ok with alternative fix (with setting `None` to arguments) then, I'll update this PR tomorrow. Thank you very much. Only one thing that I don't really like in this fix is that we need to set these elements to `None` in both cases (after executed function and also in case when exception was generated). -- 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]
