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

Test failed because file missing #2

Open
BackMountainDevil opened this issue Nov 23, 2023 · 3 comments · Fixed by #7
Open

Test failed because file missing #2

BackMountainDevil opened this issue Nov 23, 2023 · 3 comments · Fixed by #7

Comments

@BackMountainDevil
Copy link
Contributor

file: crates/gmw/tests/boolean_aby2.rs

test code: fn eval_sample_lut()

run failed reason: file "test_resources/lut_circuits/lfa32_4.lut" missing. You can see only two files here named Sample LUT file.lut and minimal.lut. No lfa32_4.lut

detail:

warning: variable does not need to be mutable
   --> /home/mifen/Documents/code/remoc/remoc/src/rch/mpsc/receiver.rs:284:13
    |
284 |         let mut inner = self.inner.as_mut().unwrap();
    |             ----^^^^^
    |             |
    |             help: remove this `mut`
    |
    = note: `#[warn(unused_mut)]` on by default

warning: `remoc` (lib) generated 1 warning (run `cargo fix --lib -p remoc` to apply 1 suggestion)
warning: function `get_reg_noise_weight` is never used
   --> crates/zappot/src/silent_ot/mod.rs:683:4
    |
683 | fn get_reg_noise_weight(min_dist_ratio: f64, sec_param: usize) -> u64 {
    |    ^^^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(dead_code)]` on by default

warning: `zappot` (lib) generated 1 warning
    Finished test [unoptimized + debuginfo] target(s) in 0.26s
warning: the following packages contain code that will be rejected by a future version of Rust: traitobject v0.1.0
note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 1`
     Running tests/boolean_aby2.rs (target/debug/deps/boolean_aby2-3a7bfc06e825fa6f)

running 1 test
Error: Unable to load lut circuit file

Caused by:
    0: Unable to read LUT file
    1: No such file or directory (os error 2)
test eval_sample_lut ... FAILED

failures:

failures:
    eval_sample_lut

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

error: test failed, to rerun pass `-p gmw --test boolean_aby2`. 
 *  The terminal process "cargo 'test', '--package', 'gmw', '--test', 'boolean_aby2', '--', 'eval_sample_lut', '--exact', '--nocapture'" terminated with exit code: 101. 
 *  Terminal will be reused by tasks, press any key to close it.
This was referenced Dec 12, 2023
@robinhundt
Copy link
Collaborator

Thanks for the issue! It seems I broke a couple of things when preparing the anonymous submission repo, which this is based on, oops.

@BackMountainDevil
Copy link
Contributor Author

it seems that you upload file named lfa32.lut, not the lfa32_4.lut. so is it a typo in code?

robinhundt added a commit that referenced this issue Dec 13, 2023
@robinhundt
Copy link
Collaborator

Oh, when I fixed the path in the beginning of the test case, I forgot to update the path in the end. I've fixed the paths (now both pointing at the minimal.lut) in the test.
However, the test case is still failing. I suspect that currently our implementation of FLUTE is not correct in some cases. With that test case I was trying to debug the issue, but got sidetracked by other things. I've changed the circuit to the minimal LUT file to make debugging easier.

In the coming weeks I hope to update this repository to use the updated version of our new framework and in the process clean up the FLUTE protocol implementation. Hopefully, I'll also find and correct the bugs probably present in the current version.

I'll keep this issue open as a reminder :)

@robinhundt robinhundt reopened this Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants