-
Notifications
You must be signed in to change notification settings - Fork 522
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Copernicus Publications template #172
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you rebase against the current master branch and resolve the conflicts? You will need force push.
Could you also remove the standard LaTeX packages in this PR? Users should install these packages by themselves. If you use TinyTeX, missing LaTeX packages can be automatically installed: https://yihui.name/tinytex/.
R/copernicus_article.R
Outdated
"-autolink_bare_uris", # disables automatic links, needed for plain email in \correspondence | ||
"-auto_identifiers" # disables \hypertarget commands | ||
)) { | ||
if (inherits(base_format, "character")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about is.character()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just copied this from peerj_article
, but happy to change to is.character()
.
R/copernicus_article.R
Outdated
#' names(copernicus_journal_abbreviations()) | ||
#' copernicus_journal_abbreviations(name = "Science Data") | ||
#' | ||
#' @export |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd merge the doc of this function with copernicus_article
, i.e.
#' @rdname copernicus_article
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
8023cbf
to
928d6cd
Compare
(squashed changes) - update to template package version 5.0 - appendix may not be loaded via external file, use yaml property instead - add hints for additional metadata - add default language for babel - add default language for babel - fix tightlist in pandoc - no HTML comments with LaTeX in it - update docs - clarify usage of quations, closes #1 - clarify docs for algorithm packages - add examples for usage of \unit{}
@yihui Thanks for taking a look, and my sincere apologies for the delay. I tried to incorporate your requirements. Could you please clarify what you mean by "Could you also remove the standard LaTeX packages in this PR?" |
Contributor agreement is on the way, too. |
I meant standard packages like To check if a package is on CTAN, you may use, e.g., > tinytex:::parse_packages(files = 'natbib.sty')
tlmgr search --file --global '/natbib.sty'
[1] "natbib" |
I understand. Removed > tinytex:::parse_packages(files = 'pdfscreencop.sty')
tlmgr search --file --global '/pdfscreencop.sty'
character(0)
Warning message:
In tinytex:::parse_packages(files = "pdfscreencop.sty") :
Failed to find a package that contains pdfscreencop.sty |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! One last thing: could you also add an entry about this journal in README.md? Thanks!
The entry in the README.md was there already: https://github.com/rstudio/rticles/pull/172/files#diff-04c6e90faac2675aa89e2176d2eec7d8R55 Or would you like to have something different? |
to not confuse Copernicus Publicatoins with "Copernicus Publishers" (https://flakyj.blogspot.com/2018/03/copernicus-publishers.html)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
Yay 🎆 🎉 ! |
I've written a blog article planned to be published in the EGU's GeoLog, the academic society behind Copernicus Publications, along with some social media activity. The draft is at Feedback of course welcome, but mostly just wanted to leave this here in case anyone stumbles across this PR. |
Looks great! Thanks for sharing the post! |
Please note that I just removed the output:
bookdown::pdf_book:
base_format: rticles::copernicus_article |
Copernicus now starts advertising the template in their manuscript preparation guidelines, see e.g. journal ACP at https://www.atmospheric-chemistry-and-physics.net/for_authors/manuscript_preparation.html |
@nuest Awesome! Thanks for letting us know! |
This PR add the template by Copernicus Publications, a small open access publisher in the geosciences, based on https://publications.copernicus.org/for_authors/manuscript_preparation.html