From: "Eric Schulte" <schulte.eric@gmail.com>
To: Nicolas Goaziou <n.goaziou@gmail.com>
Cc: Org Mode List <emacs-orgmode@gnu.org>
Subject: Re: [babel][PATCH] org-babel-asymptote now handles variables.
Date: Wed, 02 Jun 2010 09:06:39 -0600 [thread overview]
Message-ID: <87typlzd40.fsf@gmail.com> (raw)
In-Reply-To: 87typpqk5p.wl%n.goaziou@gmail.com
Hi Nicolas,
This patch looks great! Thanks for the contribution. However before I
apply this patch, can I ask, have you signed the FSF papers.
As org-babel is being incorporated into the core of Org-mode and will be
considered part of Emacs we need authors of any contributions longer
than 10 lines to sign the FSF papers.
http://orgmode.org/worg/org-contribute.php
Thanks -- Eric
Nicolas Goaziou <n.goaziou@gmail.com> writes:
> Hello,
>
> Here is a patch to allow the use of variables in org-babel-asymptote.
> Types integer, string, symbol (treated as string) float and any table
> of those are recognized.
>
> Thanks,
>
> - Nicolas
>
> diff --git a/contrib/babel/lisp/langs/org-babel-asymptote.el b/contrib/babel/lisp/langs/org-babel-asymptote.el
> index 59ca291..eed1a87 100644
> --- a/contrib/babel/lisp/langs/org-babel-asymptote.el
> +++ b/contrib/babel/lisp/langs/org-babel-asymptote.el
> @@ -37,8 +37,6 @@
> ;; 3) we are adding the "file" and "cmdline" header arguments, if file
> ;; is omitted then the -V option is passed to the asy command for
> ;; interactive viewing
> -;;
> -;; 4) there are no variables (at least for now)
>
> ;;; Requirements:
>
> @@ -56,13 +54,17 @@
> (defvar org-babel-default-header-args:asymptote '((:results . "file") (:exports . "results"))
> "Default arguments to use when evaluating a asymptote source block.")
>
> -(defun org-babel-expand-body:asymptote (body params &optional processed-params) body)
> +(defun org-babel-expand-body:asymptote (body params &optional processed-params)
> + (let ((vars (second (or processed-params
> + (org-babel-process-params params)))))
> + (concat (mapconcat 'org-babel-asymptote-var-to-asymptote vars "\n") "\n" body "\n")))
>
> (defun org-babel-execute:asymptote (body params)
> "Execute a block of Asymptote code with org-babel. This function is
> called by `org-babel-execute-src-block'."
> (message "executing Asymptote source code block")
> - (let* ((result-params (split-string (or (cdr (assoc :results params)) "")))
> + (let* ((processed-params (org-babel-process-params params))
> + (result-params (split-string (or (cdr (assoc :results params)) "")))
> (out-file (cdr (assoc :file params)))
> (format (or (and out-file
> (string-match ".+\\.\\(.+\\)" out-file)
> @@ -75,12 +77,76 @@ called by `org-babel-execute-src-block'."
> (concat "-globalwrite -f " format " -o " out-file)
> "-V")
> " " cmdline " " in-file)))
> - (with-temp-file in-file (insert body))
> + (with-temp-file in-file (insert (org-babel-expand-body:asymptote body params processed-params)))
> (message cmd) (shell-command cmd)
> out-file))
>
> (defun org-babel-prep-session:asymptote (session params)
> (error "Asymptote does not support sessions"))
>
> +(defun org-babel-asymptote-var-to-asymptote (pair)
> + "Convert an elisp val into a string of asymptote code specifying a var
> +of the same value."
> + (let ((var (car pair))
> + (val (if (symbolp (cdr pair))
> + (symbol-name (cdr pair))
> + (cdr pair))))
> + (cond
> + ((integerp val)
> + (format "int %S=%S;" var val))
> + ((floatp val)
> + (format "real %S=%S;" var val))
> + ((stringp val)
> + (format "string %S=\"%s\";" var val))
> + ((listp val)
> + (let* ((dimension-2-p (not (null (cdr val))))
> + (dim (if dimension-2-p "[][]" "[]"))
> + (type (org-babel-asymptote-define-type val))
> + (array (org-babel-asymptote-table-to-array
> + val
> + (if dimension-2-p '(:lstart "{" :lend "}," :llend "}")))))
> + (format "%S%s %S=%s;" type dim var array))))))
> +
> +(defun org-babel-asymptote-table-to-array (table params)
> + "Convert values of an elisp table into a string of an asymptote array.
> +Empty cells are ignored."
> + (labels ((atom-to-string (table)
> + (cond
> + ((null table) '())
> + ((not (listp (car table)))
> + (cons (if (and (stringp (car table))
> + (not (string= (car table) "")))
> + (format "\"%s\"" (car table))
> + (format "%s" (car table)))
> + (atom-to-string (cdr table))))
> + (t
> + (cons (atom-to-string (car table))
> + (atom-to-string (cdr table))))))
> + ;; Remove any empty row
> + (fix-empty-lines (table)
> + (delq nil (mapcar (lambda (l) (delq "" l)) table))))
> + (orgtbl-to-generic
> + (fix-empty-lines (atom-to-string table))
> + (org-combine-plists '(:hline nil :sep "," :tstart "{" :tend "}") params))))
> +
> +(defun org-babel-asymptote-define-type (data)
> + "Determine type of DATA. DATA is a list.
> +Type symbol is returned as 'symbol. The type is usually the type
> +of the first atom encountered, except for arrays of int where
> +every cell must be of int type."
> + (labels ((anything-but-int (el)
> + (cond
> + ((null el) nil)
> + ((not (listp (car el)))
> + (cond
> + ((floatp (car el)) 'real)
> + ((stringp (car el)) 'string)
> + (t
> + (anything-but-int (cdr el)))))
> + (t
> + (or (anything-but-int (car el))
> + (anything-but-int (cdr el)))))))
> + (or (anything-but-int data) 'int)))
> +
> (provide 'org-babel-asymptote)
> ;;; org-babel-asymptote.el ends here
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode
next prev parent reply other threads:[~2010-06-02 15:08 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-30 19:11 [babel][PATCH] org-babel-asymptote now handles variables Nicolas Goaziou
2010-06-02 15:06 ` Eric Schulte [this message]
2010-06-02 21:12 ` Nicolas Goaziou
2010-06-04 15:51 ` Eric Schulte
-- strict thread matches above, loose matches on Subject: below --
2010-05-30 9:47 [babel] [PATCH] " Nicolas Goaziou
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87typlzd40.fsf@gmail.com \
--to=schulte.eric@gmail.com \
--cc=emacs-orgmode@gnu.org \
--cc=n.goaziou@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).