zhoubolei / CAM

Class Activation Mapping
http://cnnlocalization.csail.mit.edu/
MIT License
1.84k stars 466 forks source link

Maybe a mistake in mergeTenCrop.m #14

Closed mumuhe closed 7 years ago

mumuhe commented 7 years ago

Thanks for your providing of the source code, which helps me a lot.

I found a mistake in mergeTenCrop.m.

In line 25, 26

alignImgSet(i:i+cropSize-1, j:j+cropSize-1,:,curr) = curCrop1; alignImgSet(i:i+cropSize-1, j:j+cropSize-1,:, curr+5) = curCrop2;

They maybe modified into

alignImgSet(j:j+cropSize-1, i:i+cropSize-1,:,curr) = curCrop1; alignImgSet(j:j+cropSize-1, i:i+cropSize-1,:, curr+5) = curCrop2;

You can check it whether it is right.

zhoubolei commented 7 years ago

Thanks. Fixed.