質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Visual C++

Microsoft Visual C++はWindowsのCとC++の統合開発環境(IDE)であり、コンパイラやデバッガを含んでいます。

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

OpenCV

OpenCV(オープンソースコンピュータービジョン)は、1999年にインテルが開発・公開したオープンソースのコンピュータビジョン向けのクロスプラットフォームライブラリです。

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

2回答

1997閲覧

openCVラスタースキャン

snakaz

総合スコア1

Visual C++

Microsoft Visual C++はWindowsのCとC++の統合開発環境(IDE)であり、コンパイラやデバッガを含んでいます。

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

OpenCV

OpenCV(オープンソースコンピュータービジョン)は、1999年にインテルが開発・公開したオープンソースのコンピュータビジョン向けのクロスプラットフォームライブラリです。

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

0クリップ

投稿2021/10/06 08:54

前提・実現したいこと

openCV(C++)を用いて、二枚の画像の画素値をそれぞれ比較し、小さいほうを取り出して一つの画像を作ろうとしています。

発生している問題・エラーメッセージ

現在、プログラミング自体は動いているのですが、横軸の画素値比較がうまく機能していない状態です。
利用している画像のpixelはともに(320×236)であることは確認済みです。
イメージ説明
イメージ説明
イメージ説明
画像を出力すると以上のように途中でスキャンが止まってしまい困っています。

ソースコードのfor文内の

high
width

input1.rows
input1.cols

などに変換してみても、直りませんでした。
しかし、数値を直接入力すると(236より大きくすると)スキャンが進みます。

どのような原因で、このようなことが起きているのか一人では解明できなかったので、ご教授お願い致します。

該当のソースコード

C++

1#include <opencv2/opencv.hpp> 2#include <iostream> 3#include <algorithm> //min関数を使うために必要 4 5using namespace std; 6using namespace cv; 7 8int main() { 9 10 const int k = 2; //画像枚数 11 12 const int width = 320; //画像の幅 13 const int high = 236; //画像の高さ 14 15 16 17 //const int nop = width * high; //number of pixels 18 19 20 //一枚目の入力画像// 21 22 23 Mat input1 = imread("cat.jpg"); 24 if(!input1.data) { 25 cout << "cannot open input file" << endl; 26 return 1; 27 } 28 29 30 //二枚目の入力画像// 31 32 33 Mat input2 = imread("sample.jpg"); 34 if (!input2.data) { 35 cout << "cannot open input file" << endl; 36 return 1; 37 } 38 39 //int input[k][nop] = {}; //初期化 40 41 42 Mat output = input1.clone(); //格納する枠組み// 43 44 45 for (int y = 0; y < high; y++) { 46 for (int x = 0; x < width; x++) { 47 48 int pixel[k] = { {input1.at<unsigned char>(y, x)},{input2.at<unsigned char>(y, x)} }; 49 50 int min = *min_element(pixel ,pixel + k); 51 52 output.at<unsigned char>(y, x) = min; 53 } 54 } 55 56 57 imwrite("output.bmp", output); 58 59 return 0; 60 61}

試したこと

ソースコードのfor文内の

high
width

input1.rows
input1.cols

などに変換

補足情報(FW/ツールのバージョンなど)

openCV4.5.8

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

cv::min 関数ではだめなんでしょうか?

投稿2021/10/06 10:03

yuki23

総合スコア1448

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

snakaz

2021/10/07 02:14

返信ありがとうございます。 min関数は使っているんですが、なぜかうまくいかなくて困ってました… cv::はnamespaceを用いて省略してあります。
yuki23

2021/10/07 05:19

質問文に書いてあるのはcv::minではなくstd::min_elementですよね? このような無駄なループを書く必要はないので調べてみてください
fana

2021/10/08 02:42

> 実現したいこと である 「2つの画像データから,画素毎に値が小さい側を採用した新しい画像を作る」処理 をやってくれる関数が OpenCV にありますよ,っていう 情報提供 ですね. (「どのような原因で、このようなことが起きているのか…」に関する話ではなく.)
guest

0

ベストアンサー

ループ箇所は 1ch データを期待した記述になっているように見えますが,
対して,
画像データが 3Ch になっているのではないでしょうか.

(本来は imread でグレースケール化するつもりだった?)

投稿2021/10/06 10:01

fana

総合スコア11996

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

snakaz

2021/10/07 02:13

返信ありがとうございます。 画像データを前もってグレースケール化をしたつもりでいたのですが、うまくできていなかったようです。 fanaさんのおかげでうまく動作しました。 本当にありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問