-
Notifications
You must be signed in to change notification settings - Fork 752
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
Basic/fpga_tests/fpga_pipes.cpp fails in post-commit #16693
Comments
This fpga_pipe.cpp test has been passing because it was using the wrong binary operation to fold together test results. I fixed this in this and some other tests recently. My PR passed the CI because, apparently, we don't have exercise the FPGA accelerator (or emulator) there. fpga_pipe.cpp tests 12 different combinations, the last two ( |
to address post-commit failure here: #16693 This fpga_pipe.cpp test has been passing because it was using the wrong binary operation to fold together test results. I fixed this in this and some other tests recently. My PR passed the CI because, apparently, we don't have exercise the FPGA accelerator (or emulator) there. fpga_pipe.cpp tests 12 different combinations, the last two ( `test_array_th_nb_pipe` and `test_array_th_bl_pipe` ) are failing. Opening a JIRA.
@KornevNikita - The PR that disables the two failing cases has been merged and a JIRA opened for them. This test should be passing in post-commit testing now. Is it OK to close this GitHub issue? |
@cperkinsintel I don't mind, but I guess this should contain jira number rather than link to this issue. |
GH issues are preferrable as everybody has access to them. |
Agree, but it would be strange to link the closed issue. I assume it normally means the root cause was fixed. I guess we should have a jira number with some comment at least here, or just let it be opened. |
Describe the bug
e2e-lin (Intel GEN12 Graphics with Level Zero, ["Linux", "gen12"], level_zero:gpu;opencl:fpga, true) / Intel GEN12 Graphics with Level Zero
To reproduce
Environment
clang++ --version
]sycl-ls --verbose
]Additional context
No response
The text was updated successfully, but these errors were encountered: