search

PreTeXtBook / pretext

PreTeXt: an authoring and publishing system for scholarly documents
https://pretextbook.org
Other
254 stars 202 forks source link

Use `Streams` interface for xsl info/warings/errors #2136

Open siefkenj opened 2 months ago

siefkenj commented 2 months ago

Python supports Streams which allow for forwarding data as it arrives.

It would be nice if the CLI could filter/group warnings and info that is generated by the XSL. One way to do this would be if pretext core produced a stream rather than writing to stdout/stderr.

@oscarlevin Would a stream help the CLI make the output more friendly?

rbeezer commented 2 months ago

Perhaps related: XSL processing log is now communicated better. Comments on the CLI side:

https://github.com/PreTeXtBook/pretext-cli/issues/689

Code:

512775a1a8e2988a51064b4242b827e060f10353

Beautiful error messages are not a priority. Complete, helpful, robust messages are. There's lots of room for improvment here, something I started to work on back in Ann Arbor, June 2022.

We use threads to communicate the XSL log on-the-fly. Maybe streams would be just as good, and better for other aspects. But unlikely to bubble-up in my queue anytime real soon.