pommedeterresautee / fastrtext

R wrapper for fastText
https://pommedeterresautee.github.io/fastrtext/
Other
101 stars 15 forks source link

Can I mimic package structure to build starspace r wrapper? #16

Closed mrchypark closed 6 years ago

mrchypark commented 6 years ago

I'll try to make R package using starspace from facebook.

fastrtext package is very similar to my purpose. so if you don't mind, can i mimic your package structure make StarspaceR package?

Thanks in advance.

pommedeterresautee commented 6 years ago

I m currently doing it :-) Will release soon.

mrchypark commented 6 years ago

@pommedeterresautee OMG ok. i'll wait haha. I just want to study making package with cpp using Rcpp. Thanks again.

pommedeterresautee commented 6 years ago

Sincerly, I would not start by copying an existing package. Many tricks are already applied and you will learn very little about Rcpp and package maintenance. My advice would be to start a package from scratch (even a wrapper of an existing C++ project).

mrchypark commented 6 years ago

I agree. I don't understand something why you create. Thank you for your advise.

pommedeterresautee commented 6 years ago

You mean there is something you don't understand in the source code of this package? What is it?

mrchypark commented 6 years ago

of course. I don't know what s4 is. And I don't know cpp. I realized it's hard for me to try. :_(

pommedeterresautee commented 6 years ago

You will need to learn C++ without R first (really, there is no way otherwise). Regarding S4, it s a way to perform OOP on R, I use it because Rcpp modules generates them. I think I made an error to rely on it... (lots of code to manage their behavior.

mrchypark commented 6 years ago

Sounds great.... I'll try to study cpp first.... I knew it necessary....

OOP is also on list of study for me. thanks again.

anyway I'm looking forward to package starspace ;)