Skip to content

Conversation

ArtemSkrebkov
Copy link

Поменял параметр sigma на radius, поэтому изменил регрессионный тест.
Попытался перейти на работу с целыми числами в случае линейной свертки(opacity) всё просто, хорошо и гораздо быстрее.
В случае overlay всё хуже, в исходнике три варианта реализации этой функции:
overlay - по-обычному с float матрицами
overlay2 -попытка перейти на операции над матрицами из opencv, вышло не очень гораздо дольше чем цикл по всем пикселам.
overlay3 - цикл по всей картинке, операции с целыми числами, правда тест не проходит, результат работы overlay3 и overlay отличается на некоторых пикселах на единицу. Overlay3 на 5-7 мс быстрее overlay.

В общем хотелось бы overay3 поправить чтобы результаты от overlay не отличались и как-нибудь ускорить размытие Гаусса...

@ArtemSkrebkov
Copy link
Author

Добавил размытие с помощью boxFilter, он быстрее, и эффект получше смотрится, добавил в параметры выбор размытия, это просто пока для сравнения, потом оставлю только boxFilter. Но всё равно его время 150-200 мс для fullhd изображения

@ArtemSkrebkov
Copy link
Author

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

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 7103f67 on stragger:glow_impl into * on UNN-VMK-Software:master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 14122b4 on stragger:glow_impl into * on UNN-VMK-Software:master*.

@ArtemSkrebkov
Copy link
Author

Оставил размытие только с помощью box filter, в прошлый раз я что-то напутал с замерами похоже, время работы этого фильтра 30мс.
Итоговое время - 80мс, замеры добавил в таблицу.
Поправил регрессионный тест, т.к. изменил способ размытия.

Copy link
Contributor

Choose a reason for hiding this comment

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

Сохранение в файл здесь ни к чему.

@druzhkov-paul
Copy link
Contributor

Табуляции надо заменить на пробелы.

druzhkov-paul added a commit that referenced this pull request Jul 7, 2014
@druzhkov-paul druzhkov-paul merged commit c4fd692 into itlab-vision:master Jul 7, 2014
@druzhkov-paul
Copy link
Contributor

Если будут еще идеи, как можно ускорить работу данного фильтра, предлагайте, пробуйте, делайте pull request'ы.

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.

3 participants