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

Incorrect rendered formulas #8732

Merged
merged 1 commit into from
Feb 19, 2025
Merged

Conversation

albert-github
Copy link
Contributor

The formula in Documentation/doc/biblio/cgal_manual.bib

 @article{cgal:cgm-fobbo-11,
   title={Fast Oriented Bounding Box Optimization on the Rotation Group $\SO(3, \mathrm{R})$},
   author={Chang, Chia-Tche and Gorissen, Bastien and Melchior, Samuel},

renders incorrectly (see Optimal_bounding_box/citelist.html) as there is no command \SO in LaTeX

The formula in doc/Kernel_d/CGAL/Kernel_d/Aff_transformation_d.h

\pre \f$ sin_num^2 + cos_num^2 = den^2\f$ and \f$ 0 \leq e_1 < e_2 < d\f$.

renders in an unexpected way (see Kernel_d/classCGAL_1_1Aff__transformation__d.html) as the _ is seen as subscript operator in LaTeX

The formula in Documentation/doc/biblio/cgal_manual.bib
```
 @Article{cgal:cgm-fobbo-11,
   title={Fast Oriented Bounding Box Optimization on the Rotation Group $\SO(3, \mathrm{R})$},
   author={Chang, Chia-Tche and Gorissen, Bastien and Melchior, Samuel},

```
renders incorrectly (see Optimal_bounding_box/citelist.html) as there is no command `\SO` in LaTeX

The formula in doc/Kernel_d/CGAL/Kernel_d/Aff_transformation_d.h
```
\pre \f$ sin_num^2 + cos_num^2 = den^2\f$ and \f$ 0 \leq e_1 < e_2 < d\f$.
```
renders in an unexpected way (see Kernel_d/classCGAL_1_1Aff__transformation__d.html) as the `_` is seen as subscript operator in LaTeX
@albert-github
Copy link
Contributor Author

Old renderings:

image

image

New renderings:

image

image

@afabri
Copy link
Member

afabri commented Feb 10, 2025

@sloriot let's just merge it in master without going through the testsuite.

@MaelRL MaelRL added the Doc bug label Feb 13, 2025
@MaelRL MaelRL added this to the 6.1-beta milestone Feb 13, 2025
@sloriot
Copy link
Member

sloriot commented Feb 19, 2025

Successfully tested in CGAL-6.1-Ic-90

@sloriot sloriot merged commit ad215de into CGAL:master Feb 19, 2025
8 checks passed
@sloriot sloriot deleted the feature/bug_formulas branch February 19, 2025 06:15
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.

4 participants