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

Unsure of value for first row of current derivation column of layer_info #8

Open
annakrystalli opened this issue Mar 30, 2020 · 3 comments
Assignees
Labels

Comments

@annakrystalli
Copy link

Hi,

I'm trying to fill from above the '' values in a couple of columns in the tables retuned via get_layer_info() but I'm having trouble identifying the correct value in a table in which the first value is ''. See below.

head(sdmpredictors::get_layers_info()$current$derivation)
#> [1] "''"                                              
#> [2] "mean, thin-plate smoothing splines"              
#> [3] "mean range, thin-plate smoothing splines"        
#> [4] "''"                                              
#> [5] "standard deviation, thin-plate smoothing splines"
#> [6] "maximum, thin-plate smoothing splines"

Created on 2020-03-30 by the reprex package (v0.3.0)

Any ideas?

@samuelbosch
Copy link
Collaborator

Hi Anna,

Not sure why it is showing up as "''", it should be NA or empty as it is defined in the "original" file which can be found in data-raw/layers.csv. Sometimes the derivation is not provided, or the layer is itself the result of a calculation of two "derived" layers.

@annakrystalli
Copy link
Author

Thanks for the swift response @samuelbosch !

OK so I can safely ignore it as NA then right? I am interpreting " properly in the rest of the rows though right? ie fill from above?

@bart-v bart-v added the question label Oct 5, 2021
@salvafern
Copy link
Contributor

Hi, sorry for the late reply. In hand writing, a list with '' may indicate that the value is the same than above, but it doesn't seem to be the case here, and we are not writing by hand here either :)

These '' are given already here: https://github.com/lifewatch/sdmpredictors/blob/master/data-raw/layers.csv, so we rather remove these and just let it be NA.

I'll do this whenever I find some time.

Thanks for spotting @annakrystalli !

@salvafern salvafern self-assigned this Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants