Re: master e32c7d2: Change Python eval to send directly instead of using temporary files

[Mark Barton]

On Sep 5, 2021, at 10:18 AM, Mark Barton <mbarton98@gmail.com> wrote:

On Sep 5, 2021, at 1:10 AM, Augusto Stoffel <arstoffel@gmail.com> wrote:

On Sun,  5 Sep 2021 at 06:13, "Barton, Mark" <Mark.Barton@disney.com> wrote:

Today I was trying to export my monthly org document that uses org babel python blocks to
produce tables. I can get it to export fine if I revert python.el to the version before e32c7d2a8d
- Change Python eval to send directly instead of using temporary files.

Below I show the python session buffer, first where it is working and then where it is broken for
me. I’m running Emacs on macOS 11.5.1 and compile from the master branch.

In the “Broken session buffer” below, I recognize the code from that commit, but I really don’t
understand the problem. Any ideas on what other information I can gather that would be
useful?

Two more things you could try:

1) set `python-shell-completion-native-enable' to nil
2) pull the current master, after commit 1fdd898704

I'd be curious to see the effect of 1) before doing 2), if you have the
time for this quick test.

I created a test org and csv file to test. Before I added the inline call to the test file, changing the python-shell-completion-native-enable to nil did get the my def pd2org to work. Otherwise it would work on the second export if I did not kill the python-chain session. The inline call to file_date returns an error “Inline error: multiline result cannot be used”

The attached pdf shows the successful export when I use the previous python.el.

I just compiled from master at commit c5b654b3f1

Recent commits

c5b654b3f1 master origin/master Autoload cl-struct-slot-info

7c7a47b86e ; * etc/NEWS: Fix a recent change.

3d0276e98b Improve the documentation around the read-key/minibuffer prompting

73a90cda4a Clarify completion-list-mode NEWS entry

8f2e8add98 ; * doc/emacs/maintaining.texi (Looking Up Identifiers): Fix last change.

0972cbe42f * lisp/progmodes/xref.el: Fix defcustoms (bug#50067)

2ed2999ce5 Improve documentation of new Xref options

1fdd898704 Fixes for 'python-shell-send-string' and 'python-shell-send-file'

ba84ec8bd9 Fix error handling in 'ispell-init-process'

e6f936eb4d Cleanup tramp-tests.el

Setting the python-shell-completion-native-enable to nil cleans up the session buffer python-chain

Python 3.9.6 (default, Jun 29 2021, 05:25:02) 

[Clang 12.0.5 (clang-1205.0.22.9)] on darwin

Type "help", "copyright", "credits" or "license" for more information.

>>> 

org_babel_python_eoe

>>> 

2021-08-24org_babel_python_eoe

>>> 

org_babel_python_eoe

>>> 

org_babel_python_eoe

>>> 

2021-09-05org_babel_python_eoe

>>> 

But I now get the error that prevents the pdf export: “Inline error: multiline result cannot be used"

<ts_org_babel_python.csv>

<ts_org_babel_python.org>

<ts_org_babel_python.pdf>

Here is a simpler one to attempt to reproduce. If I comment out the #+PROPERTY line to not use a session then it works. With the #+PROPERTY line, I get the “Inline error: multiline result cannot be used”.

#+TITLE: Test Org Babel Export with Python Blocks

#+AUTHOR: Mark Barton

#+DATE: 2021-09-05

#+OPTIONS: toc:nil H:4 num:nil ^:nil

#+EXCLUDE_TAGS: ignore

#+LATEX_CLASS: article

#+LaTeX_HEADER: \usepackage[letterpaper,margin=0.5in, bottom=1in]{geometry}

#+latex_header: \usepackage{float}

#+LaTeX_HEADER: \usepackage{enumitem}\setlist[itemize]{nosep}

#+LaTeX_HEADER: \usepackage{needspace}

#+LaTeX_HEADER: \usepackage{booktabs}

#+LaTeX_HEADER: \usepackage{xcolor}

#+LaTeX_HEADER: \let\OldRule\rule

#+LaTeX_HEADER: \renewcommand{\rule}[2]{\OldRule{\linewidth}{#2}}

#+LaTeX_HEADER: \setlength{\parindent}{0pt}

#+LaTeX_HEADER: \setlength{\parskip}{12pt}

#+PROPERTY: header-args:python :session python-chain :exports results :results raw replace

      

*** Initialize python                                              :ignore:

#+NAME: file_date

#+begin_src python :var file="Roster_Report.csv" :exports none

import os

import time

print(time.strftime("%Y-%m-%d",time.localtime(os.path.getmtime(file))), end="")

#+end_src

*** Summary

Test inline call_file_date[:results output :results raw](file="./ts_org_babel_python.csv")