-
Notifications
You must be signed in to change notification settings - Fork 227
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
Implement BSI Addition #260
Comments
This has been implemented. I am preparing a pull request with this and some other enhancements. |
@guymolinari Did you close the issue deliberately ? |
…ements and features.
I am testing the new feature now and will submit a pull request. But Yes.
…On Tue, Sep 8, 2020, 8:28 PM Daniel Lemire ***@***.***> wrote:
@guymolinari <https://github.com/guymolinari> Did you close the issue
deliberately ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#260 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADZZAUWVCQUAHJM4KXF3SITSE3DZVANCNFSM4PE4KLBQ>
.
|
@guymolinari My proposal is to reopen the issue and close it when the PR will have been merged. Less confusing that way. |
Sounds good. Didn't know the process. My apologies.
…On Wed, Sep 9, 2020, 3:46 PM Daniel Lemire ***@***.***> wrote:
@guymolinari <https://github.com/guymolinari> My proposal is to reopen
the issue and close it when the PR will have been merged. Less confusing
that way.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#260 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADZZAUSNHOQUA7P6NVBH23LSE7LSFANCNFSM4PE4KLBQ>
.
|
Do not apologize. I have reopened the issue. When creating a pull request, add "Fixes #260" as part of your description. This will link the issue with the PR. You can also link the issue explicitly by entering it the addresses at the right locations in the web pages. |
Cool. I have written unit test coverage of the new features. Just wanted
to do some more application testing for good measure.
…On Wed, Sep 9, 2020, 8:23 PM Daniel Lemire ***@***.***> wrote:
Sounds good. Didn't know the process. My apologies.
Do not apologize.
I have reopened the issue.
When creating a pull request, add "Fixes #260
<#260>" as part of your
description. This will link the issue with the PR. You can also link the
issue explicitly by entering it the addresses at the right locations in the
web pages.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#260 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADZZAUQT6ZCYKMLNI6FGDALSFAMAVANCNFSM4PE4KLBQ>
.
|
Implement issue #260 BSI Add() method and other enhancements
Issue resolved by @guymolinari |
@guymolinari For BSI, the function transpose can be used for vectorization. I found the fucntion use a RoraingBitmap to return. It make me confused. Could u tell me how can we use it to process groupBy computing? I think the return value RoraingBitmap discard the original Info. |
It is designed to transpose values in the BSI and convert those values to
columnIDs in a Bitmap.
Cheers,
Guy
…On Fri, Nov 27, 2020 at 12:37 AM hhl ***@***.***> wrote:
@guymolinari <https://github.com/guymolinari> For BSI, the function
transpose can be used for vectorization. I found the fucntion use a
RoraingBitmap to return. It make me confused. Could u tell me how can we
use it to process groupBy computing? I think the return value RoraingBitmap
discard the original Info.
Thanks a lot!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#260 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADZZAUV7CGJWALWJS77D43TSR5QMZANCNFSM4PE4KLBQ>
.
|
@guymolinari If you don't mind me asking,could you give me an example of how to "transpose values in the BSI and convert those values to columnIDs in a Bitmap". That would be very useful for me! |
The best way to understand an open source project (at least one that is
properly run), is to look at the test cases.
…On Sat, Nov 28, 2020 at 9:06 PM hhl ***@***.***> wrote:
@guymolinari <https://github.com/guymolinari> If you don't mind me
asking,could you give me an example of how to "transpose values in the BSI
and convert those values to columnIDs in a Bitmap". That's very usefule for
me!
Thanks a lot!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#260 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADZZAUSFBMVR5R3ZO3KYXKTSSHJGXANCNFSM4PE4KLBQ>
.
|
@guymolinari I had read and run all your code .... Thanks anyway! |
Bit Slice Indexes basically provide a map[int]int with some very fast operations. One operation that can be implemented against BSIs is addition, which basically sums up the values by key for two BSI's. This can be entirely accomplished using bitwise arithmetic. The primary functions you need are addToValues(*Bitmap) and addDigit(*Bitmap, int). pseudo-code for the inplace versions is something like
The text was updated successfully, but these errors were encountered: