Re: Preview musings (and a question about pnmcrop)

2024-08-16 Thread José Matos
On Fri, 2024-08-16 at 10:10 +0100, José Matos wrote: > Question: > Does anyone have a reason to keep the current two stages process? The new code becomes simply: def crop_files(pnmcrop, basename): for file in glob.glob(f"{basename}*.ppm"): tmp = tempfile.TemporaryFile() conv_c

Re: 68 tests failing due to recent commits

2024-08-16 Thread Scott Kostyshak
On Thu, Aug 15, 2024 at 09:28:00AM GMT, Jürgen Spitzmüller wrote: > Am Mittwoch, dem 14.08.2024 um 18:40 -0400 schrieb Scott Kostyshak: > > Below are tests that are failing due to recent commits. I'm on > > vacation so I'd rather not dig into these myself. If necessary, I can > > provide more infor

Re: Preview musings (and a question about pnmcrop)

2024-08-16 Thread José Matos
On Fri, 2024-08-16 at 10:10 +0100, José Matos wrote: > Hi, >   my main focus now is to try to clean the python scripts that we > use. One question that I noticed while searching for this. Is it possible now to use preview when using minted? minted requires to invoke LaTeX with the -shell-escape

Re: [LyX/master] fix get_quoted_value

2024-08-16 Thread Jürgen Spitzmüller
Am Freitag, dem 16.08.2024 um 10:00 +0100 schrieb José Matos: > Thank you for implementing these changes in the code. :-) I have to thank for the instruction! -- Jürgen -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel

Preview musings (and a question about pnmcrop)

2024-08-16 Thread José Matos
Hi, my main focus now is to try to clean the python scripts that we use. Musing: I need to look immediately into the preview stuff since I found that in python 3.13 the pipe module is gone (it is a small wrap around os.popen). The only place where we use it to call consecutively pnmcrop with op

Re: [LyX/master] fix get_quoted_value

2024-08-16 Thread José Matos
On Thu, 2024-08-15 at 15:28 +0200, Jürgen Spitzmüller wrote: > Am Donnerstag, dem 15.08.2024 um 13:03 +0100 schrieb José Matos: > > Honestly here the code, in this context, makes more sense like > > this: > > > > if val.startswith('"') and val.endswith('"'): > >     val =  val[1:-1] > > > > So it