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

Updating documentation #1212

Merged
merged 3 commits into from
Apr 27, 2023
Merged

Conversation

AnHeuermann
Copy link
Member

Related Issues

Realted to OpenModelica/OpenModelica#10542

Purpose

  • Fixing some warning for sphinx build of user's guide of OpenModelica

  - Fixing some warning for sphinx build of user's guide
@AnHeuermann AnHeuermann self-assigned this Apr 21, 2023
@AnHeuermann AnHeuermann marked this pull request as ready for review April 25, 2023 11:49
@AnHeuermann
Copy link
Member Author

@arun3688 The OMSimulator User's Guide is broken for many API functions, where e.g. #CAPTION# is used, see e.g. https://openmodelica.org/doc/OMSimulator/master/html/api/setVariableStepSize.html.

We could remove those in this PR as well and fix the documentation.

@AnHeuermann
Copy link
Member Author

The doc itself isn't broken, but the HTML files for each API functions should not be added to the final documentation.

@arun3688
Copy link
Contributor

@AnHeuermann but we are not adding any html files, do you say that while building the documentation the html files are added ?

@AnHeuermann
Copy link
Member Author

Looks like it. Maybe we add all .rst files at some place? What if we change them to be something like .rst.inc?

@arun3688
Copy link
Contributor

@AnHeuermann, this is happening with your changes ?

@AnHeuermann
Copy link
Member Author

The online documentation has an HTML file for each API function, see https://openmodelica.org/doc/OMSimulator/master/html/api/.
But I'm not sure why those files are generated in the first place. In doc/UsersGuide/source/conf.py only snippets from each api function.rst file are used.
I can try to change this in this PR as well.

@AnHeuermann
Copy link
Member Author

Okay, now api/ is excluded from the spinx build. This also removes a bunch of warnings from the build.

@AnHeuermann AnHeuermann merged commit eaf993a into OpenModelica:master Apr 27, 2023
@AnHeuermann AnHeuermann deleted the update-doc branch April 27, 2023 09:04
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