dpc10ster / RJafroc

Artificial Intelligence: Evaluating AI, optimizing AI
19 stars 8 forks source link

Use readxl package for loading Excel files #55

Closed pwep closed 4 years ago

pwep commented 4 years ago

This is the first step in addressing #54, to be merged into your developer branch.

readxl seems lightweight and quick to set up to replicate structures returned from openxlsx's read.xlsx()

Writing Excel files is the next step, and will follow in a separate pull request tomorrow.

codecov[bot] commented 4 years ago

Codecov Report

Merging #55 into developer will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           developer      #55   +/-   ##
==========================================
  Coverage      69.86%   69.86%           
==========================================
  Files             79       79           
  Lines           9815     9815           
==========================================
  Hits            6857     6857           
  Misses          2958     2958           
Impacted Files Coverage Δ
R/DfReadCrossedModalities.R 70.00% <100.00%> (ø)
R/ReadJAFROCNewFormat.R 76.92% <100.00%> (ø)
R/ReadJAFROCOldFormat.R 77.77% <100.00%> (ø)

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 6a3c7a2...5ce3414. Read the comment docs.