Edge detection error message

1 次查看（过去 30 天）
Ba Ba Black Sheep!
Ba Ba Black Sheep! 2017-2-12
评论： Walter Roberson 2017-8-16
Why isn't the following code working?
>> str=sprintf('Scratch1.jpg');
>> img_ORG = imread(str);
>> fsc = edge(img_ORG, 'Canny');
Error using edge
Expected input number 1, I, to be two-dimensional.
Error in edge>parse_inputs (line 449)
validateattributes(I,{'numeric','logical'},{'real','nonsparse','2d'},mfilename,'I',1);
Error in edge (line 189)
[a,method,thresh,sigma,thinning,H,kx,ky] = parse_inputs(varargin{:});

请先登录，再回答此问题。

采纳的回答

Image Analyst
Image Analyst 2017-2-12
It's color, not gray scale like you had expected. Use this code to convert to gray scale:
grayImage = imread(fullFileName);
% Get the dimensions of the image.
% numberOfColorChannels should be = 1 for a gray scale image, and 3 for an RGB color image.
[rows, columns, numberOfColorChannels] = size(grayImage);
if numberOfColorChannels > 1
% It's not really gray scale like we expected - it's color.
% Use weighted sum of ALL channels to create a gray scale image.
grayImage = rgb2gray(grayImage);
% ALTERNATE METHOD: Convert it to gray scale by taking only the green channel,
% which in a typical snapshot will be the least noisy channel.
% grayImage = grayImage(:, :, 2); % Take green channel.
end
  3 个评论
显示 1更早的评论
Image Analyst
Image Analyst 2017-2-12
The problem is that your code throws an error if img_ORG is already a gray scale image. My code only calls rgb2gray() if img_ORG is an RGB image. See how I check numberOfColorChannels and only call it if numberOfColorChannels is 3? Therefore no error will ever be thrown with my code, whereas it could with your code if you ever read in a gray scale image. So my code is more robust than your code. Writing code for almost 40 years. mostly for one of the world's largest companies, has taught me that it's better to be safe and robust, even if you think you don't need to be.
Secondly your use of cryptic, non-descriptive variable names like img_ORG and fsc, and the lack of comments, make the code you've posted unmaintainable should someone ever need to inherit your code - this is intolerable at a large company with mission critical software especially if people's lives or safety are at risk.
Walter Roberson
Walter Roberson 2017-8-16
mohd abdul-raouf comments to Image Analyst:
I agree , Robustness is must requirement for a Good Programmer

请先登录，再进行评论。

更多回答（0 个）

类别

在 Help Center 和 File Exchange 中查找有关 Detection and Tracking 的更多信息

标签

产品

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!

Translated by