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

[Feature request]: sphinx documentation for gempyor API #458

Open
3 tasks
pearsonca opened this issue Jan 8, 2025 · 3 comments
Open
3 tasks

[Feature request]: sphinx documentation for gempyor API #458

pearsonca opened this issue Jan 8, 2025 · 3 comments
Assignees
Labels
documentation Relating to ReadMEs / gitbook / vignettes / etc. high priority High priority.

Comments

@pearsonca
Copy link
Contributor

pearsonca commented Jan 8, 2025

Label

documentation

Priority Label

low priority

Is your feature request related to a problem? Please describe.

Users reported difficulty with documentation for gempyor API. Some portion of that is definition the absence of that documentation, but likely some was also difficulty in finding / accessing the extant documentation.

Is your feature request related to a new application, scenario round, pathogen? Please describe.

No response

Describe the solution you'd like

Incorporate sphinx documentation generation into build. There are few stages here to implement:

  • available for local viewing - i.e. people can start a local docs server and take advantage of the like-a-webpage interface to the docs
  • as part of CI, such that sphinx is indicating docstring errors / missingness / etc for build process
  • available for web viewing - i.e. an api.flepimop.org site
@emprzy emprzy added documentation Relating to ReadMEs / gitbook / vignettes / etc. low priority Low priority. labels Jan 10, 2025
@emprzy emprzy changed the title [Feature request]: sphinx documentation [Feature request]: sphinx documentation for gempyor API Jan 10, 2025
@emprzy
Copy link
Collaborator

emprzy commented Jan 10, 2025

Note: inference.py (containing GempyorInference class) and model_info.py (containing ModelInfo class) would most immediately benefit from more thorough documentation.

@emprzy emprzy self-assigned this Jan 10, 2025
@TimothyWillard
Copy link
Contributor

TimothyWillard commented Jan 10, 2025

A good and brief tutorial on using autodoc with sphinx: https://jackmckew.dev/automatically-generate-documentation-with-sphinx.html.

My preference would be to do the first two tasks under this issue and punt the third task to its own issue and then move this from low to high priority. Thoughts?

@TimothyWillard TimothyWillard added this to the Software Quality milestone Jan 10, 2025
@emprzy
Copy link
Collaborator

emprzy commented Jan 10, 2025

I think that's a good plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Relating to ReadMEs / gitbook / vignettes / etc. high priority High priority.
Projects
None yet
Development

No branches or pull requests

3 participants