-
Notifications
You must be signed in to change notification settings - Fork 13
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
R-project integration (provides client to connect to remote Rserve server) #31
base: master
Are you sure you want to change the base?
Conversation
…ly doesn't actually work at the moment.
…ist (not needed?). Fixup spacing.
* Use R to generate temporary file names on R server (using tempfile() function). * Don't "use" Rserve::Connection; instead preload modules into Safe compartment using configuration in defaults.config. * Move creation of image file into Rserve module (function evalStringToFile()).
Updating my branch
Changes for integration with R statistics package (via Rserve)
Hi Djun, This looks great! I tried this and ran into two small issues. After adding Statistics::RserveClient to
I think anything that is used in a pg module has to also be listed in
Starting with perl 5.16 defined(@array) is deprecated for testing if an array has elements. Great work! Jason |
Hi Jason,
Thanks for finding the issue with defined(). I was pleasantly surprised to find that at least one other person has downloaded the CPAN and used it, filed an issue and submitted a bug fix for this! I'll push his changes to CPAN, hopefully later today. I'll put together a few additional examples and post them somewhere for people to try out. Thanks again! Djun On 2013-02-08, at 11:44 AM, Jason Aubrey wrote:
|
Hi Djun, Do you want me to pull this request? or wait until you make changes? |
I'm really curious how this might be related to @aubreyja's work on the sage cell server plugin (which I'd love to start using :). |
Hi Jason (et al.)
On 2013-02-19, at 11:25 AM, Jason Grout wrote:
|
This introduces a new macro library (macros/RserveClient.pl) and makes a few minor changes to Translator.pm (whitespace, spelling)