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

Tutorial PDF build failing in PDB section #4229

Closed
peterjc opened this issue Feb 8, 2023 · 4 comments · Fixed by #4230
Closed

Tutorial PDF build failing in PDB section #4229

peterjc opened this issue Feb 8, 2023 · 4 comments · Fixed by #4230

Comments

@peterjc
Copy link
Member

peterjc commented Feb 8, 2023

Looks to be a failure from 214c4f6 on #3898, not noticed at the time since LaTeX takes too long for use on CI:

$ cd Docs/
$ make
...
! LaTeX Error: Bad math environment delimiter.

See the LaTeX manual or LaTeX Companion for explanation.
Type  H <return>  for immediate help.
 ...                                              
                                                  
l.1136 ...'1A8O', 0, 'A', \(' ', 153, ' '\)\) ILE}
                                                  ) are optional, and will be
? 
! Double superscript.
<recently read> ^
                 
l.1136 ...'1A8O', 0, 'A', \(' ', 153, ' '\)\) ILE}
                                                  ) are optional, and will be
? 
! Double superscript.
<recently read> ^
                 
l.1136 ...'1A8O', 0, 'A', \(' ', 153, ' '\)\) ILE}
                                                  ) are optional, and will be
? 

! LaTeX Error: Bad math environment delimiter.

See the LaTeX manual or LaTeX Companion for explanation.
Type  H <return>  for immediate help.
 ...                                              
                                                  
l.1136 ...'1A8O', 0, 'A', \(' ', 153, ' '\)\) ILE}
                                                  ) are optional, and will be
? 
...

From here in Doc/Tutorial/chapter_pdb.tex:

\subsubsection{Protein Internal Coordinate (.pic) files and default values}

A file format is defined in the \texttt{PICIO} module to describe protein chains as hedra and
dihedra relative to initial coordinates.  All parts of the file other than the residue sequence
information (e.g. \texttt{\('1A8O', 0, 'A', \(' ', 153, ' '\)\) ILE}) are optional, and will be
filled in with default values if not specified and \texttt{read\_PIC\(\)} is called with the
\texttt{defaults=True} option.  Default values are calculated from Sep 2019 Dunbrack
cullpdb\_pc20\_res2.2\_R1.0.

It is not immediately obvious what it wrong (all to common with LaTeX), but probably verbatim would be better than \texttt{...} here?

@peterjc
Copy link
Member Author

peterjc commented Feb 8, 2023

@rob-miller does this display as you'd like it?

A file format is defined in the \texttt{PICIO} module to describe protein chains as hedra and
dihedra relative to initial coordinates.  All parts of the file other than the residue sequence
information (e.g. \verb|('1A8O', 0, 'A', (' ', 153, ' ')) ILE|) are optional, and will be
filled in with default values if not specified and \verb|read_PIC()| is called with the
\texttt{defaults=True} option.  Default values are calculated from Sep 2019 Dunbrack
cullpdb\_pc20\_res2.2\_R1.0.

Our existing verbatim usage is split between \verb|...| and \verb+...+, either delimiter seems fine here.

@JoaoRodrigues
Copy link
Member

Linking to #4227

@rob-miller
Copy link
Contributor

In case you don't want to use the PR it is just removing the \'s escaping the parens - sorry I missed this.

information (e.g. \texttt{('1A8O', 0, 'A', (' ', 153, ' ')) ILE}) are optional

@peterjc
Copy link
Member Author

peterjc commented Feb 8, 2023

Thank you 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants