-
Notifications
You must be signed in to change notification settings - Fork 32
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
Psi_Rs_multiThrd_test failed #75
Comments
I also have a question about the max size of RsOpprf: # receiver
oc::span<const block> keyspan(key.data() + offset, binSize);
oc::MatrixView<block> outputspan(outputs.data() + offset * outputs.cols(), binSize, outputs.cols());
auto p = receiver.receive(binSize * context.max_in_bin, keyspan, outputspan, prng, context.threads, chl);
# sender
oc::span<const block> keyspan(key.data() + offset, binSize * context.max_in_bin);
oc::MatrixView<block> valuespan(value.data() + offset * value.cols(), binSize * context.max_in_bin, value.cols());
auto p = sender.send(binSize, keyspan, valuespan, prng, context.threads, chl); In this situation, the size of COPROTO_ASSERT(data.size() < std::numeric_limits<u32>::max()); where So my question is, how many additional information will be attached to the payloads? |
You could modify the code to use u64. I believe the size is I'll try to look at the multi threading at some point. Might not be soon. |
Thank you for your reply. |
OK. So apart from the multi thread thing your are good? Also, you likely don't want to do a psi with more than a million items. If you do have that many, you can shard your dataset and process each shard independently. |
Yes, and this assertion `B2 <= B' will aslo failed in RsOPPRF with some specific input size. |
what sizes |
Well, here is some tests I made:
Where |
This is the gdb backtrace:
And this assertion is also appeared in other test, like test 47:
The text was updated successfully, but these errors were encountered: