rticles icon indicating copy to clipboard operation
rticles copied to clipboard

GigaScience rticle support

Open eschen42 opened this issue 6 years ago • 23 comments

Works with the skeleton, albeit with a few workarounds:

  • the file tlmgr_pkg adds a few Tex*Live packages needed to generate PDFs from this and other rticles.
    1. The gigascience test passes
    2. Some other rticle tests do not pass because of missing fonts; I don't know how to resolve these.
  • knitr produces a PDF with broken citation links and numbers.
  • the work around is:
    1. Open the .tex file in RStudio (it has the same file-name as the .Rmd file)
    2. Click "Compile PDF" three times. (The first two times fail because of issues with how the bibliography is generated.)

Tested on rserver in my eschen42/devplan Docker environment, doi: 10.5281/zenodo.1157078

eschen42 avatar Feb 20 '18 15:02 eschen42

Okay, I'm working on it. FYI, when I tried to install rtciles from a git clone it failed because of unmet dependency xfun - fortunately, I thought to search for it on GitHub and installed rticles. Now the fun really begins ....

eschen42 avatar Jul 20 '18 18:07 eschen42

xfun is a CRAN package. Either devtools::install_github('rstudio/rticles') or install.packages('rticles') should install it automatically, unless you are using a very old version of R.

yihui avatar Jul 20 '18 18:07 yihui

Good to know. Probably I am using an old version of R! I will retry sometime with a newer version. With my version of RStudio, I didn't find xfun when I tried "Tools > Install Packages..." in RStudio.

FYI, and no need to respond since it sounds expected: In my archaic corner of the universe, devtools::install_github('rstudio/rticles') did not install xfun.

RStudio 1.1.383 R version 3.4.1 (2017-06-30) -- "Single Candle" Copyright (C) 2017 The R Foundation for Statistical Computing Platform: x86_64-pc-linux-gnu (64-bit)

eschen42 avatar Jul 20 '18 18:07 eschen42

I don't know how to get 'testit' to produce results that I can see and review.

testit::test_pkg("rticles")

produces no output (instantly) and gives no evidence that it has run.

The move away from 'testthat' makes devtools::test() fail. Since testthat is the only way that I know how to test a package before installing it, I did the following workaround (without adding it to git) to make sure that my article's test passes:

mkdir tests/testthat
cd tests/testthat
ln -s ../testit/test-formats.R'

in bash and then

library(testit)
devtools::test()

in R

eschen42 avatar Jul 23 '18 01:07 eschen42

Oh BTW, did you sign the contributor agreement? https://github.com/rstudio/rticles/blob/master/.github/PULL_REQUEST_TEMPLATE.md

yihui avatar Jul 23 '18 03:07 yihui

Contributor agreement received.

jjallaire avatar Jul 25 '18 11:07 jjallaire

Dear @yihui ,

Please understand that I do not wish to bite the hand that feeds me. I very much appreciate your hard work on rmarkdown and how it makes life so much better for so many people.

Right now, I'm trying to put together a GigaScience paper in TeX/LaTeX, and they have a two-column format and a specific document class. Of course, I'm new to rmarkdown and trying to get things like figures and bibliographies to work with their template. I just want to get a draft done!

