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

Fixed Some Syntax Errors #147

Merged
merged 1 commit into from
Jun 4, 2023
Merged

Conversation

Sam0h51
Copy link
Contributor

@Sam0h51 Sam0h51 commented Jun 3, 2023

I went back through the README.md and the Generalized-Swift-Hohenberg-Solver.cc files and made some syntax changes. I adjusted how I was declaring the matrices in the README, and I changed the path on the inline images so that they will render correctly. In the .cc file, I changed the style of the doxygen comments to better handle multi-line descriptions.

…ices, fixed the image paths in the README so that images appear inline, changed the comment style in the Generalized-Swift-Hohenberg-Solver.cc file to (hopefully) render multi-line comments better
@marcfehling
Copy link
Member

marcfehling commented Jun 3, 2023

Your commit states "hopefully" it will fix the problem. You can build the documentation on your local machine to double-check your results. Please have a look at the code-gallery README.

@bangerth bangerth merged commit 1031a96 into dealii:master Jun 4, 2023
@bangerth
Copy link
Member

bangerth commented Jun 4, 2023

Let's see what happens. It isn't going to make things worse in any case.

@bangerth
Copy link
Member

bangerth commented Jun 4, 2023

Update to #146.

@bangerth
Copy link
Member

bangerth commented Jun 4, 2023

OK, at least the pictures now show up correctly. @Sam0h51: Are the formulas at https://dealii.org/developer/doxygen/deal.II/code_gallery_Swift_Hohenberg-Solver.html as you want them?

@Sam0h51
Copy link
Contributor Author

Sam0h51 commented Jun 4, 2023

Unfortunately they aren't. The matrices still aren't showing up as matrices, and it seems the comment blocks I reformatted have just disappeared.

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 this pull request may close these issues.

3 participants