reagro / carob

Aggregating agronomic data
GNU General Public License v3.0
18 stars 28 forks source link

New dataset (hdl:11529/10547963) #500

Closed cedricngakou closed 2 weeks ago

cedricngakou commented 2 weeks ago

@rhijmans this script take some time to run over. It is due to the number of files in the raw dataset . is it possible to subset it into multiple files ( 3 or 4) to reduce the running time script?

rhijmans commented 2 weeks ago

It is not allowed to split a dataset. Also, I do not think it is a problem that it takes long to run.

rhijmans commented 2 weeks ago

I get:

 required records variable name(s) missing:  yield

Also, glyphosate is a herbicide (often sold as RoundUp)

This is wrong: fertilizer_price=as.character(r4$Total.fertiliser.cost.Tk.ha) (a price is not the same as the total cost)

Can you please fix the above?

I added a lon/lat that seemed obvious (you had NA) please check.

rhijmans commented 2 weeks ago

I see that there is dmy_storage (so you do not need to specify yield). We need to refine checking of this.