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

Censor shape can be changed #186 #187

Merged
merged 7 commits into from Apr 6, 2017
Merged

Conversation

@mingrisch
Copy link
Contributor

@mingrisch mingrisch commented Apr 6, 2017

Addressing #186, i have implemented a keyword argument censor.shape. This argument accepts numerical values for ggplot shapes and defaults to 3 (equivalent to the previously used "+").

Censor shape can be changed to "|" by passing censor.shape =124 (see http://sape.inf.usi.ch/quick-reference/ggplot2/shape). I have updated README.rmd, but I was unable to knit README.md due to some badges, this still needs to be done.

This is my first pull request ever, so I am not so sure if I have done everything as I should.

mingrisch and others added 7 commits Apr 6, 2017
Hint: Shape values need to be passed numerically, see http://sape.inf.usi.ch/quick-reference/ggplot2/shape

"|", which is a sensible shape for censors, has the code 124.
Hint: Shape values need to be passed numerically, see http://sape.inf.usi.ch/quick-reference/ggplot2/shape

"|", which is a sensible shape for censors, has the code 124.
@kassambara kassambara merged commit b07181d into kassambara:master Apr 6, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@kassambara
Copy link
Owner

@kassambara kassambara commented Apr 6, 2017

Thank you for the pull request. Much appreciated!

kassambara added a commit that referenced this pull request Apr 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.