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

Polish documentation #135

Open
2 of 3 tasks
SeanBryan51 opened this issue Sep 8, 2023 · 7 comments
Open
2 of 3 tasks

Polish documentation #135

SeanBryan51 opened this issue Sep 8, 2023 · 7 comments
Labels
documentation Improvements or additions to documentation priority:medium Medium priority issues to become high priority issues after a release.

Comments

@SeanBryan51
Copy link
Collaborator

SeanBryan51 commented Sep 8, 2023

A few improvements that can be made to the documentation:

@SeanBryan51 SeanBryan51 added the documentation Improvements or additions to documentation label Sep 8, 2023
@ccarouge
Copy link
Collaborator

About the API docs, since Sphinx can do that automatically using the Python docstrings, would it make sense to move the documentation to .rst and Sphinx? Or is the API small enough it isn't worth it?

I'm not sure I understand about the nested config options in config.yaml. I can't see a difference between our doc and the example link. Or do you mean to add an example for each nested option instead of one at the end?
I am wondering though if it would be worth limiting the table of contents to show the top option only (fluxsite but not the nested options).

ccarouge added a commit that referenced this issue Sep 11, 2023
@SeanBryan51
Copy link
Collaborator Author

SeanBryan51 commented Sep 12, 2023

About the API docs, since Sphinx can do that automatically using the Python docstrings, would it make sense to move the documentation to .rst and Sphinx? Or is the API small enough it isn't worth it?

I think docstrings won't actually be useful to us since the API for benchcab is really a command line interface. But it might be an idea for the future if we want to expose a python API? For the API documentation for the CLI, I had something like the GitHub CLI documentation in mind.

Yes I think an example for each option would help, especially for the options that are three levels deep as it is hard to tell from the reader if these options have a nested structure.

I don't really mind if the table of contents does not show all the options, but I want to keep the permanent links for each option so we can easily refer to specific options by sharing a link.

@ccarouge
Copy link
Collaborator

I agree the API is only the command line interface. But it would be great if we could use something that takes the documentation from the code and presents it on a nice web page. So we don't have 2 versions of the documentation: in the code, in the --help message and on the web. I was going to say it isn't possible with mkdocs, but I was so wrong: mkautodoc. It might be worth organising the documentation with this for the API.

I have made a start about config.yaml documentation, mainly because I needed to see it to know if that would be better or not. I've now opened a draft pull request so you can see what it could look like: #137

@tammasloughran
Copy link
Collaborator

On this page, I wanted to know more about the forty-two-site-test and the five-site-test. But the links to modelevalutation.org return a "page not found" page.

What where these intended to link to?

@SeanBryan51
Copy link
Collaborator Author

SeanBryan51 commented Oct 27, 2023

@tammasloughran the links should point to the respective experiments on modelevaluation.org, but you will need to be logged in for the links to work.

@tammasloughran
Copy link
Collaborator

I am :), still no go. Maybe they aren't shared with me?

@ccarouge
Copy link
Collaborator

Sorry @tammasloughran . We have put the wrong link in the documentation. It's the link to our own workspace for testing and development. We will update in the doc. In the mean time, these links should work for you:

  • forty-two sites
  • five sites
    All the experiments in benchcab are under the benchcab-evaluation workspace if you want to find them directly.

@ccarouge ccarouge added the priority:medium Medium priority issues to become high priority issues after a release. label May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority:medium Medium priority issues to become high priority issues after a release.
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants