# 2023-01-03 Sculpt/Texture/Paint Module Meeting Note: If you're planning to join the meeting, feel free to add talking points to the agenda. ## Attendees - Julien Kaspar - Joe Eagar - Daniel Bystedt ## Agenda ### Needs Commit None :+1: ### High Priority Bugs - [⚓ T103454 Sculpt Color/Mesh/Cloth filter auto-masking crash when occlusion is used (missing StrokeCache)](https://developer.blender.org/T103454) Joe will fix it. ### In Review - [⚙ D16414 Sculpt: Stroke Texture Mapping](https://developer.blender.org/D16414) Seems unrealistic to get into 3.5 before tomorrow. We'll give this more time for polish and review. Opinions on Hans notes? Joe will talk it through with Hans and implement a better way of handling curves. The current approach could also eventually be used for strokes in general to have improved stroke behavior in scene spacing. With the new curve based strokes, it coiuld be explored to have multiple continuous strokes semelessly blend toegether: stroke > orbit > continue stroke > confirm for once seamless stroke Feedback from testing the patch? Overrall works great. Mostly about fixing bugs. - [⚙ D14900 PBVH Texture Paint Node Splitting](https://developer.blender.org/D14900) Are the inline commetns adressed? Joe will adress them. Then Jeroen can be notified. - [⚙ D16544 Sculpt Mode: Improvements to operators that rely on cursor position](https://developer.blender.org/D16544) Joe missed it. Will check this out. - [⚙ D15298 Support Automasking For Texture Paint](https://developer.blender.org/D15298) Joe, any updates? - [⚙ D16511 Sculpt Mode: Resolve `Shift R` shortcut conflicts](https://developer.blender.org/D16511) Asked Aaron if the identifiers should be removed or not. - [⚙ D16330 Sculpt: Fix crash with no active brush data-block](https://developer.blender.org/D16330) Joe will comment. Julien will mark as high priority. ### Current Topics #### Target for 3.5? Today is the last day of Bcon1. Roll brush is unlikely to land in master before that. Focus on auto-masking UI improvmeents and behavior fixes and bugs. ### Upcoming To Do's Joe: - Wrap up Roll brush feature - Dyntopo refactor Julien: - Working on Studio - More manula fixes Daniel: - More involved with Hair nodes ### Other Topics #### Inverted Expand and Auto-Create [⚙ D16434 Sculpt Mode: Invert Expand behavior for masking consistency](https://developer.blender.org/D16434) Was committed but [a9cb66b856e8](https://developer.blender.org/rBa9cb66b856e80d0542a9ad3fec0b0fb47d28f805) counteracted the patch. Juliens notes on that second commit: Can you revert that commit and we can discuss this first? I don't know why this was committed. That commit also causes issues because: - Auto-create can easily clear the mask of hidden geometry because it assumes that everything is unmasked. - Auto-create option will fill the entire mesh as masked, but then clears the mask on disconnected mesh islands anyway - Auto-create step adds an additional undo step - The Expand by Normals is not properly inverting when using F I do like the changed "Invert" option for the operator. The issue there is that it will clear the mask on any disconnected mesh island. #### Sculpt Mode Painting [⚓ T97957 Sculpt Mode Painting](https://developer.blender.org/T97957) Simplified description. Question: Should we keep the autoamtic viewport color switching? Current behavior is very intuitive says Daniel. We should build on top of that. Pablos patch for the canvas picker in the header. And Jeroens Canvas Picker in experiemental to override shading on active obejct. We can try out both combined with current behavior and then decide if it should be adjusted further. #### Image syncing as an explicit action [⚙ D16811 Texture paint: sync from Node to Image Editor even if active already](https://developer.blender.org/D16811) Currently it's using the last active image texture node, which leads to a lot of frustration. Because selecting nodes is done a lot more often, it causes accidental syncing of any image texture between editors. A way to solve this is to make the syncing between editors a more explicit action. So instead of using active selection, it could use the active image of the material. ![](https://i.imgur.com/2047Dmu.png) For the UI this could be represented with an icon int he corner and toggled either by clicking on it or for example by double clicking anywhere on the node. ![](https://i.imgur.com/z6b4tgS.png) Both DFaniel and Joe approve. #### Incremental Topology Auto-masking for all brushes This would help avoid sculpting on disconnected geoemtry within the brush radius on any brush other than the grab brush. How we handle PBVH tree leaves needs imrpvements. They need to be subdivided more. Turning the mesh into a hexahedral mesh (essentially treated as a volume). But in theory it's possible. Kelvenlets elastic brushes could benefit from taht too. General performance and geodesic expand performance would also heavily benefit from that. --- Joe can you cehck out https://developer.blender.org/T102889 again.