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

Slope Blur Update #664

Open
MitchWoodin opened this issue Feb 21, 2024 · 3 comments
Open

Slope Blur Update #664

MitchWoodin opened this issue Feb 21, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@MitchWoodin
Copy link

MitchWoodin commented Feb 21, 2024

Hi,

I've recently been messing around with Material Maker as a substance alternative. I was checking out the slope blur and I think the implementation is at least partly wrong. (At least as far as I'm aware)
At the moment the node is doing a directional blur towards the white part of a greyscale texture. However the way it should be behaving is by warping in the direction and adding those all together. There is a very good video explaining how the slope blur node works in substance here:
https://www.youtube.com/watch?v=H0c1SffwllY&t=308s

I think making this change in Maker would be a good improvement to make. Currently the slope blur node isn't producing the result exactly how I would expect them to when compared to Substance.

Edit:
It seems like the MultiWarp is almost doing the right thing but just in the opposite direction. Is there a way to switch the direction it works? This might be a solution?

EditEdit:
Actually it seems as though the MultiWarp is doing the right thing. I'll just use that node!

@MitchWoodin MitchWoodin added the enhancement New feature or request label Feb 21, 2024
@RodZill4
Copy link
Owner

Material Maker is not a SD clone (I absolutely never used SD), and its slope blur node is not supposed to have the exact same behavior (it's a gaussian blur guided by the slope of its input).
It is probably possible to create a node that behaves the same, but "fixing" that node would just break compatibility, which won't happen (does not mean the new node could not be named "Slope Blur 2" or something).

@MitchWoodin
Copy link
Author

Yep, I think the MultiWarp is actually doing what I'm seeing in Substance!

@RodZill4
Copy link
Owner

Yeah, multiwarp is closer to SD's Slope Blur.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants