-
Notifications
You must be signed in to change notification settings - Fork 320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Raw LaTeX tables with cross-references no longer render #7236
Comments
Thanks for the report. There's a bit of a regression that we will fix right away, but that's not the entire situation. Some of this LaTeX code will unfortunately have to change a bit. Specifically, the problem here is that |
Quick workaround first. If you need this to work right away, the following will get you back on track:
Note, for future reference, that we expect latex raw blocks to be surrounded by I'm going to fix the regression so that the following syntax works:
Note the rawblock syntax, and note the lack of |
Ah cool, this workaround is great. I'm actually fine with this syntax:
since it feels more Quarto-y and less hacky than using the I personally don't use raw LaTeX like this, though, but @cbgoodman does |
All the following raw block syntaxes should be respected once the PR lands:
|
This may have re-regressed? This doesn't work with 1.4.504: ---
title: "Raw TeX cross referencing"
format:
pdf: default
---
See @tbl-tex:
```{=latex}
\begin{tabular}{@{}lll@{}}
\label{tbl-tex}
\caption{Some letters with LaTeX}
\toprule
Column 1 & Column 2 & Column 3 \\ \midrule
A & B & C \\
D & E & F \\
G & H & I \\ \bottomrule
\end{tabular}
``` It causes this error:
|
In 1.4.504, I actually can't seem to run anything in this test https://github.com/quarto-dev/quarto-cli/blob/cfebb28fedb4ddd530bdcd7f4d7ffd782a5a6509/tests/docs/smoke-all/2023/10/16/7236.qmd —it leads to the same error:
|
Our test passes though... 😓 🤔 |
That error makes me suspect a stale |
Yes I can verify that this happens only when the document is inside a quarto project, even of If I remove @cscheid should we open a new issue maybe ? I don't know well this |
And now I can't repro anymore :( We should never crash. With that said, I'll reiterate my previous "using raw latex for \label{}s etc is fraught, and you should really really consider if that's what you need to do" stance. |
Agree with that - especially I don't expect a rendering for a same .qmd working when outside a project context and not working inside a project context, especially with default type project. |
This error does indeed happen inside a quarto project haha I'm all for avoiding using raw latex :) This was just one of my attempts to try sticking a longtable (#7604 (comment)) in the document, but then I ran into this weird issue 🤷♂️ |
Andrew can you please not use our software in a completely reasonable manner? :lolsob: |
This is a "fun" one that happens more broadly than we might expect. What's going on is that since we now have the ability to catch and treat raw latex blocks as real cross-referenceable elements inside Quarto, the caption content can itself be entirely a raw block. That confuses In 1.5 we're probably going to do away with the distinction between project and non-project renders entirely, which should make this class of bugs go away. (PR incoming.) |
Is this problem being resolve in |
@likanzhan please open a GutHub Discussion if you are not sure there is still a bug. You can reference this issue. |
Bug description
In Quarto 1.3, it was possible to use Quarto's cross references with raw LaTeX tables by including
{#tbl-id}
in the table caption (see here). In Quarto 1.4.x, this doesn't work anymore.Steps to reproduce
This document would render fine in 1.3:
Expected behavior
That document would render into this (lol horribly placed table)
Actual behavior
Quarto 1.4 produces this warning before invoking
xelatex
:And then after starting
xelatex
and showing the dreadedupdating tlmgr
message, it produces this error:Your environment
Quarto check output
The text was updated successfully, but these errors were encountered: