Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

use Rcpp vignette style#153

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
eddelbuettel merged 2 commits intomasterfromfeature/vignette-polish
Aug 1, 2017
Merged

Conversation

@eddelbuettel
Copy link
Member

@binxiangni I updated the vignette a little, see a(temp) copy of the pdf for a quick glance.

thirdwing and binxiangni reacted with thumbs up emoji
@binxiangni
Copy link
Contributor

Looks good. There might be an inconsistency due to me.Subsection 2.7.3 includeslibrary(Matrix), but others don't. Maybe we can just remove it. The other part is great. You can merge it. Thank you!

@eddelbuettel
Copy link
MemberAuthor

Maybe we can just remove it.

Done!

@eddelbuettel
Copy link
MemberAuthor

BTW, one downside is that you need a helper script such as the snippet below to compile the vignette.

#!/usr/bin/Rscript## use given argument(s) as target files, or else default to .Rnw files in directoryfiles<-if (length(commandArgs(TRUE))==0) dir(pattern="*.Rnw")else commandArgs(TRUE)## convert all files from Rnw to pdf using the highlight driverinvisible(sapply(files,function(srcfile) {    Sweave(srcfile,driver=highlight::HighlightWeaveLatex())tools::texi2pdf(gsub(".Rnw",".tex",srcfile))}))

@eddelbuetteleddelbuettel merged commit1779ced intomasterAug 1, 2017
@binxiangni
Copy link
Contributor

Not familiar with that. I just compile my original one in Rstudio. Thanks for reminding. Really learned a lot from you this summer.

@eddelbuettel
Copy link
MemberAuthor

eddelbuettel commentedAug 1, 2017
edited
Loading

It's the non-standard (but perfectly fine and permitted) vignette engine we use: based onhighlight. All pretty easy on Linux, and (I presume) possible on macOS...

@eddelbuetteleddelbuettel deleted the feature/vignette-polish branchAugust 1, 2017 03:24
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eddelbuettel@binxiangni

[8]ページ先頭

©2009-2025 Movatter.jp