You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For quite some time there is a hidden feature that allows caching of API responses from the STATcube REST API. This is very useful for our internal web application and we will have to decide how we deal with this in the upcoming CRAN release.
Leaving it as a hidden feature might create a bad impression during reviews. Removing it would make it necessary to implement the caching logic elsewhere, which might be tricky. Therefore, it is probably best to document and export the behavior. Documentation is already available in ?sc_cache.
One problem with the current implementation is that the hashes are created via serialize() and therefore they are not reusable in different R versions. It would be very handy to use something like digest::digest() but adding another dependency package just for the hashes seems unnecessary. Maybe tools::md5sum() could be used in a different way to get a satisfying result.
TODOs
review reference documentation
think about creation of hashes
Add @export to make the caching available without environment variables
Remove @internal to include the man pages in the index page of the documentation
Maybe add something similar to od_cache_summary() which provides an overview about the cache contents. This would probably require some kind of cache_index.csv so we don't need to parse the cache entries.
The text was updated successfully, but these errors were encountered:
For quite some time there is a hidden feature that allows caching of API responses from the STATcube REST API. This is very useful for our internal web application and we will have to decide how we deal with this in the upcoming CRAN release.
Leaving it as a hidden feature might create a bad impression during reviews. Removing it would make it necessary to implement the caching logic elsewhere, which might be tricky. Therefore, it is probably best to document and export the behavior. Documentation is already available in
?sc_cache
.One problem with the current implementation is that the hashes are created via
serialize()
and therefore they are not reusable in different R versions. It would be very handy to use something likedigest::digest()
but adding another dependency package just for the hashes seems unnecessary. Maybetools::md5sum()
could be used in a different way to get a satisfying result.TODOs
@export
to make the caching available without environment variables@internal
to include the man pages in the index page of the documentationod_cache_summary()
which provides an overview about the cache contents. This would probably require some kind ofcache_index.csv
so we don't need to parse the cache entries.The text was updated successfully, but these errors were encountered: