csgillespie / benchmarkme

Crowd sourced benchmarking
https://csgillespie.github.io/benchmarkme/
40 stars 13 forks source link

fancier print.bytes #9

Closed richierocks closed 7 years ago

richierocks commented 7 years ago

with metric and IEC 80000 options

codecov-io commented 7 years ago

Codecov Report

Merging #9 into master will increase coverage by 0.55%.

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
+ Coverage   54.88%   55.43%   +0.55%     
==========================================
  Files          21       21              
  Lines         563      570       +7     
==========================================
+ Hits          309      316       +7     
  Misses        254      254
Impacted Files Coverage Δ
R/get_ram.R 100% <100%> (ø) :white_check_mark:

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 1cb8c45...016386e. Read the comment docs.

csgillespie commented 7 years ago

Thanks

csgillespie commented 7 years ago

@richierocks I don't suppose you could submit a PR to properly export and document this function? I just can't get the correct roxygen2 incantation to work.

richierocks commented 7 years ago

I haven't really got time to look at this, but can you just use

' @rdname get_ram

' @export

??

On 4 February 2017 at 14:14, Colin Gillespie notifications@github.com wrote:

@richierocks https://github.com/richierocks I don't suppose you could submit a PR to properly export and document this function? I just can't get the correct roxygen2 incantation to work.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/csgillespie/benchmarkme/pull/9#issuecomment-277467958, or mute the thread https://github.com/notifications/unsubscribe-auth/AAMD1WhDdMtcZuAeDvs7pgFqXD4bV9yzks5rZM4ugaJpZM4L2jDe .

-- Regards, Richie

Learning R http://shop.oreilly.com/product/0636920028352.do 4dpiecharts.com