Skip to content

Add Sphinx-based Documentation#4

Merged
domgz merged 16 commits into
masterfrom
documentation
Jan 25, 2023
Merged

Add Sphinx-based Documentation#4
domgz merged 16 commits into
masterfrom
documentation

Conversation

@domgz

@domgz domgz commented Dec 22, 2022

Copy link
Copy Markdown
Contributor

Adds Sphinx-based documentation.

@AndreaQuerio This documentation is still a draft because a diagram is missing. All the other parts are ready to go. Your review will be requested (and you will get an email notification) when the diagram is ready and the pull request is no longer a draft.

…ate setup.py and packageinfo.py

* Add Sphinx-based documentation.

* Updated README.md so that it is consistent with the documentation.

* Update year in copyright statement on LICENSE.

* Set OSP_CORE_MAX to v4.0.0.
@domgz domgz self-assigned this Dec 22, 2022
Comment thread README.md
Comment on lines +30 to +35
The wet synthesis wrappers require a working installation of
[OpenFOAM 8](https://openfoam.org/version/8/),
[SUNDIALS](https://github.com/LLNL/sundials)
([v6.1.1](https://github.com/LLNL/sundials/releases/tag/v6.1.1) is known to
work) and [7-zip](https://www.7-zip.org/). Follow the links to find
installation instructions for each tool.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@AndreaQuerio This documentation assumes that only OpenFOAM is compatible, although theoretically, it looks like things shouldm work with Ansys Fluent too, but I cannot test this, so I left it out. If things work with Fluent, please modify the documentation yourself. Be careful, this is not the only place were you would have to mentiont Fluent.

@domgz
domgz marked this pull request as ready for review December 22, 2022 16:24
@domgz
domgz requested a review from AndreaQuerio December 22, 2022 16:24
@domgz

domgz commented Dec 22, 2022

Copy link
Copy Markdown
Contributor Author

@AndreaQuerio The documentation is ready, please review it and add any modifications that you find adequate. Please pay special attention to the correctness of the diagram on the "Usage" section. You can read the documentation here. Every change you push to this branch will be reflected after a few minutes on that page.

@domgz

domgz commented Jan 12, 2023

Copy link
Copy Markdown
Contributor Author

@AndreaQuerio The documentation is ready, please review it and add any modifications that you find adequate. Please pay special attention to the correctness of the diagram on the "Usage" section. You can read the documentation here. Every change you push to this branch will be reflected after a few minutes on that page.

@AndreaQuerio Just a reminder to review what I did and either agree or add any modifications so that the documentation can be merged.

@domgz
domgz merged commit c0efe66 into master Jan 25, 2023
@domgz
domgz deleted the documentation branch January 25, 2023 17:03
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.

2 participants