On 2011-09-22 20:33, Christian Hudon wrote: > On 09/21/11 17:17, Laurent Gautier wrote: >> R itself _does_ create a temp directory each time it starts. > You are indeed correct. I had missed that, because I hadn't looked while > R was running. > >> Rpy2 has placeholders for custom callback cleaners, and it might >> interfere with the default cleanup made by an R console; this is >> currently a little-used (and little-documented) features and I can't >> tell with looking more into it. You could quickly fix it by writing a >> custom cleaner that does delete that directory. > Actually, the problem was simply that rinterface.endr() wasn't called. > There were comments about that in the code of robjects.__init__.py. I've > attached a patch to the issue tracker (since there was already an issue > open about this). > > https://bitbucket.org/lgautier/rpy2/issue/68/rpy2-does-not-remove-rtmp-folders#comment-666148 > > With this patch, rpy2 does not leave Rtmp temporary folders around anymore.
Yes. Good catch. Setting that up right during the initialization of R in the C-level rpy2.rinterface might be a more robust fix (say, using Py_AtExit() ). >> PS: The current release for rpy2 is 2.2.2. You do want to upgrade. > Actually, I've done all my testing already, and the Rtmp issue was the > last one before deployment into production. The use I make of rpy2 is > also quite simple (no passing back and forth of complex objects between > R and Python, etc.) and everything works fine, so I'll pass on upgrading > to rpy2 2.2 just now. Your call. If I were you, I would: chunks of the C-level interface have been rewritten, leading to potential segfault-causing or memory-leaking situations prevented and better performances (speed). User-visible changes are listed here: http://rpy.sourceforge.net/rpy2/doc-2.3/html/changes.html#release-2-2-0 > I'd also rather wait at least for the patch for > this bug to be integrated. You can also add: import rpy2.rinterface, atexit; atexit.register(rpy2.rinterface.endr, 0) at the top of your module using rpy2. Best, Laurent > Hoping some of the other bugs I submitted > will get closed before I update... At least this one is simple (and with > a patch): > https://bitbucket.org/lgautier/rpy2/issue/81/rpy2-ignores-settings-in-etc-r, > although you may disagree with my call. > > Thanks! > > Christian > ------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity and more. Splunk takes this data and makes sense of it. Business sense. IT sense. Common sense. http://p.sf.net/sfu/splunk-d2dcopy1 _______________________________________________ rpy-list mailing list rpy-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/rpy-list
