Matrix dimensions must agree -- error with old versions of Matlab

[mahmoudnafifi]

If you faced the following error in Matlab:

Please replace the lines 59 and 60 in WBmodel class and the lines 59 and 60 in WBmodel_GPU class with the following lines:
diff_u=abs(repmat(Iu,[1,size(A,2)])-repmat(A,[size(Iu,1),1]));
diff_v=abs(repmat(Iv,[1,size(A,2)])-repmat(A,[size(Iv,1),1]));

Please replace Line 69 in WBmodel.m and Line 69 in WBmodel_GPU.m with the following one:
hist(:,:,i)=(repmat(Iy, [1, size(diff_u,2)]) .* double(diff_u))'*double(diff_v);

Lastly, please replace line 92 in WBmodel.m and line 92 in WBmodel_GPU.m with the following code:
mf = sum(repmat(weightsH,[1, size(obj.mappingFuncs,2)]) .* obj.mappingFuncs(idH,:),1);

The old Matlab versions don't offer this broadcasting operation. In the new versions of Matlab, this repmat operation is automatically applied when the first/second operand is a single row vector and the second/first operand is a single column.

[wtzhu13]

Hello, I have faced this error. But I can not run the demo.m because the model loaded does not replace. So how can I get the new model after replacing the line 59 and 60? Thank you!

[mahmoudnafifi]

What do you mean by "the model loaded does not replace"? Could you please explain more what is the error message you got in Matlab?

[wtzhu13]

I mean that I just replace the code, but the model loaded in the demo.h is still the old one.

[mahmoudnafifi]

Thanks for clarifying. No need to replace the model .. just replace the code and it should work.

[wtzhu13]

really?Actually, I still can not run the demo.h. The model which is loaded in the demo in models folder is still the old one, I think maybe I should rebuild the model, but I can not find where to rebuild it. The version of my matlab is 2014a.

[mahmoudnafifi]

It seems that the problem is not in the model .. probably, it is related to the image itself. Do you mind to print out the size of Iy, diff_u and diff_v? Or if you can send me figure3.jpg. Thanks!

[wtzhu13]

The size of the 3 variables are printed in the picture respectively, and the figure3.jpg is still the one which you pushed to the github, I do not repalce it.

[mahmoudnafifi]

Please replace Line 69 in WBmodel.m and Line 69 in WBmodel_GPU.m with the following one:

hist(:,:,i)=(repmat(Iy, [1, size(diff_u,2)]) .* double(diff_u))'*double(diff_v); % compute the histogram

[wtzhu13]

There are still some bugs in the code, but that's ok, I'll read your code and the paper carefully first so I don't waste your valuable time. I was going to see the phenomenon and then read the code and the paper carefully, but maybe I will deal with these bugs by myself after reading the paper and code. Thank you anyway！

[mahmoudnafifi]

Could you please refer to the part that you think it is a bug? What is the error message you got?

[wtzhu13]

After replacing Line 69 in WBmodel.m and Line 69 in WBmodel_GPU.m, there are still some errors shown above.

[mahmoudnafifi]

thanks for sharing. I guess the problem is about broadcasting .. let me have a look and try to make things work for you.

[wtzhu13]

OK，thank you for your guidance, I will also read the code carefully after my work.

[mahmoudnafifi]

Replace line 92 in WBmodel.m and line 92 in WBmodel_GPU.m with the following code:
mf = sum(repmat(weightsH,[1, size(obj.mappingFuncs,2)]) .* obj.mappingFuncs(idH,:),1);

Please let me know if you still have a problem with your Matlab version.

[wtzhu13]

The code is running Ok, thanks for your guidance!