search

Gitmaxwell / lastools

‘lastools’ is an R package for reading and writing version 1.2 and version 2.0 Log ASCII Standard (LAS) and for performing common functions on LAS
Other
12 stars 2 forks source link

Fix error in package check output: Remove get_all.. alias for read_las_mnemonic() #2

Closed dcslagel closed 4 years ago

dcslagel commented 4 years ago

@Gitmaxwell

This change fixes an error in reported when running R CMD check lastools_1.0.0.tar.gz on a build lastool package or when running devtools::check() within R for lastools.

Error Message:

 >checking examples ... ERROR
  Running examples in ‘lastools-Ex.R’ failed
  The error most likely occurred in:

  > base::assign(".ptime", proc.time(), pos = "CheckExEnv")
  > ### Name: read_las_mnemonics_df
  > ### Title: Read las curve mnemonics data to data frame
  > ### Aliases: read_las_mnemonics_df
  > 
  > ### ** Examples
  > 
  > get_all_las_mnemonics(dir)
  Error in get_all_las_mnemonics(dir) : 
    could not find function "get_all_las_mnemonics"
  Execution halted

The change replaces the get_all_las_mnemonics(dir) alias for the actual name read_las_mnemonics(".") in the @examples tag. Note, a real directory (".") is used in the example to solve a subsequent error report about 'dir' not being a directory.

Let me know if this change could be accepted (or rejected) or needs some additional changes before merging.

Thanks,

DC