Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

data.land examples fail #1977

Closed
infotroph opened this issue Jun 16, 2018 · 1 comment · Fixed by #2044
Closed

data.land examples fail #1977

infotroph opened this issue Jun 16, 2018 · 1 comment · Fixed by #2044

Comments

@infotroph
Copy link
Member

Describe the bug
As mentioned in #1949: package check fails while trying to run examples. Putting it here to have an issue number to reference.

To Reproduce

$ make .check/modules/data.land
Rscript scripts/check_with_errors.R modules/data.land
log_level = NA
die_level = NA
Updating PEcAn.data.land documentation
Loading PEcAn.data.land
Loading required package: PEcAn.DB
Loading required package: PEcAn.utils

Attaching package: ‘PEcAn.utils’

The following object is masked from ‘package:utils’:

    download.file

Loading required package: dbplyr
Warning: @examples [DataONE_doi_download.R#12]: requires a value
Warning: @examples [DataONE_doi_download.R#31]: requires a value
Warning: @examples [DataONE_doi_download.R#62]: requires a value
Warning: @examples [DataONE_doi_download.R#96]: requires a value
Warning: @return [extract_soil_nc.R#9]: requires a value
Warning: @return [extract_soil_nc.R#109]: requires a value
Warning: @examples [match_species_id_OLD.R#26]: requires a value
Warning: @examples [soil_process.R#12]: requires a value
Warning: @return [soil_utils.R#228]: requires a value
Warning: @return [soil_utils.R#304]: requires a value
Warning: @examples [soil_utils.R#307]: requires a value
checking examples ... ERROR
Running examples in ‘PEcAn.data.land-Ex.R’ failed
The error most likely occurred in:

> base::assign(".ptime", proc.time(), pos = "CheckExEnv")
> ### Name: match_species_id
> ### Title: Match BETY species ID.
> ### Aliases: match_species_id match_species_id
> 
> ### ** Examples
> 
> bety <- dplyr::src_postgres(dbname = 'bety',
+                        user = 'bety',
+                        password = 'bety',
+                        host = 'localhost')
> input_codes <- c('ACRU', 'PIMA', 'TSCA')
> format_name <- 'usda'
> match_species_id(input_codes = input_codes,
+                  format_name = format_name,
+                  bety = bety)
Warning: package ‘dplyr’ was built under R version 3.4.4

Attaching package: ‘dplyr’

The following objects are masked from ‘package:dbplyr’:

    ident, sql

The following objects are masked from ‘package:stats’:

    filter, lag

The following objects are masked from ‘package:base’:

    intersect, setdiff, setequal, union

Loading required package: R.oo
Loading required package: R.methodsS3
R.methodsS3 v1.7.1 (2016-02-15) successfully loaded. See ?R.methodsS3 for help.
R.oo v1.21.0 (2016-10-30) successfully loaded. See ?R.oo for help.

Attaching package: ‘R.oo’

The following objects are masked from ‘package:methods’:

    getClasses, getMethods

The following objects are masked from ‘package:base’:

    attach, detach, gc, load, save

R.utils v2.6.0 (2017-11-04) successfully loaded. See ?R.utils for help.

Attaching package: ‘R.utils’

The following object is masked from ‘package:utils’:

    timestamp

The following objects are masked from ‘package:base’:

    cat, commandArgs, getOption, inherits, isOpen, parse, warnings

Error in tbl_vars(y) : object 'obs' not found
Calls: match_species_id ... semi_join -> semi_join.tbl_df -> check_valid_names -> tbl_vars
Execution halted 
Error: 1 errors found in modules/data.land. See above for details
Execution halted
make: *** [.check/modules/data.land] Error 1
@infotroph
Copy link
Member Author

Error above was resolved at some point, but checking examples still fails because of #2043. Not closing this until that's resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant