-
Notifications
You must be signed in to change notification settings - Fork 10
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
Create docs for xarray_filters #21
Comments
@gbrener I can work with you on the docs overhaul here and elsewhere. I'll start with the notebooks in xarray_filters and see how those may be adjusted as needed for auto-docs generation / testing as described in Elm issue 188 (Documentation structure overhaul). |
@PeterDSteinberg , do we want to keep the docs independent between |
That's a good question, @gbrener - I'm not sure what is optimal approach on docs (separate docs for 3 repos vs one docs page). My thought at this point is we should have 3 separate documentation pages, if that can be done somewhat easily with the auto-docs / notebook testing harness we are developing. So each repo has its own docs folder, but the Elm docs link to xarray_filters and Earthio as needed. For the near term until December primarily focus on usages of Elm that work with xarray_filters. Later in Phase II, we can work on more Elm docs that show machine learning for non-Earth science data structures. More complete explanations and descriptions of use cases can be given in Elm docs and if xarray_filters/earthio are more of an API-description than complete story, I think that is fine for a while. Earthio - I think the docs for the next 6+ months should treat Earthio as mainly an environment that installs "everything" needed for climate/satellite data. It would be nice if earthio issue 31 could be resolved in a way that effectively moves the functionality of what is now Best way to move forward between now and November 14 is to
|
Regarding docstrings and how many / much of them to make, I think we should follow the dask level of docstring usage, where some internal functions have shorter docstrings than public interface functions. |
Link to the Elm-docs-overhaul issue.
The text was updated successfully, but these errors were encountered: