# 2022-12-20 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

### High Priority Bugs

- [⚓ Regression: Clay Strips Brush plus an alpha texture produces artefacts.](https://developer.blender.org/T103156)
Probably simple one-line fix. Joe will take a look.

### In Review

Sculpt Mode:
- [⚙ D16414 Sculpt: Stroke Texture Mapping](https://developer.blender.org/D16414)
Needs a shared built for artists to test. Joe created a branch as `temp-sculpt-roll-mapping` at https://builder.blender.org/download/experimental/
- [⚙ D16781 Brush: Swap Texture Slots for Sculpt Brushes.](https://developer.blender.org/D16781)
Jeroen will create a built for download. This should be tested asap. Review questions are at bottom of the task description.
D16758 is a small fix that will follow after.
- [⚙ D16758 Sculpt: Vertex painting color textures](https://developer.blender.org/D16758)
- [⚙ D16434 Sculpt Mode: Invert Expand behavior for masking consistency](https://developer.blender.org/D16434)
- [⚙ D16544 Sculpt Mode: Improvements to operators that rely on cursor position](https://developer.blender.org/D16544)

Texture Paint:
- [⚙ D14900 PBVH Texture Paint Node Splitting](https://developer.blender.org/D14900)
Important patch. Joe can finalize and commit?
- [⚙ D15298 Support Automasking For Texture Paint](https://developer.blender.org/D15298)
Joe can fix built errors and commit?
- [⚙ D16811 Texture paint: sync from Node to Image Editor even if active already](https://developer.blender.org/D16811)
Julien will test after the holidays.

General:
- [⚙ D16791 Fix T101766: Curves sculptmode: orbit around selection / view selected](https://developer.blender.org/D16791)
We can ask Hans if he can help out there.
- [⚙ D16511 Sculpt Mode: Resolve `Shift R` shortcut conflicts](https://developer.blender.org/D16511)
Should `VIEW3D_MT_angle_control` be removed?
Could ask Campbell what is common to do here.
- [⚙ D16330 Sculpt: Fix crash with no active brush data-block](https://developer.blender.org/D16330)
Joe can you leave a comment. He might have already fixed it.
- [⚙ D16241 UI: Add Vertex Group and Color Attributes selector in 3D Viewport](https://developer.blender.org/D16241)
Waiting for Pablo to come back from holidays.

### Current Topics

- https://developer.blender.org/T103341
Joe already made a system to support this but just needs to be exposed in the UI.
- [⚓ T98070 Improved Rake functionality](https://developer.blender.org/T98070)
- [⚓ T101593 Auto-Masking Popover for the Header](https://developer.blender.org/T101593)
Some tasks can be set as Good First Issue for others to take over.
- [⚓ T103346 Dyntopo Refactor](https://developer.blender.org/T103346)
Questions:
Should general tool/operator support be a focus for this task?
Attrbiute API patch already did most of the hard work. So it would be easy to already add support.
Dyntopo refactor will just add needed support for temporary attributes.

Which sculpt-dev features are not targeted to be supported with this refactor?

Re-projection would be a separte feature. Dyntopo refactor is adding the possibility but for the actual support should be worked on for multires and vector dsiaplcement.

There was also a bevel-smooth feature that did not go very far.

#### Manual progress
Main patch is basically done:
[⚙ D16335 Sculpting & Painting: High level changes and new Sculpting Introduction page](https://developer.blender.org/D16335)

Next are individual brush, tool and operator pages.

Joe: Maybe don't mention vertices specifically but talk about "Mesh", "Geometry" or "Surfaces". More contextual.
Only for specific brushes like Draw Face Sets, use "Faces".

### Upcoming To Do's

tba

### Other Topics

#### Good First Issues

While Joe & Jeroen are working on major features, we can document and tag smaller tasks as Good First Issue.
Examples:
- [⚓ T102437 Sculpt Mode: Cavity Inverted consistency](https://developer.blender.org/T102437)
- [⚓ T102971 Gray out Scene Auto-Masking option if equivalent Brush option is used](https://developer.blender.org/T102971)
- [⚓ T102377 Auto-Masking: Propagation Steps needed as a global setting](https://developer.blender.org/T102377)

Julien will let Thomas Dinges know to direct new developers to these tasks to help out.
We just need to add any important info to the tasks and make them straight forward to work on.

Going forward we can do the same for Dyntopo refactor & Paint Mode.

#### Revisit [T97957: Sculpt Mode Painting](https://developer.blender.org/T97957)

Simplify task description down to actionable info:

- Use `X` for color swapping and `Shift X` for Draw brush
- Eventually get rid of default brush shortcuts in keymap anyway
- Use D16241 UI in sculpt mode. Always visible but grayed out if viewport shading color is not `Attribute`
- In the popover, add a button to switch the setting. Replaces current behavior of automatically switching when using a color tool
- Color tools are not working (Ideally grayed out) if colors are not used in the viewport
- Eventual add painting support for any sculpting brushes/tools as an opt-in feature

Eventual steps after that task:
- Dyntopo support via T103346
- Basic multires support (mostly about assempling the right devs and agree on an approach)
- Smoother Voxel remesher preservation
- Improved Tools
- Color support for regular sculpting tools

#### Sculpt Layers discussion

More notes and ideas have been share in the module. A document is beign written for use cases and ideas.