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

WIP: Docs #49

Merged
merged 37 commits into from
May 17, 2022
Merged

WIP: Docs #49

merged 37 commits into from
May 17, 2022

Conversation

CFGrote
Copy link
Contributor

@CFGrote CFGrote commented Feb 11, 2022

This PR adds several improvements to the documentation. Mostly following PR #48 plus better structure in the Reference Manual

@CFGrote CFGrote changed the title Docs WIP: Docs Feb 11, 2022
@CFGrote
Copy link
Contributor Author

CFGrote commented Feb 11, 2022

Important: Merge PR #47 first and then rebase this one

@CFGrote CFGrote requested review from JunCEEE, mads-bertelsen and shervin86 and removed request for JunCEEE and mads-bertelsen February 11, 2022 16:24
@CFGrote CFGrote marked this pull request as draft February 11, 2022 16:25
@JunCEEE JunCEEE mentioned this pull request Feb 25, 2022
Carsten Fortmann-Grote added 5 commits May 3, 2022 22:56
* Renamed 'pointer' to 'reference' (there are no pointers in python)
* Removed commented code
* Removed logging since not used.
* Typo in authors list
* Remove implementation of init_parameters(), it's an abstract class, so raise
* NotImplementedError.
* black reformatting.
* Expose all user facing classes in the main __init__.
* Add summary table and toc entries to each class in the reference manual
* Add pandoc to doc requirements
* Fix mispelled module name in refman.rst
Adding material to Problem statement and insert headings according to issue #48
README.md Outdated Show resolved Hide resolved
@shervin86 shervin86 changed the base branch from develop to master May 5, 2022 08:01
@JunCEEE JunCEEE mentioned this pull request May 5, 2022
8 tasks
@CFGrote
Copy link
Contributor Author

CFGrote commented May 16, 2022

can we merge this asap please?

@JunCEEE JunCEEE marked this pull request as ready for review May 17, 2022 13:37
@JunCEEE JunCEEE merged commit 605e687 into master May 17, 2022
@shervin86
Copy link
Contributor

can we delete the docs branch now?

@JunCEEE
Copy link
Contributor

JunCEEE commented May 18, 2022

Yes

@JunCEEE JunCEEE deleted the docs branch May 18, 2022 11:33
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.

4 participants