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

Rename vignettes #267

Merged
merged 4 commits into from
Nov 14, 2020
Merged

Rename vignettes #267

merged 4 commits into from
Nov 14, 2020

Conversation

ryantibs
Copy link
Member

So that they will appear in the order we want them to.

@ryantibs
Copy link
Member Author

@capnrefsmmat I didn't mean to include the first commit f279444, that was a branch mistake (I branched from my covidhub-eval branch). But that branch should be merged into main anyway, per #240. Once Katie merges that, you can just merge this one, to save yourself a cherry-picking task.

@ryantibs
Copy link
Member Author

@capnrefsmmat I just fixed the correlation vignette to study case and death rates (rather than counts), since the former is the only thing that makes sense for "sliced by time". (Oops! what a goof.) And for "sliced by location", they're equivalent.

Note: I force added the html file to save you from rerunning it, since it takes so long to knit. You can see I rewrote the text a bit because the results changed. You can delete the html file from the repo once you've looked at it.

@capnrefsmmat
Copy link
Contributor

FYI adding the HTML is unnecessary, since I have to rebuild the vignette to make the pkgdown site anyway. Soon #266 will make the vignette super-fast and this won't be a problem.

I'll rebuild the site, check the result, and merge this in.

@ryantibs
Copy link
Member Author

Thanks! (I just meant to add the html so you could look at it right away, and if you spotted anything, you could reply without needing to rebuild the pkgdown site.)

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

Successfully merging this pull request may close these issues.

None yet

2 participants