Skip to content

Никоноров И.Д. ЛР№2 Линейная фильтрация изображений (горизонтальное разбиение). Ядро Гаусса 3x3. OpenMP#152

Open
CosmonautComrad wants to merge 12 commits intovalentina-kustikova:masterfrom
CosmonautComrad:OpenMP
Open

Никоноров И.Д. ЛР№2 Линейная фильтрация изображений (горизонтальное разбиение). Ядро Гаусса 3x3. OpenMP#152
CosmonautComrad wants to merge 12 commits intovalentina-kustikova:masterfrom
CosmonautComrad:OpenMP

Conversation

@CosmonautComrad
Copy link
Copy Markdown

No description provided.

Comment thread Gaussian_filter/OpenMP/OpenMP.cpp Outdated
void Gauss(Mat input, Mat output, double** kernel, int kern_size)
{
int radius = int(kern_size / 2);
#pragma omp parallel for num_threads(6)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Количество потоков должно задаваться через аргументы командной строки и устанавливаться внутри функции.

Comment thread Gaussian_filter/OpenMP/OpenMP.cpp Outdated

int main() {
Mat image;
image = imread("55fc619dea60e.jpg", IMREAD_COLOR);
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Уже писала в комментариях к последовательной версии, что все названия файлов надо передавать через аргументы командной строки.

Comment thread Gaussian_filter/OpenMP/OpenMP.cpp Outdated
int main() {
Mat image;
image = imread("55fc619dea60e.jpg", IMREAD_COLOR);
if (!image.data)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это старый интерфейс, надо использовать image.empty()

Comment thread Gaussian_filter/OpenMP/OpenMP.cpp Outdated
Gauss(gray_image, my_result, Kernel, k);
auto end = std::chrono::steady_clock::now();
auto elapsed_ms = std::chrono::duration_cast<std::chrono::milliseconds>(end - begin);
cout << "The time is: " << elapsed_ms.count() << " ms\n";
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Необходимо добавить сравнение результатов фильтрации, полученных с использованием последовательной и параллельной реализации.

@valentina-kustikova
Copy link
Copy Markdown
Owner

@CosmonautComrad, надо внести исправления, которые были описаны в комментариях к последовательной реализации.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants