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

Pixel Blur #1777

Merged
merged 8 commits into from
May 16, 2023
Merged

Pixel Blur #1777

merged 8 commits into from
May 16, 2023

Conversation

harisreedhar
Copy link
Contributor

chaiNNer-pixel_blur-2023-5-4_21-15

@RunDevelopment
Copy link
Member

Maybe I'm misreading the code, but won't the dimensions of the output image be a multiple of the given block size? E.g. with an input image of width 100 and a block size of 16, won't the width of the output image be 96?

added padding
fix greyscale image
@harisreedhar
Copy link
Contributor Author

Maybe I'm misreading the code, but won't the dimensions of the output image be a multiple of the given block size? E.g. with an input image of width 100 and a block size of 16, won't the width of the output image be 96?

You are right. I have added padding.

@RunDevelopment
Copy link
Member

I have no idea, why the CI fails. Pyright suddenly says that it can't find numpy.bool8 anymore in totally unrelated files. I thought that it might be that a new numpy version 1.24.3 broke something, but we already have other successful CI runs with 1.24.3, so it's not that.


About the node itself: I think the implementation looks fine now, but I'm not sure about the name and category. Is this a blur? I'm not sure. For reference, Paint.net has its Pixelate effect under the Distortion category. Why did you call this node Pixel Blur? (I'm not saying it's bad, I just want to know.)

@harisreedhar
Copy link
Contributor Author

harisreedhar commented May 6, 2023

Blur literally means making an image unclear and less distinguishable. This node does the same thing, so I put it in the blur category. Different programs use different names and categories. In Gimp, it is under the blur category called "pixelize," and in Photoshop, it is called "mosaic."

@joeyballentine
Copy link
Member

in Photoshop, it is called "mosaic."

It's also not under the blur category, it's in "Pixelate". Tbh I'm not sure what to do here. Pixelation doesn't scream "blur" to me but I can see how it fits in with the idea of obscuring information.

@harisreedhar
Copy link
Contributor Author

Do you have any suggestion?

@RunDevelopment
Copy link
Member

Maybe we could call the node Pixelate and move it into Image Filters > Misc? We don't have a "Distortion" category yet, so misc seems fitting. I think that the node name should have "pixel" in it somewhere (for searchability), so I'm against calling it "Mosaic."

Copy link
Member

@RunDevelopment RunDevelopment left a comment

Choose a reason for hiding this comment

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

Thank you @harisreedhar!

@joeyballentine joeyballentine merged commit f5cb007 into chaiNNer-org:main May 16, 2023
8 checks passed
@harisreedhar harisreedhar deleted the pixel-blur branch May 17, 2023 04:55
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

3 participants