◐ Shell
reader mode source ↗
Skip to content

[WIP] Add documentation#59

Closed
suargi wants to merge 20 commits into
su2code:developfrom
suargi:feature_additional_documentation
Closed

[WIP] Add documentation#59
suargi wants to merge 20 commits into
su2code:developfrom
suargi:feature_additional_documentation

Conversation

@suargi

@suargi suargi commented Aug 25, 2021

Copy link
Copy Markdown

Add documentation from my personal notes, pull requests and the template config file into SU2 website.

I have been taking notes on how to use SU2 and important features that I think are missing in the documentation. I would like to contribute to the documentation. Besides, in my opinion the template config file and pull requests are not a source for documentation, although the config file can support the documentation. So, I gathered major point from them and included into the website documentation.

It is a little bit difficult to link the website content and the two references in https://github.com/su2code/Documentation/tree/master/Articles as they have "different" content and the papers are somehow outdated.

There are certain aspects that I am not sure and would be nice if you could double check. I have marked them with a "?" for further discussion.

Sorry for grammar errors :)

suargi added 2 commits August 25, 2021 12:32
Add personal documentation and documentation from pull requests and the template config file
@pr-triage pr-triage Bot added the PR: draft label Aug 25, 2021
@suargi suargi changed the base branch from master to develop August 25, 2021 12:47

@pcarruscag pcarruscag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

Awesome! Thank you for adding these much-needed pages on gradients/limiters and turbulence models.
I have still have to finish reading the turbulence part, but here are some suggestions:

8 hidden conversations Load more…
suargi and others added 14 commits August 26, 2021 11:00
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
Co-authored-by: Pedro Gomes <38071223+pcarruscag@users.noreply.github.com>
@suargi suargi removed the PR: draft label Sep 24, 2021
@suargi suargi marked this pull request as ready for review September 24, 2021 12:09
@suargi

suargi commented Jun 28, 2022

Copy link
Copy Markdown
Author

This PR is quite old and I do not remember exactly what is its status. I will create additional but smaller pull request, e.g., #81 in order to include all the updated document ion from this PR. At the end, this PR will be useless and can be closed.

@suargi suargi closed this by deleting the head repository Apr 16, 2024
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.

2 participants