From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Ecay Subject: Re: R code block produces only partial output Date: Thu, 28 Aug 2014 01:24:40 -0400 Message-ID: <87lhq9tcdj.fsf@gmail.com> References: <87iom8zd24.fsf@gmail.com> <877g2oz9gv.fsf@gmail.com> <87lhr27oap.fsf@gmail.com> <87r40uwavs.fsf@gmail.com> <8761i5kg8f.fsf@gmail.com> <87ppgcrg8n.fsf@gmail.com> <87lhr0qimr.fsf@gmail.com> <87wqa9owhv.fsf@gmail.com> <87oavkp2xa.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55550) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XMsCb-0000lw-9x for emacs-orgmode@gnu.org; Thu, 28 Aug 2014 01:25:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XMsCR-0002Oi-Ce for emacs-orgmode@gnu.org; Thu, 28 Aug 2014 01:24:52 -0400 Received: from mail-qa0-x22a.google.com ([2607:f8b0:400d:c00::22a]:38593) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XMsCR-0002Oe-8e for emacs-orgmode@gnu.org; Thu, 28 Aug 2014 01:24:43 -0400 Received: by mail-qa0-f42.google.com with SMTP id i13so281630qae.1 for ; Wed, 27 Aug 2014 22:24:42 -0700 (PDT) In-Reply-To: List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: "Charles C. Berry" Cc: emacs-orgmode@gnu.org Hi Chuck, Thanks for your feedback. The patch you=E2=80=99ve sent looks basically co= rrect to me. 2014ko abuztuak 23an, "Charles C. Berry"-ek idatzi zuen: [...] > The old hacky way works pretty well most of the time. I suspect that it=20 > will be hard to get folks to really test a new ob-R.el in advance of its= =20 > moving to maint. ./lisp/test-ob-R.el only has one :session src block and= =20 > a few other src blocks... Agreed, more tests are something I want to add to this patch before landing it, and I=E2=80=99ll try to encourage that in the future (from myse= lf most of all!). >=20 > I was thinking of having a drop in replacement for the each of the two=20 > functions that are changed and maybe selecting which version to use with= =20 > defalias. If there is a slicker way to enable a user to revert to an=20 > earlier version without having good git skills, fine. But I think the=20 > possibility of breaking something that someone needs and not noticing til= l=20 > the changes go to maint is high. I think that having two different implementations will in the long run increase the surface area susceptible to bugs. We should do our best and then release what we have when we feel it=E2=80=99s stable. (I admit to being a bit bewildered by org=E2=80=99s release process; I just= use whatever is in git and deal with the occasional breakage. The big no-no is letting a bug slip into an emacs-bundled version of Org, since that will live for years. But it seems like it takes a long time even after reaching maint for code to make its way into emacs.) [...] >=20 > Not sure I get why the sentinel file is needed,=20 The execution of the R commands is asynchronous (from emacs=E2=80=99 POV); = org polls for the existence of this file to let it know that R has finished its execution. > but the patch here uses on.exit(file.create(...)) to ensure that that > file is created. One hiccup (not sure if it exists in master,too) is > that starting a remote session and then trying to run src blocks from > a buffer for a local file will hang (because a local temp file is used > for sentinel). So there is still stuff to do. Hmm, OK. [...] >=20 > Putting objects in userspace is considered poor practice. FWIW, CRAN=20 > disallows it. `environment-hacking' as you call it underlies much of R. I= f=20 > you must create and hold variables use the attach(NULL,name=3D"org-vars")= =20 > trick. I=E2=80=99ll defer to your judgment. Thanks again for the feedback and revised patch. I=E2=80=99ll try to get a= few tests written this weekend, and study the remote vs. local files business. --=20 Aaron Ecay