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

first vignette added #8

Merged
merged 5 commits into from
Apr 22, 2018
Merged

first vignette added #8

merged 5 commits into from
Apr 22, 2018

Conversation

erykwalczak
Copy link
Contributor

@erykwalczak erykwalczak commented Nov 15, 2017

Hi.
I'm adding a vignette that demonstrates basic functions. The code comes mostly from Matt's blog. I used a bdf file from Biosemi's site to conduct the analysis. This should make the vignette reproducible.

@erykwalczak
Copy link
Contributor Author

It seems like there was an error in Travis because I used library(tidyverse) in the vignette, instead of library(dplyr). I should have used R CMD check earlier :/

@codecov-io
Copy link

codecov-io commented Nov 15, 2017

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   20.75%   20.75%           
=======================================
  Files          17       17           
  Lines        1995     1995           
=======================================
  Hits          414      414           
  Misses       1581     1581

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f0fe07...c646601. Read the comment docs.

@bnicenboim
Copy link

Sorry, the ignorance, but It seems that the vignette is not available.

@bnicenboim bnicenboim mentioned this pull request Apr 20, 2018
@craddm craddm merged commit 9c36db7 into craddm:master Apr 22, 2018
craddm added a commit that referenced this pull request Apr 22, 2018
@craddm
Copy link
Owner

craddm commented Apr 22, 2018

Merged it into master now, thanks Eryk

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

Successfully merging this pull request may close these issues.

4 participants