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

[Numpy] Benchmark the backbone models + Some fixes + Always use python3 + Fix conversion tool #1292

Merged
merged 69 commits into from
Aug 14, 2020

Conversation

sxjscience
Copy link
Member

@sxjscience sxjscience commented Aug 10, 2020

Description

After this PR is merged, you can

  • Benchmarking the performance of backbone models in GluonNLP by running scripts/benchmarks/benchmark_gluonnlp.py
  • Always use python3 and pip3 as discussed with @szha and @leezu .
  • Test for the conversion scripts. (Next PR, this one will revise the conversion scripts)

cc @dmlc/gluon-nlp-team

update

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Create requirements.txt

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update requirements.txt

update

Update README.md

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

Update benchmark_hf.py

fix

fix

Update test_models_bart.py

Update test_models_bart.py

Update bart.py

update

Update __init__.py

Update electra.py

update

update

Update convert_bert_from_tf_hub.sh

update

Update unittests.yml

fix conversion

update

fix bert conversion

update

fix

fix

Update __init__.py

fix bug

fix css

Update benchmark_utils.py

Update benchmark_utils.py

update

update

Update misc.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

no multiprocessing

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

fix bug

Update benchmark_utils.py

Update benchmark_utils.py

try to use mxnet profiler

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

fix

update

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

fix

Update benchmark_utils.py

Update bart.py

Update bart.py

fix

fix

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_gluonnlp.py

Update benchmark_gluonnlp.py

Update benchmark_gluonnlp.py

Update benchmark_utils.py

Update benchmark_utils.py

Update benchmark_utils.py

Update README.md
@sxjscience sxjscience changed the title [WIP] Benchmark the Backbone Models + Some Fixes [WIP] Benchmark the Backbone Models + Fixes + Always use python3 Aug 10, 2020
@sxjscience sxjscience changed the title [WIP] Benchmark the Backbone Models + Fixes + Always use python3 [WIP] Benchmark the backbone models + Some fixes + Always use python3 + Test conversion tool Aug 10, 2020
@sxjscience sxjscience changed the title [WIP] Benchmark the backbone models + Some fixes + Always use python3 + Test conversion tool [Numpy][WIP] Benchmark the backbone models + Some fixes + Always use python3 + Test conversion tool Aug 10, 2020
@codecov
Copy link

codecov bot commented Aug 10, 2020

Codecov Report

Merging #1292 into master will increase coverage by 0.74%.
The diff coverage is 61.81%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1292      +/-   ##
==========================================
+ Coverage   83.62%   84.36%   +0.74%     
==========================================
  Files          42       42              
  Lines        6382     6397      +15     
==========================================
+ Hits         5337     5397      +60     
+ Misses       1045     1000      -45     
Impacted Files Coverage Δ
setup.py 0.00% <ø> (ø)
src/gluonnlp/layers.py 86.78% <ø> (ø)
src/gluonnlp/models/albert.py 95.47% <ø> (ø)
src/gluonnlp/models/bert.py 94.86% <ø> (ø)
src/gluonnlp/models/electra.py 76.45% <15.38%> (-0.65%) ⬇️
src/gluonnlp/models/bart.py 86.25% <50.00%> (+1.08%) ⬆️
src/gluonnlp/utils/misc.py 51.89% <93.75%> (+8.49%) ⬆️
src/gluonnlp/models/__init__.py 96.66% <100.00%> (+0.11%) ⬆️
src/gluonnlp/op.py 95.08% <100.00%> (+35.08%) ⬆️
src/gluonnlp/models/transformer.py 98.94% <0.00%> (+0.42%) ⬆️
... and 3 more

@szha szha changed the base branch from numpy to master August 13, 2020 02:29
@sxjscience
Copy link
Member Author

Initial numbers obtained via the benchmark:

See the shared google sheet: https://docs.google.com/spreadsheets/d/1ohQ2P4PNDxciz1_gWJaGzD-LeXsvX5MSe-qMoqnRaxc/edit?usp=sharing

@sxjscience sxjscience changed the title [Numpy][WIP] Benchmark the backbone models + Some fixes + Always use python3 + Test conversion tool [Numpy] Benchmark the backbone models + Some fixes + Always use python3 + Test conversion tool Aug 14, 2020
@sxjscience sxjscience changed the title [Numpy] Benchmark the backbone models + Some fixes + Always use python3 + Test conversion tool [Numpy] Benchmark the backbone models + Some fixes + Always use python3 + Fix conversion tool Aug 14, 2020
Copy link
Member

@zheyuye zheyuye left a comment

Choose a reason for hiding this comment

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

Looks good overall

@zheyuye zheyuye merged commit 32e87d4 into dmlc:master Aug 14, 2020
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.

2 participants