Mon, Jan 16
@Michael P. (forest-house) You forgot to set parameters of the Dilate/Erode Node. Setting the Distance -1 removes the artifact.
Sun, Jan 15
@Shinsuke Irie (irie) these are greate improvements to the node.
Good news! This issue is already fixed! If you want the fix head over to https://builder.blender.org/download/ and grab a nightly, otherwise this will be fix in the next official version!
Fri, Jan 13
- Forgot to update extern/smaa_areatex/README.blender
Also update the test build for Windows 64bit.
- Improve render quality
- Fix several render issues
- Remove unimportant options such as Predicated thresholding
- Remove "Blending Weights" output that is useful only for debugging
- Rename "Depth" to "Value"
- Modify UI texts
- Move smaa_areatex.cpp to extern/smaa_areatex/
- Style cleanup
- Update copyright year
Fri, Jan 6
I tested this new node for some cases, and I'm really impressed.
Dec 20 2016
I can confirm. I will try to fix this myself.
Dec 18 2016
Dec 16 2016
Style and comment fixes, remove own utility functions
Dec 15 2016
Here is a test build for Windows 64bit.
Looks neat! I'd be happy to test it on some vfx type things.
Dec 14 2016
Dec 13 2016
For such patches it's always a good idea to include render tests demonstrating the results. How does this compares to existing AA operation? If it's considered more superior quality, can existing AA operation be switched to this code?
Dec 12 2016
Dec 9 2016
happy to - and already did. :)
Hrm, yeah that's confusing.
Masks are stored in the normalized space (coordinates 0 to 1) which are then mapped to the "canvas". By default, we map them to scene resolution, so when you do alpha-over or things like that in compo you always have same result regardless of render percentage.
Dec 7 2016
Here, the problem is not the mask.
It is the image.
Nov 26 2016
Just curious why this would be the intended behavior? Isn't the render percentage only supposed to modify the final output? By having it affect the way the compositor works, you have to adjust your mask nodes every time you adjust the final render output. This seems cumbersome and unintuitive.
Hmm, this is intended behavior but for example, the mask node has the Size property to change how this works.
Nov 25 2016
if the animated gif won't load, check out the gif here..
Nov 14 2016
maybe duplicate of T49857
Nov 11 2016
Oct 18 2016
Aug 25 2016
Not treating 0.0/1.0 as named constants
reverting to reparate lres & res buffer designation
Aug 24 2016
The patch has been committed at rB262adf7, thanks!
Using proper constant names is surely good, but i'm not sure about lres/res change. It doesn't really make things easier to follow, and even if it's the same block, semantically code in some places easier to follow with more clear separation..
Aug 23 2016
I also coded D2165 while trying to trace this down, but D2165 is definitely not appropriate to merge while close to a release. Should I create a ticket for that patch, or is it fine to leave it out there? (Is there some sort of patch review queue somewhere separate from open tickets?)