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

make CI more debug-friendly #312

Merged
merged 3 commits into from Aug 23, 2018
Merged

make CI more debug-friendly #312

merged 3 commits into from Aug 23, 2018

Conversation

szha
Copy link
Member

@szha szha commented Aug 22, 2018

Description

@szha szha requested a review from leezu August 22, 2018 21:45
@szha szha changed the title force install in CI make CI more debug-friendly Aug 22, 2018
@codecov
Copy link

codecov bot commented Aug 23, 2018

Codecov Report

Merging #312 into master will increase coverage by 2.5%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #312     +/-   ##
=========================================
+ Coverage   77.29%   79.79%   +2.5%     
=========================================
  Files          80       80             
  Lines        7077     6503    -574     
  Branches     1168     1022    -146     
=========================================
- Hits         5470     5189    -281     
+ Misses       1346     1062    -284     
+ Partials      261      252      -9
Impacted Files Coverage Δ
scripts/nmt/utils.py 78.57% <0%> (-15.55%) ⬇️
scripts/nmt/loss.py 61.01% <0%> (-10.75%) ⬇️
gluonnlp/model/__init__.py 89.47% <0%> (-10.53%) ⬇️
scripts/nmt/bleu.py 84.09% <0%> (-4.68%) ⬇️
conftest.py 79.24% <0%> (-3.78%) ⬇️
scripts/sentiment_analysis/sentiment_analysis.py 86.36% <0%> (-2.53%) ⬇️
gluonnlp/vocab/vocab.py 94.48% <0%> (-0.69%) ⬇️
scripts/language_model/word_language_model.py 0% <0%> (-0.31%) ⬇️
scripts/nmt/_constants.py 100% <0%> (ø) ⬆️
scripts/language_model/cache_language_model.py 0% <0%> (ø) ⬆️
... and 4 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 70bb562...6d31d55. Read the comment docs.

@szha szha force-pushed the force_install branch 3 times, most recently from 2d259bc to 6d31d55 Compare August 23, 2018 05:33
@mli
Copy link
Member

mli commented Aug 23, 2018

Job PR-312/5 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-312/5/index.html

@mli
Copy link
Member

mli commented Aug 23, 2018

Job PR-312/6 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-312/6/index.html

@szhengac szhengac merged commit e945770 into dmlc:master Aug 23, 2018
@szha szha deleted the force_install branch August 23, 2018 15:35
paperplanet pushed a commit to paperplanet/gluon-nlp that referenced this pull request Jun 9, 2019
* force install

* echo xargs commands

* update dep
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.

None yet

3 participants