-
Notifications
You must be signed in to change notification settings - Fork 113
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
the array index out of range #14
Comments
It should work fine with any resolution. We have tested it on projectors with 2 different resolutions. Also if it is possible, could you send me the data you used, I will take a look when I get a chance. Have a great day! |
I`m already use CPU version. |
I also get the same error. my configuration is widows 10, visual studio, cpu version, data is alexander. |
Hey there! |
This problem maybe a bug for Convert bitarray to gray code. In header file "DynamicBitset.h" , function "glm::uvec2 to_uint_gray ()", this code use a static bit convert like this
but in fact , it is deside by you projector resolution. in my situation, my projector resolution is 912x1120, so is like this:
for reference only, not guaranteed to be accurate |
make make[2]: *** [src/lib/ReconstructorCUDA/CMakeFiles/sls_gpu.dir/build.make:79: src/lib/ReconstructorCUDA/CMakeFiles/sls_gpu.dir/sls_gpu_generated_ReconstructorCUDA.cu.o] Error 1 how to resolve this issue? |
I have study struct light recently, and useing this code, it can generate point cloud data with you given test data.but when I use myself data, the code can not work and it will raise in this code
bkts[vec2Idx.x * projHeight + vec2Idx.y].push_back(getRay(i));
I find that this calculate will cause array index of bkts out of range and code running exception.
On my occasion, I use projector resolution 912x1140, and the camera resolution is 1280x1024, and camera calibration is well done.why this problem occur? the code just support projector resolution 768x1024?
The text was updated successfully, but these errors were encountered: