-
Notifications
You must be signed in to change notification settings - Fork 200
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
Unit tests fail #30
Comments
Thanks! Ok, there are a few different test failures there. I don't have a plan for how to fix them yet, but I'll go through the probable cause of each one:
Summary:
|
Hi. It looks like I was using a |
commited change f3f8829 , which skips kernel 2 in testforward.compare_1_n_biased_pad, if |
Ah, it looks like the errorcode -55 error is actually a bug: missing a guard, and so the kernel crashes. Thank-you for pointing this error out! Committed e52d1a2 , which adds a guard to one of the backprop kernels. I think the test will likely still fail, but the error message might be a bit more compact now. |
I think that covers all the test failures you highlighted above. Can you pull down the updates, and retry please? |
The -1984MB issue is fixed. Code -55 still shows though, not sure if that's an issue. |
Hi pzawal, I'm not seeing an error -55 test failure any more. I'm seeing the following failures:
I will look at making the tests skip these two kernels, for these specific geometries, on AMD. I dont think these test failures are indicating any fundamental problem with DeepCL, on your hardware/os configuration. At runtime, these kernels would simply be skipped, and other kernels used instead. |
Ok, I've modified the tests to modify the geometry slightly for the kernels that are failing. I think it's good for the tests to run on these kernels anyway, to check correctness, though we cant test on incompatible geometries, hence modify the geometries slightly. Commit 7634e54 |
There were no unit test failures due to code -55, I meant that it still shows in output in some subtests. I guess it's all ok then :) thanks |
No, you're right actually, let's fix those. Committed 719e82b , which adds some guards, and hopefully removes the remaining error -55 s, from the output. Can you pull this down, and check if there are still any error -55s in the output? |
They disappeared, all nice now. Btw, you should update your clBLAS fork to include |
Cool :-)
Ah, good info. Thanks! :-) |
Hi, I ran unit tests and ran into some errors.
Compiled with Visual C++ 2015 x64 on Windows 7 with Radeon 7970.
Different errors:
"unknown file: error: C++ exception with description "memallocsize too small to use this kernel on this device. Need: 0MB, but only have: -1984MB max alloc size" thrown in the test body.
[ FAILED ] testforward.compare_1_n_biased_nopad (2230 ms)"
"error: Expected: (0.1f) >= (loss), actual: 0.1 vs 2.72727
clblas teardown
[ FAILED ] testsimpleconvolvenet.imagesize_5_3_2layers_filtersize_3_3_biased_n18 (11310 ms)"
"ForwardAuto: kernel 5: this instance cant be used: For ForwardFc, filtersize and inputimagesize must be identical"
"Something went wrong, code -55" thrown in the test body.
[ FAILED ] testbackward.compare_1_n_kgsgo_32c5 (843 ms)"
"ForwardAuto: kernel 2: this instance cant be used: cannot use forward2, since outputimagesize * outputimagesize > maxworkgroupsize"
Full log http://pastebin.com/v9d7ZMFu
Gpuinfo output:
platform index: 0:
platform id: 000007FEDD2AF180
platform vendor: Advanced Micro Devices, Inc.
platform name: AMD Accelerated Parallel Processing
platform num devices: 2
device index: 0
device id: 0000000000379A20
device type: 4
global memory size: 3072MB
local memory size: 32KB
global cache size: 16KB
global cacheline size: 64
max memory alloc size: 2112MB
max compute units: 32
max workgroup size: 256
max workitem dimensions: 3
max workitem sizes: 256 256 256
device name: Tahiti
opencl c version: OpenCL C 1.2
opencl device version: OpenCL 1.2 AMD-APP (1800.8)
frequency MHz: 925
The text was updated successfully, but these errors were encountered: