quanteda / readtext

an R package for reading text files
https://readtext.quanteda.io
119 stars 28 forks source link

Travis fix #136

Closed amatsuo closed 6 years ago

amatsuo commented 6 years ago

To fix #134

It took quite a lot of trials and errors but the solution I ended up with is to update the version of gcc, g++ and gfortran to 5.

For some reason, RSpectra (or maybe RcppEigen) can't work with gcc-6 (Build 409, the build terminated because compilation of a package didn't finish within 4MB log-file limit), and I decided to update it to gcc-5.

amatsuo commented 6 years ago

The issue look like this:

So, in the end, gcc/g++/gfortran-5 is the sweetspot (probably I could also try 4.9. ). For quanteda, everything works with the default 4.8..

codecov-io commented 6 years ago

Codecov Report

Merging #136 into master will decrease coverage by 5.27%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   85.81%   80.53%   -5.28%     
==========================================
  Files           8        8              
  Lines         585      524      -61     
==========================================
- Hits          502      422      -80     
- Misses         83      102      +19
Impacted Files Coverage Δ
R/encoding.R 58.49% <0%> (-9.37%) :arrow_down:
R/nexis.R 91.37% <0%> (-8.63%) :arrow_down:
R/readtext.R 79.77% <0%> (-5.08%) :arrow_down:
R/get-functions.R 68.5% <0%> (-4.16%) :arrow_down:
R/utils.R 89.18% <0%> (-3.92%) :arrow_down:
R/readtext_options.R 97.36% <0%> (-2.64%) :arrow_down:

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 b32818f...573560e. Read the comment docs.