[GitHub] [tvm] leandron commented on pull request #8331: Allow tvmc to compile models with AOT executor in MLF

[GitBox <gi...@apache.org>]

leandron commented on pull request #8331:
URL: https://github.com/apache/tvm/pull/8331#issuecomment-871212538


   This is is failing in the latest version of this PR. I'm pasting the error below, to save a trip to Jenkins:
   
   ```
   ==================================== ERRORS ====================================
   
   ____________ ERROR collecting tests/python/driver/tvmc/test_mlf.py _____________
   /usr/local/lib/python3.6/dist-packages/pluggy/hooks.py:286: in __call__
       return self._hookexec(self, self.get_hookimpls(), kwargs)
   /usr/local/lib/python3.6/dist-packages/pluggy/manager.py:93: in _hookexec
       return self._inner_hookexec(hook, methods, kwargs)
   /usr/local/lib/python3.6/dist-packages/pluggy/manager.py:87: in <lambda>
       firstresult=hook.spec.opts.get("firstresult") if hook.spec else False,
   /usr/local/lib/python3.6/dist-packages/_pytest/python.py:249: in pytest_pycollect_makeitem
       res = list(collector._genfunctions(name, obj))
   /usr/local/lib/python3.6/dist-packages/_pytest/python.py:470: in _genfunctions
       self.ihook.pytest_generate_tests.call_extra(methods, dict(metafunc=metafunc))
   /usr/local/lib/python3.6/dist-packages/pluggy/hooks.py:324: in call_extra
       return self(**kwargs)
   /usr/local/lib/python3.6/dist-packages/pluggy/hooks.py:286: in __call__
       return self._hookexec(self, self.get_hookimpls(), kwargs)
   /usr/local/lib/python3.6/dist-packages/pluggy/manager.py:93: in _hookexec
       return self._inner_hookexec(hook, methods, kwargs)
   /usr/local/lib/python3.6/dist-packages/pluggy/manager.py:87: in <lambda>
       firstresult=hook.spec.opts.get("firstresult") if hook.spec else False,
   conftest.py:35: in pytest_generate_tests
       tvm.testing._auto_parametrize_target(metafunc)
   python/tvm/testing.py:800: in _auto_parametrize_target
       for mark in metafunc.definition.iter_markers("parametrize")
   python/tvm/testing.py:801: in <listcomp>
       for arg in mark.args[0].split(",")
   E   AttributeError: 'list' object has no attribute 'split'
   =============================== warnings summary ===============================
   ../usr/local/lib/python3.6/dist-packages/tensorflow/python/autograph/utils/testing.py:21
     /usr/local/lib/python3.6/dist-packages/tensorflow/python/autograph/utils/testing.py:21: DeprecationWarning: the imp module is eprecated in favour of importlib; see the module's documentation for alternative uses
       import imp
   
   - Docs: https://docs.pytest.org/en/stable/warnings.html
   - generated xml file: /workspace/build/pytest-results/python-integration-driver-ctypes.xml -
   =========================== short test summary info ============================
   ERROR tests/python/driver/tvmc/test_mlf.py - AttributeError: 'list' object ha...
   !!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
   ========================= 1 warning, 1 error in 0.97s ==========================
   ```
   
   Apart from that, LGTM. Happy to review/merge once we get CI working.


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org