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 2021/01/13 15:02:46 UTC

[GitHub] [tvm] gromero opened a new pull request #7269: [µTVM] Fix two warnings when deprecated forms are used

gromero opened a new pull request #7269:
URL: https://github.com/apache/tvm/pull/7269


   Hi,
   
   This patchset addresses the following two warnings on microTVM:
   
   [0]
   ```
   [...]
   /home/gromero/git/tvm/python/tvm/micro/contrib/zephyr.py:391: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
     doc = yaml.load(f)
   [...]
   ```
   
   [1]
   ```
   [...]
   /home/gromero/git/tvm/python/tvm/target/target.py:460: UserWarning: tvm.target.create() is being deprecated. Please use tvm.target.Target() instead
     warnings.warn("tvm.target.create() is being deprecated. Please use tvm.target.Target() instead")
   [...]
   ```
   
   Warning [0] is related to the deprecated form (no loader specified) of yaml.load(input), while the second one is related to the use of  deprecated tvm.target.create().
   
   Both warnings appear  when running `micro_tflite.py`. The first one [0] when `TARGET = tvm.target.target.micro("stm32f746xx")` and the second one [1] when `TARGET = tvm.target.target.micro("host")`, which is the default target.
   
   Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from [Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers) by @ them in the pull request thread.
   


----------------------------------------------------------------
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] [tvm] areusch commented on pull request #7269: [µTVM] Fix two warnings when deprecated forms are used

Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #7269:
URL: https://github.com/apache/tvm/pull/7269#issuecomment-759717592


   thanks for fixing these @gromero! @tmoreau89 @tqchen 


----------------------------------------------------------------
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] [tvm] tmoreau89 commented on pull request #7269: [µTVM] Fix two warnings when deprecated forms are used

Posted by GitBox <gi...@apache.org>.
tmoreau89 commented on pull request #7269:
URL: https://github.com/apache/tvm/pull/7269#issuecomment-759864070


   Thanks @gromero @areusch the PR is merged.


----------------------------------------------------------------
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] [tvm] tmoreau89 merged pull request #7269: [µTVM] Fix two warnings when deprecated forms are used

Posted by GitBox <gi...@apache.org>.
tmoreau89 merged pull request #7269:
URL: https://github.com/apache/tvm/pull/7269


   


----------------------------------------------------------------
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] [tvm] gromero commented on pull request #7269: [µTVM] Fix two warnings when deprecated forms are used

Posted by GitBox <gi...@apache.org>.
gromero commented on pull request #7269:
URL: https://github.com/apache/tvm/pull/7269#issuecomment-759509646


   @areusch Hi. Could you review that small patchset, please? For warning [0] I'm assuming microTVM is not locked with any old PyYAML version (i.e. < 5.1). If it's locked we could simply silence the warning, which is not so nice I think. Thanks!


----------------------------------------------------------------
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