search

chainer / chainer-chemistry

Chainer Chemistry: A Library for Deep Learning in Biology and Chemistry
MIT License
622 stars 129 forks source link

add mark.slow flag in test_flow_scaler #379

Closed corochann closed 4 years ago

corochann commented 4 years ago

test_flow_scaler takes time because it internally trains model inside fit method. This PR adds "slow" flag to skip test for CI.

Instead I added simple test to only check to run each method.

mottodora commented 4 years ago

/test

pfn-ci-bot commented 4 years ago

Successfully created a job for commit 21344be:

corochann commented 4 years ago

/test

corochann commented 4 years ago

please review & merge

codecov-io commented 4 years ago

Codecov Report

Merging #379 into master will decrease coverage by 0.37%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage    86.1%   85.72%   -0.38%     
==========================================
  Files         219      219              
  Lines       10671    10692      +21     
==========================================
- Hits         9188     9166      -22     
- Misses       1483     1526      +43
codecov-io commented 4 years ago

Codecov Report

Merging #379 into master will decrease coverage by 0.37%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage    86.1%   85.72%   -0.38%     
==========================================
  Files         219      219              
  Lines       10671    10692      +21     
==========================================
- Hits         9188     9166      -22     
- Misses       1483     1526      +43
codecov-io commented 4 years ago

Codecov Report

Merging #379 into master will decrease coverage by 0.37%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage    86.1%   85.72%   -0.38%     
==========================================
  Files         219      219              
  Lines       10671    10692      +21     
==========================================
- Hits         9188     9166      -22     
- Misses       1483     1526      +43
codecov-io commented 4 years ago

Codecov Report

Merging #379 into master will decrease coverage by 0.37%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage    86.1%   85.72%   -0.38%     
==========================================
  Files         219      219              
  Lines       10671    10692      +21     
==========================================
- Hits         9188     9166      -22     
- Misses       1483     1526      +43
codecov-io commented 4 years ago

Codecov Report

Merging #379 into master will decrease coverage by 0.37%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage    86.1%   85.72%   -0.38%     
==========================================
  Files         219      219              
  Lines       10671    10692      +21     
==========================================
- Hits         9188     9166      -22     
- Misses       1483     1526      +43
codecov-io commented 4 years ago

Codecov Report

Merging #379 into master will decrease coverage by 0.37%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage    86.1%   85.72%   -0.38%     
==========================================
  Files         219      219              
  Lines       10671    10692      +21     
==========================================
- Hits         9188     9166      -22     
- Misses       1483     1526      +43