-
Notifications
You must be signed in to change notification settings - Fork 282
Description
Description
The book mentions many PEcAn functions and packages but does not link to the rendered pkgdown documentation where users can find detailed API reference (parameters, return values, examples). The pkgdown documentation exists at https://pecanproject.github.io/package-documentation/develop/ but is underutilized.
The book should focus on explanation (concepts, workflows, how things fit together), while referencing pkgdown for detailed function documentation. Currently function names appear as plain text instead of clickable links.
Current state
Functions mentioned without links to pkgdown:
book_source/03_topical_pages/03_pecan_xml.Rmd:
- `PEcAn.DB::db.open`
- `PEcAn.workflow::start_model_runs`
- `PEcAn.uncertainty::write.ensemble.configs`
- `PEcAn.uncertainty::run.ensemble.analysis`
book_source/03_topical_pages/95_remote_execution.Rmd:
The `PEcAn.remote::remote.execute.cmd`
...see the `PEcAn.remote` package documentation.
(Says "see the package documentation" but doesn't provide a link)
09_rabbitmq.Rmd:
PEcAn.remote::rabbitmq_post_message function
Vignettes linked to raw github Rmd instead of rendered html:
01_introductions_user.Rmd has a table with vignettes
The "html" column is empty, but pkgdown renders these vignettes to html. These could link to the rendered versions at
e.g:
- https://pecanproject.github.io/package-documentation/develop/PEcAn.allometry/articles/AllomVignette.html
- https://pecanproject.github.io/package-documentation/develop/PEcAn.assim.batch/articles/AssimBatchVignette.html
"Under construction" sections that could link to packages:
03_pecan_xml.Rmd -- These mention packages but don't link to their pkgdown documentation.
Proposed approach:
- Convert plain text function references to hyperlinks pointing to pkgdown reference pages
- e.g. :
PEcAn.DB::db.open--> PEcAn.DB::db.open
- e.g. :
- Link "see the package" statements to actual pkgdown pages
- e.g. : "see the PEcAn.remote package documentation" -- link to the package index
- Fill in rendered html links for vignettes table
- the vignette table has an "html" column that's currently empty
- these should link to the pkgdown rendered versions
- Replace "under construction" placeholders with package links
- if detailed content isn't in the book, at least provide a link to pkgdown
- Audit the book to identify other places where package names or functions are mentioned and add links to the appropriate pkgdown pages where a reference already exists.