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

Error for ingesting soil_wise data #49

Open
yunkepeng opened this issue Jan 1, 2023 · 0 comments
Open

Error for ingesting soil_wise data #49

yunkepeng opened this issue Jan 1, 2023 · 0 comments

Comments

@yunkepeng
Copy link

yunkepeng commented Jan 1, 2023

Hi,

I just found in this line of code:

https://github.com/computationales/ingestr/blob/master/R/ingest_wise_byvar.R#L42

rename(ID = wise30sec_fin) should be changed to rename(ID = MU_GLOBAL)

When I use ingest(source="wise"..) to ingest soil wise data - it leads to bug - then when trying to solve it, I found here the variable was wrongly renamed. Because in this line the function aims to interpolate each site's unique number, and its variable name should be MU_GLOBAL (not wise30sec_fin). By correcting the code like above, the whole function works successfully.

Yunke

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

No branches or pull requests

1 participant