search

dmlc / gluon-nlp

NLP made easy
https://nlp.gluon.ai/
Apache License 2.0
2.56k stars 538 forks source link

[CI] [TVM Integration] Install from tlcpack wheels #1400

Open sxjscience opened 3 years ago

sxjscience commented 3 years ago

Description

Currently, we compile TVM from scratch in our CI: https://github.com/dmlc/gluon-nlp/blob/8c8b0c9cda0853caa88fdbf4e0544986fbef243c/.github/workflows/unittests.yml#L60-L74.

As suggested by @tqchen , we can later install from https://github.com/tlc-pack/tlcpack . However, as mentioned in https://github.com/tlc-pack/tlcpack/issues/7, the pip nightly is still WIP. We may wait until then to revise our CI pipeline.

tqchen commented 3 years ago

FYI the conda nightly is already available https://anaconda.org/tlcpack That does mean a dependency of a conda env instead of normal python though

leezu commented 3 years ago

There can be issues when mixing system and conda toolchain. When using conda, we need to be careful to avoid that or we may experience issues such as https://github.com/dmlc/gluon-nlp/issues/1371 again. Thus the easiest route may be waiting for a https://www.python.org/dev/peps/pep-0599/ compliant build