Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blending using the GPU #255

Merged
merged 11 commits into from Apr 13, 2023
Merged

Blending using the GPU #255

merged 11 commits into from Apr 13, 2023

Conversation

royshil
Copy link
Collaborator

@royshil royshil commented Apr 12, 2023

Instead of our OpenCV (CPU) based image blending - use the GPU and the HLSL shaders to do the application of the mask to the image. It's faster and relieves the CPU from rendering duties,

@royshil royshil requested a review from umireon April 12, 2023 18:11
@royshil royshil self-assigned this Apr 12, 2023
Comment on lines 466 to 471

# ignore .DS_Store files
if(_DATA_FILE MATCHES ".DS_Store")
continue()
endif()

Copy link
Member

Choose a reason for hiding this comment

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

We don't need this - All the releases from CI and CI won't generate .DS_Store.
It would be great to keep this file identical to template's one.

src/background-filter.cpp Outdated Show resolved Hide resolved
@umireon
Copy link
Member

umireon commented Apr 12, 2023

The contour is not smooth.

スクリーンショット 2023-04-13 7 43 20

@umireon
Copy link
Member

umireon commented Apr 12, 2023

スクリーンショット 2023-04-13 8 44 23 1

The edge is flickering when the blur is turned on.

royshil and others added 5 commits April 12, 2023 20:29
Co-authored-by: Kaito Udagawa <umireon@gmail.com>
Co-authored-by: Kaito Udagawa <umireon@gmail.com>
@royshil royshil requested a review from umireon April 13, 2023 03:20
data/effects/mask_alpha_filter.effect Outdated Show resolved Hide resolved
src/background-filter.cpp Outdated Show resolved Hide resolved
data/effects/mask_alpha_filter.effect Outdated Show resolved Hide resolved
Co-authored-by: Kaito Udagawa <umireon@gmail.com>
Copy link
Member

@umireon umireon left a comment

Choose a reason for hiding this comment

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

LGTM!

@royshil royshil merged commit 198d492 into main Apr 13, 2023
7 checks passed
@royshil royshil deleted the roy.blending_effect_filter branch April 27, 2023 14:26
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.

None yet

2 participants