oganm
commented
2 years ago This was a bit of an architectural problem which would have caused caching to fail for every function called from within another function. here i borrow the caching method from my old gemmaAPI.R where memoised functions are called directly from the main function body
As pointed out in the review here: https://github.com/Bioconductor/Contributions/issues/2533#issuecomment-1135413964 our high level functions (everything within convenience.R) don't seem to have memoisation implemented. Working on this at the high_level_memo branch but if this was intentional instead of an oversight please let me know.