search

BLE-LTER / BLEdatatools

R package to download and collate BLE LTER data from EDI repository
Other
0 stars 0 forks source link

Include option to exclude meta data #13

Open akshoelace opened 2 years ago

akshoelace commented 2 years ago

option in the download data function?

atn38 commented 2 years ago

We haven't even touched metadata yet! We should do that first, haha.

atn38 commented 2 years ago

Sasha @akshoelace : I'd love to see an argument or option in the function "download_data" that takes a TRUE or FALSE. If the user says TRUE, download_data also returns the metadata file along with the data file (preferably the metadata file is named appropriately). If FALSE, the function just returns data. Look into the function read_metadata in EDIutils https://github.com/ropensci/EDIutils/blob/main/R/read_metadata.R

akshoelace commented 2 years ago

read_metadata has been added to download_data function - default with include meta data, if FALSE it will exclude

atn38 commented 2 years ago

hey @akshoelace the work we did is actually still on your branch, so we need to merge it into development -- and we need to write metadata to file successfully too -- before we can close this