RcppCore / rcpp-gallery

Source code for the Rcpp Gallery website
http://gallery.rcpp.org
Other
69 stars 71 forks source link

Update R package requirements in "Local Development Configuration" wiki #121

Open ArtemSokolov opened 4 years ago

ArtemSokolov commented 4 years ago

After performing a local build, I noticed that the following R packages were required, but not specified on the "Local Development Configuration" wiki:

Furthermore, the benchmark package listed in the dependencies seems to be archived on CRAN.

Lastly, I think it might be nice to give wiki readers an install.packages() line they can copy to their R environment, similar to what's provided with the sudo apt-get install ... lines. Consider including something like

install.packages( c("Rcpp", "RcppEigen", ...) )
devtools::install_github( "RcppCore/RcppNT2" )
devtools::install_github( "dcdillon/RcppHoney" )
eddelbuettel commented 4 years ago

Yes. There is potentially a lot more on a per-post basis, depending on whether you want to recreate the posts. (I have been noodling with a pdf variant ...)

The benchmark is a typo. I meant rbenchmark.

I guess we could clean this up but I am not sure how high a priority it is. Thoughts, @jjallaire ?

jjallaire commented 4 years ago

Not something I have time for right now but definitely worthwhile!

On Fri, Jan 3, 2020 at 11:18 AM Dirk Eddelbuettel notifications@github.com wrote:

Yes. There is potentially a lot more on a per-post basis, depending on whether you want to recreate the posts. (I have been noodling with a pdf variant ...)

The benchmark is a typo. I meant rbenchmark https://cran.r-project.org/package=rbenchmark.

I guess we could clean this up but I am not sure how high a priority it is. Thoughts, @jjallaire https://github.com/jjallaire ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/RcppCore/rcpp-gallery/issues/121?email_source=notifications&email_token=AAAZPR5RWHHYP2N37KPNSETQ35QMJA5CNFSM4KCPFIAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIBPQVY#issuecomment-570619991, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAZPR6KSISACM3SAYDGQTDQ35QMJANCNFSM4KCPFIAA .

eddelbuettel commented 4 years ago

Maybe the simplest fix would be to ... update the wiki page?

@ArtemSokolov as you have fresh eyes for the topic would you want to take a stab at it?

ArtemSokolov commented 4 years ago

Honestly, other than the R package dependencies, I think the wiki page is in a pretty good shape! I'm happy to make the changes I proposed, but I don't think GitHub has an easy way to submit a pull request for wiki pages.

Let me know how you would like to proceed.

eddelbuettel commented 4 years ago

Yes, the wiki pages are often left behind. I think one ... just edits as on any other wiki but storage is in git so one could roll-up if the need arose.

ArtemSokolov commented 4 years ago

I think there's a permission control somewhere. Currently, there is no edit option visible to me in the Wiki tab.

eddelbuettel commented 4 years ago

Hm. Do you see the Clone this wiki locally on the right side when you are on a post?

ArtemSokolov commented 4 years ago

Yes. However, as discussed in this blog post, "while you can clone the wiki and make a change to your copy, you cannot submit a pull request as your copy is not “linked” with the original repository."

So, we just need to figure out an effective way to get any changes I make to appear in this primary fork.

eddelbuettel commented 4 years ago

Maybe make the changes and email me a diff? I don't have time to dig now if there is mode somewhere between wiki 'locked and open to owners' and 'open to rest of world'.