Page MenuHome

Shading: Extend Musgrave node to other dimensions.
ClosedPublic

Authored by Omar Ahmad (OmarSquircleArt) on Fri, Aug 23, 9:54 AM.

Details

Summary

This patch extends Musgrave noise to operate in 1D, 2D, 3D, and 4D
space. The Color output was also removed because it was identical
to the Fac output.

This patch was created from the noise-node branch, and it depends on D5560.

Diff Detail

Repository
rB Blender

Event Timeline

  • Update versioning code.

This patch was supposed to depend on D5560, but apparently the commits from D5560 are now in this patch as well. So, I am not entirely sure what to do now.

Omar Ahmad (OmarSquircleArt) planned changes to this revision.Wed, Sep 4, 7:10 PM

Merge master and update patch.

Omar Ahmad (OmarSquircleArt) planned changes to this revision.Wed, Sep 4, 9:17 PM

The musgrave code needs to refactored to avoid writing to inputs and to follow best practices, in particular, we want the declarations to be closer to the initializations and limit the scope of variables.

  • Refactor musgrave code.
Brecht Van Lommel (brecht) requested changes to this revision.Mon, Sep 9, 4:09 PM

Looks good overall.

source/blender/blenloader/intern/versioning_cycles.c
796

Check if link->fromnode && here, this can be NULL in some cases and causes automated tests to crash.

This revision now requires changes to proceed.Mon, Sep 9, 4:09 PM
  • Update patch and merge master.
Omar Ahmad (OmarSquircleArt) marked an inline comment as done.Mon, Sep 9, 7:57 PM
This revision is now accepted and ready to land.Mon, Sep 9, 8:04 PM