Skip to content

Conversation

@sckott
Copy link
Collaborator

@sckott sckott commented Sep 22, 2020

  • each fxn gains a new parameter retry_opts
  • the retry_opts parameter accepts a call to either the bs_retry_options or entrez_retry_options function from each respective package
  • the retry options is a named list and gets passed to either rentrez or rbace to configure retry http settings
  • defaults are used for retry options, but can be set by the user on each get_papers function call

fix #465

each fxn gains a new parameter retry_opts
the retry_opts parameter accepts a call to the retry_options function from each respective package
the retry options is a named list and gets passed to either rentrez or rbace to configure retry http settings
defaults are used for retry options, but can be set by the user on each get_papers function call
@sckott
Copy link
Collaborator Author

sckott commented Sep 22, 2020

  • rentrez: remotes::install_github("sckott/rentrez@http-post")

@sckott
Copy link
Collaborator Author

sckott commented Sep 22, 2020

  • rbace: remotes::install_github("ropensci/rbace") - avoid the CRAN version for now

@chreman chreman merged commit 38051a8 into master Sep 27, 2020
chreman added a commit to chreman/Headstart that referenced this pull request Oct 13, 2021
…t-retries

changes to base and pubmed get_papers methods to use retry

Former-commit-id: 38051a8
@chreman chreman deleted the sckott-retries branch October 14, 2021 12:30
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

Successfully merging this pull request may close these issues.

RETRY - where in the code do we want to toggle retry options?

3 participants