Skip to content
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

Fix doctest in src/sage/algebras/quantum_groups/quantum_group_gap.py #36149

Merged

Conversation

dcoudert
Copy link
Contributor

We fix the following doctest reported on sage-release

sage -t --long --warn-long 216.1 --random-seed=336915508420415526544989948662812522502 src/sage/algebras/quantum_groups/quantum_group_gap.py
**********************************************************************
File "src/sage/algebras/quantum_groups/quantum_group_gap.py", line 1581, in sage.algebras.quantum_groups.quantum_group_gap.QuantumGroupModule._latex_
Failed example:
    latex(S)
Expected:
    \begin{tikzpicture}
    ...
    \end{tikzpicture}
Got:
    \begin{tikzpicture}[>=latex,line join=bevel,]
    %%
    \node (node_0) at (144.52bp,292.63bp) [draw,draw=none] {$\langle 1 {(1 v_0 \otimes 1 v_0)} \rangle$};
      \node (node_1) at (144.52bp,221.71bp) [draw,draw=none] {$\langle 1 {(1 v_0 \otimes F[a1] v_0)} + q^{-1} {(F[a1] v_0 \otimes 1 v_0)} \rangle$};
      \node (node_2) at (52.519bp,150.47bp) [draw,draw=none] {$\langle 1 {(F[a1] v_0 \otimes F[a1] v_0)} \rangle$};
      \node (node_3) at (236.52bp,150.47bp) [draw,draw=none] {$\langle 1 {(1 v_0 \otimes F[a1+a2] v_0)} + q^{-1} {(F[a1+a2] v_0 \otimes 1 v_0)} \rangle$};
      \node (node_4) at (144.52bp,79.224bp) [draw,draw=none] {$\langle 1 {(F[a1] v_0 \otimes F[a1+a2] v_0)} + q^{-1} {(F[a1+a2] v_0 \otimes F[a1] v_0)} \rangle$};
      \node (node_5) at (144.52bp,8.3018bp) [draw,draw=none] {$\langle 1 {(F[a1+a2] v_0 \otimes F[a1+a2] v_0)} \rangle$};
      \draw [blue,->] (node_0) ..controls (144.52bp,274.4bp) and (144.52bp,255.54bp)  .. (node_1);
      \definecolor{strokecol}{rgb}{0.0,0.0,0.0};
      \pgfsetstrokecolor{strokecol}
      \draw (153.02bp,257.33bp) node {$1$};
      \draw [blue,->] (node_1) ..controls (119.45bp,201.84bp) and (89.94bp,179.63bp)  .. (node_2);
      \draw (118.02bp,186.09bp) node {$1$};
      \draw [red,->] (node_1) ..controls (169.43bp,201.96bp) and (198.49bp,180.09bp)  .. (node_3);
      \draw (209.02bp,186.09bp) node {$2$};
      \draw [red,->] (node_2) ..controls (77.09bp,130.97bp) and (106.75bp,108.65bp)  .. (node_4);
      \draw (118.02bp,114.85bp) node {$2$};
      \draw [blue,->] (node_3) ..controls (211.6bp,130.71bp) and (182.55bp,108.85bp)  .. (node_4);
      \draw (209.02bp,114.85bp) node {$1$};
      \draw [red,->] (node_4) ..controls (144.52bp,60.535bp) and (144.52bp,41.477bp)  .. (node_5);
      \draw (153.02bp,43.604bp) node {$2$};
    %
    \end{tikzpicture}
**********************************************************************
1 item had failures:
   1 of   6 in sage.algebras.quantum_groups.quantum_group_gap.QuantumGroupModule._latex_
    [537 tests, 1 failure, 9.28 s]
----------------------------------------------------------------------
sage -t --long --warn-long 216.1 --random-seed=336915508420415526544989948662812522502 src/sage/algebras/quantum_groups/quantum_group_gap.py  # 1 doctest failed

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@github-actions
Copy link

Documentation preview for this PR (built with commit 25232ec; changes) is ready! 🎉

Copy link
Collaborator

@kwankyu kwankyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@vbraun vbraun merged commit 24de879 into sagemath:develop Sep 1, 2023
15 of 18 checks passed
@mkoeppe mkoeppe added this to the sage-10.2 milestone Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants