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
Reduce Parameter API #121
Reduce Parameter API #121
Conversation
faf56dc
to
2923a32
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the new api documentation - particularly the table showing classes with the one-liner description. It's much easier to navigate.
I miss the readme landing page for the documentation with our badges and "Word from our sponsor" section, also giving continuity between the readthedocs and github view. Perhaps we move detailed install and config out of the readme and into the relevant pages in the docs and keep a simplified readme?
- expose ParameterList and Narrative directly on smif.parameters - move implementation to hidden submodules - create Parameter class to hold individual parameter metadata Intended to reduce the API size and be more explicit about data required (replacing plain dicts with data object).
- gives finer control over auto-generated docs - requires better-apidoc package, added to test-requirements
…l for ReadTheDocs
395274c
to
564eca6
Compare
Testing an opinion on how to organise subpackages and expose classes in
__init__.py
Also introducing a
Parameter
object and some (hopefully) simplification ofParameterList
.