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

Update pkgdown documentation #15

Open
8 of 16 tasks
sdhutchins opened this issue Dec 14, 2018 · 22 comments
Open
8 of 16 tasks

Update pkgdown documentation #15

sdhutchins opened this issue Dec 14, 2018 · 22 comments

Comments

@sdhutchins
Copy link
Contributor

sdhutchins commented Dec 14, 2018

ToDo

More ToDos via @grabear #15 (comment)

  • Add an About tab on the navbar
  • Add vignette's for shaurita's functions
    • Stacked Bar
    • Ordination
    • Permanova
    • Alpha Diversity
  • Hunt down pkgdown warnings:
    • image
    • image
  • Add search
  • Look at some of the pkgdown articles to get any other ideas
@sdhutchins
Copy link
Contributor Author

sdhutchins commented Dec 29, 2018

Rebuild pkgdown docs

library(pkgdown)
build_site()

To build or create a new vignette

library(pkgdown)
build_article("vignette_name")

@grabear
Copy link
Member

grabear commented Jan 3, 2019

Can we merge the "test" branch? @sdhutchins I'm working on vignettes on my current branch "pkg-doc-branch" and I wanted to start messing with pkgdown.

@grabear
Copy link
Member

grabear commented Jan 3, 2019

Or can I merge the dev-master into test, and then test into my own branch?

@sdhutchins
Copy link
Contributor Author

I think the test branch might be too messy to merge - I did a lot of weird things over there if I recall correctly (non pkgdown things)...

If you want I can do a quick pkgdown branch with just some of the base pkgdown things I implemented in the test branch.

@grabear
Copy link
Member

grabear commented Jan 3, 2019

That would be legit!

@sdhutchins
Copy link
Contributor Author

Also, check out this package's pkgdown. Nice example imo.

https://valr.hesselberthlab.org/index.html

@grabear
Copy link
Member

grabear commented Jan 4, 2019

Do you know if there is a quicker way to build the reference based on the roxygen tag @family?? @sdhutchins

That's what I'm going to do anyway, but I can't find anything in the docs (so far).

@sdhutchins
Copy link
Contributor Author

sdhutchins commented Jan 4, 2019

Yes/kinda? check out this link: https://pkgdown.r-lib.org/reference/build_reference.html#yaml-config

@sdhutchins
Copy link
Contributor Author

image

@sdhutchins
Copy link
Contributor Author

reference:
- title: Data Visualization
  contents:
  - ends_with("_plot")

I don't know how helpful that will be....

@grabear
Copy link
Member

grabear commented Jan 4, 2019

Yea, I was hoping I could port all of the @family Data Visualization functions and make that a reference section, but it seems that pkgdown doesn't do that out of the box..

I filed an issue, so we should hopefully get some insight. r-lib/pkgdown#954

@sdhutchins
Copy link
Contributor Author

Reference r-lib/pkgdown#248 as well. Someone asked there.

@sdhutchins
Copy link
Contributor Author

They made it where you can base it on the @concept tag.

@grabear
Copy link
Member

grabear commented Jan 4, 2019

So do we need to do that with roxygen? @sdhutchins

@sdhutchins
Copy link
Contributor Author

sdhutchins commented Jan 4, 2019

So do we need to do that with roxygen? @sdhutchins

Yeah, but it may be that it's easy enough to implement selecting by @family instead of us having to go back and add the @concept tag to all of our functions.

About the @concept tag:

Aliases form the index that ? searches. Use @aliases space separated aliases to add additional aliases.

@concept add extra keywords that will be found with help.search()

@grabear
Copy link
Member

grabear commented Jan 4, 2019

Hrmmm... I'm just going to use RStudio to get all of the functions with the family tag:

Edit > Find in Files "@family"

It actually works nicely and it points out the inconsistencies in family names:

image

@sdhutchins
Copy link
Contributor Author

Oh dear! lol Does work though!

@sdhutchins
Copy link
Contributor Author

I bet there is a regex that lets you do this and you could build a function for it but....

@grabear
Copy link
Member

grabear commented Jan 4, 2019

More Todos:

  • Add an About tab on the navbar
  • Add vignette's for shaurita's functions
    • Stacked Bar
    • Ordination
  • Hunt down pkgdown warnings:
    • image
    • image
  • Add search
  • Look at some of the pkgdown articles to get any other ideas

@grabear
Copy link
Member

grabear commented Jan 4, 2019

Should I commit all of the build_site() files? @sdhutchins

@sdhutchins
Copy link
Contributor Author

Should I commit all of the build_site() files? @sdhutchins

Yeah, go ahead and do so.

On the warnings, typically the topic must be a valid R expression means the function does not exist or there is no .rd file for it.

I don't know if pkgdown supports the pretty print tag. The seealso is self-explanatory.

@sdhutchins
Copy link
Contributor Author

FYI @grabear

This week, I'm going to try to add more in-depth documentation to the alpha_diversity and ordination functions. I'm also going to integrate the search box for our docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants