search

NelleV / moanin

Timecourse transcriptomic analysis
https://nellev.github.io/moanin/
Other
5 stars 0 forks source link

Need a print function for moanin object #30

Closed epurdom closed 4 years ago

epurdom commented 4 years ago

It would be good to create a print.moanin_model function, so there is a concise summary printed, rather than the whole list. It would be nice for the workflow too.

epurdom commented 4 years ago

I think I fixed the problem with the print function creating problem. A method has to have at minimum the arguments of the generic (including the …), though it can have more. So I have just done a pull request fixing that and addressing some issues I hit on R CMD CHECK.

The branch checks fine, but the merge with master doesn’t (even though I merged the most recent master in with my branch). The error isn’t an R error, but something to do with running scripts. @NelleV do you know what this problem is?

Writing create_moanin_model.Rd
Writing plot_splines_data.Rd
Rscript .scripts/install_package.R
Error: HTTP error 403.
  API rate limit exceeded for 34.73.34.132. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)
  Rate limit remaining: 0/60
  Rate limit reset at: 2020-07-03 00:46:42 UTC
  To increase your GitHub API rate limit
  - Use `usethis::browse_github_pat()` to create a Personal Access Token.
  - Use `usethis::edit_r_environ()` and add the token as `GITHUB_PAT`.
Execution halted
make: *** [Makefile:10: install] Error 1
The command "docker run -ti -v $(pwd):/home/rstudio/repo -w /home/rstudio/repo nellev/rbio /bin/bash -e ".build_tools/travis/test_script.sh"" exited with 2.