ICESAT-2HackWeek / coastal_topobathy

Team project aimed at extracting sections of tracklines, and identifying bathymetry and coastal morphology - can be extended to other questions!
https://icesat-2hackweek.github.io/coastal_topobathy/
4 stars 7 forks source link

docs code added to the refactoring branch #7

Closed eeholmes closed 2 years ago

eeholmes commented 2 years ago

@iled Hi Julio, I forked the repo and added docs code to the refractoring branch so that the documentation is built automatically off your docstrings you added to the functions. You can see the docs that it build here

https://eeholmes.github.io/coastal_topobathy/index.html

The documentation is being built automatically with sphinx-doc autoapi.extension. I figured that was easiest because we then just add docstrings to the functions and the documentation automatically appears without us having to write .rst files.

Here is the code. I added the docs and .github/workflows folders. https://github.com/eeholmes/coastal_topobathy/tree/refactoring

Let me know if this looks ok and I'll put in a pull request.

iled commented 2 years ago

Hi Eli! Sorry for the delay, I'm going through a couple super busy weeks.

I saw the docs page, it is looking great! I really like that examples become more accessible. Thanks!

That sphix extension is very useful, indeed. That is yet another reason why it is worth to put the effort in documenting the code! Then writing .rst files can be reserved to write up separate pages, not to document the code itself.

I wrote a couple comments in #8

Would it be worthwhile to have the compiled docs reside in the project domain?

iled commented 2 years ago

Sweet! Docs already in main as of f027c49 and under the project domain. Thanks @eeholmes!