# 2022-7-13 Sculpt/Texture/Paint Module Meeting

## Attendees

- Joe
- Julien
- Ramil

## Agenda

### Needs Commit

- [⚓ T99036 Add Color Attribute: hex colors don't work](https://developer.blender.org/T99036)
- Poke Jeroen. If he doesn't have time, poke someone else.

### High Prio Bugs

#### Color attribute bugs

- [⚓ T99288 Color attributes of index above 7 deletes after entering edit mode](https://developer.blender.org/T99288)
- This needs a separate fix
- [⚓ T99282 Regression: Object invisible in Sculpt Mode when using >8 color attributes](https://developer.blender.org/T99282)
- [⚓ T98889 Regression: Sculpting on a linked copy of an objects crashes Blender](https://developer.blender.org/T98889)
- [⚓ T98731 Vertex Paint mode can edit only 8 color attributes](https://developer.blender.org/T98731)

Any updates with these tasks?
- [⚙ D15428 PBVH Draw Cleanup](https://developer.blender.org/D15428)
- More involved change
- Poke Clement and Kevin for feedback. This is a high priority for bug fixes
- [⚙ D15368 Remove Color Attribute Limit (and associated refactors)](https://developer.blender.org/D15368)
- Easy part for bug fixes

#### Other
- [⚓ T96810 Regression: Shading artefacts in sculpt mode](https://developer.blender.org/T96810)
- Reopened since the issue is still present.
- Poke Hans in chat

### In Review

- [⚙ D14970 3D Texturing: Fix seam bleeding.](https://developer.blender.org/D14970)
- Needs review
- [⚓ T99294 Voxel Remesher text is resized based on object scale](https://developer.blender.org/T99294)
- Joe, Campbell can review this
- Joe has some concern with the patch
- He'll comment a fix
- [⚓ T99203 Support Automasking For Texture Paint](https://developer.blender.org/T99203)
- Poke Jeroen
- [⚓ T97353 PBVH: Split PBVH Pixels into more nodes.](https://developer.blender.org/T97353)
- Poke Jeroen
- [⚓ T98382 Expose Face Sets In Python API](https://developer.blender.org/T98382)
- Still needs feedback adjustments from Joe
- [⚓ T97106 Options to convert color attributes to other domain and data types](https://developer.blender.org/T97106)
- Jeroen/Sin Ran?
- Maybe Julien will poke Sin Ran instead

### In Progress

- [⚓ T96225 PBVH image texture painting implementation](https://developer.blender.org/T96225)
- [⚓ T98383 Sculpt Mode Cavity Masking](https://developer.blender.org/T98383)
- Any update on [⚙ D14272 Port SculptVertRef refactor to master](https://developer.blender.org/D14272) for performance?
- Still needs review update from Brecht
- Filter support?
- Joe will test this
- How to expose inverted cavity (or also boundary)
- We could expose it as 3 options that are muturally exclusive
- Disabled
- Enabled
- Enabled Inverted
- It was also propsoed to use nested pie menus
- This is not supported by the currentdesign though
- We could try keeping the 2 options in the pie mneu but implement the 2 as a bool
- Bake Cavity design
- Implement cavity mask operator as replacement for dirty mask
- Also implement normal mask oeprator
- Use automask settings by default
- Tweak oeprator settings afterwards with adjust last peration popup
- [⚓ T98593 Paint Mode: Backface Occlusion](https://developer.blender.org/T98593)
- Filter support
- Still seems to crash
- Joe will investigate
- But area normal automasking wouldn't be supported by filters anyway
- Sculpting tools support seems to be fixed
- Occlusion feature still missing

### Next To Do's

- Joe
- High prio bugs
- Automasking modes
- Jeroen
- ?
- Julien
- Test patches
- Design tasks

### New Tasks

- [⚓ T99607 Resolve "Shift R" shortcut conflict](https://developer.blender.org/T99607)
- This should be prioritised since the hair curves project is mimicking shortcut conflicts in sculpt mode
- `Alt R` is not convinient
- `Ctrl Shift R` is making commonshortcut too complex
- Or new shortcut for Repeat Last?
- Only `Ctrl Alt R` would be availible
- Would be too complex for users that use it a lot
- Also not convinient on multiple presses
- We need to investige. It's a shortcut that affects all of Blender
- [⚓ T99609 Use Scene Spacing Distance by default](https://developer.blender.org/T99609)
- A possible imporvement to investigate further
- Julien will test the behaviour to try to find more issues
- Joe wants to look into one fix: Incease the minimum allowed spacing of stroke points with scene spacing
- [⚓ T99614 Sculpt Mode: Color Picker Gradient](https://developer.blender.org/T99614)
- We should prioritise polishing the sculpt mode painting features for the upcoming release
- Joe suspects this might be easy to include. He'll investigate
- [⚓ T99615 Sculpt Paint: Smoothing Strength](https://developer.blender.org/T99615)
- Pressing issue only due to missing Painting Brush smoothing strength control
- Further issues would eb good to discuss and add to the design
- Shift smoothing is copying all brush settings from current smooth brush called "Smooth"
- If Smooth brush is set to surface smoothing or enhancing details, this affects Shfit smoothing
- Shift smoothing will in some cases take stroke settings from currently active brush
- Sculpt-Dev uses a single global slider
- Should we use a single global slider that controls mesh, color and mask smoothing (Shortcut on Alt F)?
- With further brush engine refactors we can include settings to overide the global smoothing strength on the brush -> Long term

### Other Topics

- Updated [⚓ T97957 Sculpt Mode Painting](https://developer.blender.org/T97957) shortcut design
- D is off limits because of annotate
- We should compare X for color swap shortcut with other software
- THis is the industry default, so chaning it might not be great
- We could also change Draw brush shortcut to V
- We could also discuss this as a bigger topic
- Remove most shortcuts (or all shortcuts) for brushes?
- This would enable users to define their own shortcuts for brushes they actually use
- We can discuss this further & recycle old discussions & tasks