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

Teach catmaid_fetch to cache some return values #140

Open
jefferis opened this issue Aug 12, 2019 · 2 comments
Open

Teach catmaid_fetch to cache some return values #140

jefferis opened this issue Aug 12, 2019 · 2 comments

Comments

@jefferis
Copy link
Collaborator

Ideal arrangement would be that

  • you could choose which calls are cached
  • you could choose different cache time outs

(see also #20 for which this might be useful)

@jefferis
Copy link
Collaborator Author

This would be useful for e.g. catmaid_get_annotationlist which is starting to get expensive (>1.5s).

@jefferis
Copy link
Collaborator Author

It would be better to do this after merging the http::VERB to simplify this section

https://github.com/jefferis/rcatmaid/blob/bfb757f73561a12aec39da2be77b457d42a694ab/R/catmaid_connection.R#L364-L373

In particular it is essential that the httr::config is part of the memorisation arguments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant