From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rick Frankel Subject: Re: process diagrams with dot and some glue using org Date: Fri, 19 Jul 2013 13:37:31 -0400 Message-ID: References: <87vc46ea4y.fsf@ucl.ac.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:44069) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V0Ece-0006cR-CE for emacs-orgmode@gnu.org; Fri, 19 Jul 2013 13:37:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V0EcW-0006pE-1D for emacs-orgmode@gnu.org; Fri, 19 Jul 2013 13:37:40 -0400 Received: from [204.62.15.78] (port=55086 helo=mail.rickster.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V0EcV-0006oy-T1 for emacs-orgmode@gnu.org; Fri, 19 Jul 2013 13:37:31 -0400 In-Reply-To: <87vc46ea4y.fsf@ucl.ac.uk> 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: Eric S Fraga Cc: emacs-orgmode On 2013-07-19 13:23, Eric S Fraga wrote: > a couple of weeks ago you developed some code to convert a pair of > tables to a graphviz digraph and you wrote a very useful Worg page > about > it: > > http://article.gmane.org/gmane.emacs.orgmode/74280 > http://orgmode.org/worg/org-tutorials/org-dot-diagrams.html > > I just wanted to say two things about it: > > 1. if the typical use case is to use the code with #+call:, it would be > more helpful to delete the #+header: line in the source code in the > Worg page. This header line presumes that the particular tables in > your first example exist. Deleting this line means that the code > can only be used via a #+call statement but I think this is more > logical. I (sort of) disagree. I think specifying required arguments as header vars makes the calling requirements clearer. Perhaps: #+HEADER: :var nodes='() graph='() would be better... rick