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

Modularize R API wrapper functions #90

Open
3 tasks
bmeluch opened this issue Nov 13, 2024 · 0 comments
Open
3 tasks

Modularize R API wrapper functions #90

bmeluch opened this issue Nov 13, 2024 · 0 comments

Comments

@bmeluch
Copy link
Contributor

bmeluch commented Nov 13, 2024

Currently, the taxonomy and NOM R notebooks define functions for interacting with the NMDC API. The functions are essentially the same. (e.g. cell 2 here)

Rather than copy pasting the same code into every new notebook, it would be better to pull those functions out into an R package accessible by all notebooks in this repo.

Completion criteria:

  • An R package exists in this repo containing the API interaction functions we've written so far
  • The existing notebooks use functions from the package and no longer define them internally
  • It plays nicely with our renv .lock file (someone newly cloning the repo can run the notebooks after using renv::restore() )
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

When branches are created from issues, their pull requests are automatically linked.

1 participant