Skip to content
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

Road Map #1

Open
ibayer opened this issue Jun 28, 2017 · 4 comments
Open

Road Map #1

ibayer opened this issue Jun 28, 2017 · 4 comments

Comments

@ibayer
Copy link

ibayer commented Jun 28, 2017

What a are the main challenges (please edit)?

  • calling fastFM lib
  • designing the rfastFM interface
  • documentation

Does it make sense to start with fastFM-core (C) now and switch to the C++ version later?
The C++ interface if much nicer but it will take me > 3 weeks to give to a limited dev version.

@adrtod
Copy link
Owner

adrtod commented Jun 28, 2017

The main challenge is certainly the design.
Calling C/C++ code is not a problem in R. See the excellent book from H. Whickham for an introduction: http://r-pkgs.had.co.nz/src.html
I think of adapting the python interface to R idioms.
The documentation can also be adapted from the python one using roxygen.

I think it's ok to start interfacing with the C code and switch later to the C++. Meanwhile we can progress on the design of the rfastFM interface, which should not change much when we make the switch.

@adrtod
Copy link
Owner

adrtod commented Jun 28, 2017

For the documentation website, we can use the excellent pkgdown package. And use R markdown vignettes for Tutorials and Guide.

@ibayer
Copy link
Author

ibayer commented Jun 28, 2017

Sounds good to me.
Just ping me I you need something. I'll concentrate on the c++ rewrite for now.

@adrtod
Copy link
Owner

adrtod commented Jun 28, 2017

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants