NIEHS / beethoven

BEETHOVEN is: Building an Extensible, rEproducible, Test-driven, Harmonized, Open-source, Versioned, ENsemble model for air quality
https://niehs.github.io/beethoven/
Other
2 stars 0 forks source link

packagedown website for project #212

Open kyle-messier opened 7 months ago

kyle-messier commented 7 months ago

https://pkgdown.r-lib.org/

kyle-messier commented 6 months ago

@eva0marques See my open pull request #232 . The nlcd_ratio vignette is causing pkgdown issues. I noticed there are hard-coded paths to "/Voumes/set"

  1. Can we make those relative paths? I think this would bring up data access issues, which I know is a current issue. We don't have the nlcd data in ./input/data yet
  2. Alternatively, I'd be fine ignoring this for now. I think adding it to the .Rbuildignore could bypass the issue

Thoughts @sigmafelix @eva0marques ?

eva0marques commented 6 months ago

Yes I see two solutions: 1) as you said, put it in the gitignore 2) create nlcd testdata on a very small area to show to future users how NLCD extraction works (this is what I did on my own project).

kyle-messier commented 6 months ago

@eva0marques ok, trying (1) for now. Thanks!

sigmafelix commented 6 months ago

@Spatiotemporal-Exposures-and-Toxicology It appears the landing page is set readme.html. Other pages, for example, https://spatiotemporal-exposures-and-toxicology.github.io/NRTAPmodel/reference/index.html, are working.

sigmafelix commented 6 months ago

Looks like it works great now.

kyle-messier commented 6 months ago

@sigmafelix indeed it does. looks great! thank you as well @mitchellmanware for helping. I'll move it to review for now. low priority refactoring/customizing.

kyle-messier commented 6 months ago