Function to convert DataCite metadata to EML: good fit for this package? about eml HOT 7 OPEN

The suggested datacite_to_eml() function is now documented at https://inbo.github.io/movepub/reference/datacite_to_eml.html. I think the EML package would be a better home for it.

from eml.

As another convenience method to template an EML record from existing metadata, this seems useful to me and I would support its inclusion. @cboettig woiuld you have any objections? If not, maybe @peterdesmet could submit a PR?

from eml.

👍 yeah seems like this would be helpful! PRs welcome!

from eml.

Cool, I'll see when I have some time for that. The function relies quite a lot on the purrr package. Is it fine if this is added as a dependency?

from eml.

For the packages I maintain, I try to keep dependencies to a minimum, especially for large, packages or packages that entrain a complex ecosystem, as they usually cause maintenance headaches down the road. We spend a fair number of cycles just treading water on package dependencies trying to keep packages on CRAN. Backwards incompatible changes or a package being supplanted by a "newer" version (as is common for RStudio packages) has caused a lot of churn for us. That said, if you really need it, then that is what they are there for. But keep in mind that each dependency is a potential future upgrade problem.

from eml.

I share your sentiment. I'll see if I can replace my three uses of purrr::map_chr() and two uses of purrr::map() with a base R alternative, if it remains readable (code at https://github.com/inbo/movepub/blob/main/R/datacite_to_eml.R#L26-L49).

from eml.

map is essentially lapply, and map_chr is essentially vapply with a template type. e.g.

  keywords <- purrr::map_chr(metadata$subjects, "subject")
## is the same as
  keywords <- vapply(metadata$subjects, `[[`, character(1L), "subject")

(yes, [[ is a the familiar sub-setting function, recall in R everything is a function). (Not tested)

That said, purrr is a light dependency compared to some things EML already pulls in....

from eml.