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.

Sign up for GitHub

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

Jump to bottom

Bugfix #27

Closed
wants to merge 2 commits into from
Closed

Bugfix #27

wants to merge 2 commits into from

Conversation

Dongcan-Jiang
Copy link
Contributor

No description provided.

@Dongcan-Jiang
Fix a bug on rowAnys, which is due to NA
907d0f5 
Signed-off-by: Dongcan Jiang <dongcan.jiang@gmail.com>
@Dongcan-Jiang
fix a bug, due to NA_character_
feae18d 
The bug is found on "colAlls", "allValue", "rowAnys", "colAnys", "anyValue".

Signed-off-by: Dongcan Jiang <dongcan.jiang@gmail.com>
@HenrikBengtsson
Copy link
Owner

Thanks for these. I've pull this into the develop branch, cf. commit b41ccc5.

FYI, the master branch is kept the same as the version that is on CRAN such that if someone installs the package with install_github('HenrikBengtsson/matrixStats') or via install.packages('matrixStats'), they'll get the same version.

HenrikBengtsson added a commit that referenced this pull request May 14, 2015
@HenrikBengtsson
Tweaked test and added NEWS entry [PR #27]
2cba115
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Dongcan-Jiang @HenrikBengtsson