I see that I have expressed opinions strongly, even in my skeleton.Rmd! :( These really are ill-informed impressions - I see that better now. So, I appreciate your coaching and patience very much.

I do want the skeleton.Rmd to give authors as much help as possible to complete their GigaScience paper quickly. So, I want it to provide best practices, hints, and alternatives.

I hope that I have not been too abrasive.

Thank you again for your hard work!

eschen42 avatar Jul 25 '18 18:07 eschen42

@eschen42 Thanks for your explanation! I can see your position better now. My main point is that if a certain thing can be achieve through either LaTeX or Markdown, I tend to use Markdown instead. However, I'm not completely against using raw LaTeX (and sometimes it is just not avoidable). It will be nice if we can point out both ways to users and let them decide. If you are not familiar with the Markdown way yet, it is totally fine to only introduce the LaTeX way for now.

This PR really only has a small number of minor issues. I guess we can merge it after another revision. Thanks again!

yihui avatar Jul 25 '18 20:07 yihui

@eschen42 Would you be open to pick this issue up again, or would you mind me attempting to fix the remaining issues?

nuest avatar Apr 15 '20 08:04 nuest

@eschen42 friendly ping 😄

what is the status of this PR ? Do you plan on working on it or can @nuest work on it ?

cderv avatar Aug 17 '20 16:08 cderv

@eschen42 friendly ping 😄

what is the status of this PR ? Do you plan on working on it or can @nuest work on it ?

@cderv Thank you so much for the ping! @nuest Please feel free to resolve the issues in ANY way that you see fit, if you are still inclined to do so. I am VERY sorry that I didn't notice your request. I found myself somewhat out of my depth regarding following up on the PR.

eschen42 avatar Aug 21 '20 15:08 eschen42

Would you be open to pick this issue up again, or would you mind me attempting to fix the remaining issues?

@nuest if you are still willing to help on this one, feel free to chime in. Thanks !

cderv avatar Aug 24 '20 16:08 cderv

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: cderv
:x: eschen42
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Sep 24 '20 18:09 CLAassistant

Hi @eschen42 @nuest,

is anyone of you willing to continue this old PR ? It has been a long time and there may have been some updates on the GigaScience underlying template.

Just seeing what is the status on this one. Thanks

cderv avatar Jun 09 '21 13:06 cderv

Another friendly ping on this template @nuest. Are you interesting to keep the work on this ?

I leave this PR open - I added the label help wanted in case anyone want to jump it to help on this.

cderv avatar Sep 07 '21 13:09 cderv

Help is welcome, I for my part have put this on my list for when I return from my extended leave in Spring next year.

nuest avatar Sep 07 '21 17:09 nuest

Hi @eschen42 @nuest,

Just for information I merge master branch into this PR so that it can be continued on good basis as we have changed some ways to organize the code. No change on the format base code.

Is this still a project that one of you is planning to help on ?

cderv avatar Nov 30 '21 10:11 cderv

No status change, it's one the list if side projects for spring.

nuest avatar Nov 30 '21 12:11 nuest

Hi @eschen42 @nuest,

should be plan to finish this PR for a gigascience template ? Just enquiring for a status update

cderv avatar Apr 19 '23 11:04 cderv

@cderv @nuest I am very sorry that I have not gotten back to this in like forever.... I will see what I can do today. Part of the issue has been that yihui requested "rebase" which I have no idea how to do and barely know what it is...

eschen42 avatar Apr 28 '23 17:04 eschen42

@cderv Even when I do renv::activate(), install required packages, and do devtools::install, I still get:

Error: 'gigascience_article' is not an exported object from 'namespace:rticles'

Whereas the other article formats work fine. gigascience_article is in my NAMESPACE file. Ideas?

eschen42 avatar Apr 28 '23 19:04 eschen42

Sorry for the delay; Let me try to rebase or at least merge main in this PR.

Regarding renv we don't use that in the package.... if you use that in your project, you need to be sure to install the PR branch to get the rticle function

cderv avatar May 04 '23 17:05 cderv

@eschen42 I spend some time today to update the PR to our latest best practice in the repo, and also dig into GIGASCIENCE doc.

I found this : https://academic.oup.com/pages/authoring/journals/preparing_your_manuscript

The OUP LaTeX template produces manuscripts matching the formatting requirements of the journals listed at the following link: Journals supported by the OUP LaTeX template.

Gigascience is found among the journal list in the Excel file provided image

It happens that we have now an updated version of the OUP template using https://ctan.org/pkg/oup-authoring-template You can find how it looks here: https://pkgs.rstudio.com/rticles/articles/examples.html#oup_v1---oxford-university-press-oup---new-2020-ctan-template-v1-0

This means that using something like

output:
  rticles::oup_article:
    oup_version: 1 # 1 = 2020 CTAN OUP CLS package 
    journal: "GigaScience"
    document_style: "contemporary" 
    papersize: "large" 
    namedate: FALSE
    number_sections: FALSE 

should give a template suitable for GigaScience if they really follow OUP template

We could also adapt the OUP template if it is not working well. What do you think ?

cderv avatar May 05 '23 19:05 cderv