Oliver M. Crook

Results 55 comments of Oliver M. Crook
trafficstars

Added a new key not sure what happened to the old one.

Thanks @lshep I made those changes, the most recent version should have the correct version update - with an updates NEWS file too. I didn't `supressMessages` on all the packages...

Hi, I'm looking forward to reviewing the package! Please fix the build errors before I can proceed.

Hi! How big the demo data - this maybe your limitation if it over 5mb you would have to store it in a data package. I favour the first option...

Thanks! Sorry I didnt see the notification. I'll get to this shortly

This is great! A few things to tidy up: - [ ] The News.md version doesnt match the version of the package, remember to bump the package version and the...

Hi @lauravongoc I hope to look at this week, thank you for removing the warnings. Best wishes, Olly

HI @lauravongoc My apologies for the delay - it's be a busy month. Please see the following review before we can accept the package: - [x] Bioconductor install instruction in...

Great, thank you! Does the sysdata.rda have to be in the R folder, could it be moved to the data or inst folders?

Hi @lauravongoc Sorry, it looks like I got confused which data was being referred where. As suggested, could you make it clear in the package what the data is and...