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

use a bitarray for DataVec NA mask #3

Closed
HarlanH opened this issue Jul 15, 2012 · 1 comment
Closed

use a bitarray for DataVec NA mask #3

HarlanH opened this issue Jul 15, 2012 · 1 comment
Labels
performance priority

Comments

@HarlanH
Copy link
Contributor

HarlanH commented Jul 15, 2012

No description provided.

@tshort tshort mentioned this issue Aug 26, 2012
Closed
HarlanH added a commit that referenced this issue Sep 23, 2012
@HarlanH
DataVec's now use a BitVector instead of a Vector{Bool}
7a744de 
Issue #3
@HarlanH
Copy link
Contributor Author

HarlanH commented Sep 24, 2012

This should be somewhat faster. (Not that I took the time to do benchmarks...) Note that some BitVector operations are not yet fully optimized, such as .==, so there's room for performance improvements just from speeding up that library.

@HarlanH HarlanH closed this as completed Sep 24, 2012
@milktrader milktrader mentioned this issue Dec 10, 2012
Closed
@bkamins bkamins mentioned this issue Dec 10, 2020
Merged
nalimilan pushed a commit that referenced this issue May 26, 2022
@alyst
Merge pull request #3 from JuliaStats/appveyor_ci
8cf5a10 
AppVeyor integration
nalimilan pushed a commit that referenced this issue May 26, 2022
@alyst
Merge pull request #3 from JuliaStats/appveyor_ci
964a1a0 
AppVeyor integration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
performance priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@HarlanH