Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

[TVM] Add TVM Support #1390

Merged
merged 72 commits into from
Oct 24, 2020
Merged

[TVM] Add TVM Support #1390

merged 72 commits into from
Oct 24, 2020

Conversation

sxjscience
Copy link
Member

@sxjscience sxjscience commented Oct 19, 2020

Description

Add TVM test case + profiling after apache/tvm#6699 is merged.

  • Test case
  • Profile

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

cc @dmlc/gluon-nlp-team

@github-actions
Copy link

2 similar comments
@github-actions
Copy link

@github-actions
Copy link

@codecov
Copy link

codecov bot commented Oct 19, 2020

Codecov Report

Merging #1390 into master will decrease coverage by 0.22%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1390      +/-   ##
==========================================
- Coverage   85.35%   85.12%   -0.23%     
==========================================
  Files          53       53              
  Lines        6959     6946      -13     
==========================================
- Hits         5940     5913      -27     
- Misses       1019     1033      +14     
Impacted Files Coverage Δ
src/gluonnlp/models/albert.py 95.42% <ø> (-0.05%) ⬇️
src/gluonnlp/models/bert.py 94.80% <ø> (-0.06%) ⬇️
src/gluonnlp/models/electra.py 76.90% <ø> (+0.44%) ⬆️
src/gluonnlp/models/mobilebert.py 87.85% <ø> (-0.10%) ⬇️
src/gluonnlp/models/roberta.py 93.61% <ø> (-0.06%) ⬇️
src/gluonnlp/utils/lazy_imports.py 59.75% <100.00%> (+5.43%) ⬆️
src/gluonnlp/utils/misc.py 59.20% <100.00%> (+0.37%) ⬆️
src/gluonnlp/data/loading.py 78.11% <0.00%> (-5.29%) ⬇️
src/gluonnlp/data/filtering.py 78.26% <0.00%> (-4.35%) ⬇️
src/gluonnlp/data/tokenizers/subword_nmt.py 78.50% <0.00%> (-0.94%) ⬇️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7910d6d...1e1b05d. Read the comment docs.

@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

2 similar comments
@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@github-actions
Copy link

@sxjscience
Copy link
Member Author

sxjscience commented Oct 24, 2020

@leezu I’ve removed the LLVM installation code, and the CI has passed.

Copy link
Member

@szha szha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! this is an important integration

@sxjscience sxjscience merged commit 8c8b0c9 into dmlc:master Oct 24, 2020
@tqchen
Copy link
Member

tqchen commented Oct 24, 2020

FYI https://github.com/tlc-pack/tlcpack now provides nightly thirdparty build for tvm, packages can be installed from https://anaconda.org/tlcpack if you are using conda, this might be easier than building tvm from scratch every time

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants