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

## Attendees

- Joe Eagar (dev)
- Hans Goudey (dev)
- Julien Kaspar(art)

## Agenda

### Needs Commit

- [⚓ T99036 Add Color Attribute: hex colors don't work](https://developer.blender.org/T99036)
- Can Jeroen commit?
- Will talk to him after the meeting

### High Prio Bugs

- [⚓ T99383 Regression: Crash in sculpting with color filter](https://developer.blender.org/T99383)
- Joe will investigate further
- [⚓ T99288 Color attributes of index above 7 deletes after entering edit mode](https://developer.blender.org/T99288)
- This should be the easiest to fix without causing much issues
- [⚓ T99282 Regression: Object invisible in Sculpt Mode when using >8 color attributes](https://developer.blender.org/T99282)
- Any 8 layer limit issues can be fixed with https://developer.blender.org/D15368
- Joe will communicate with viewport module to find a proper fix
- [⚓ T99191 Regression: Blender crashes when texture painting (caused by boolean modifers)](https://developer.blender.org/T99191)
- Hans claimed this
- [⚓ T98889 Regression: Sculpting on a linked copy of an objects crashes Blender](https://developer.blender.org/T98889)
- Joe got in contact with Viewport module?
- [⚓ T98731 Vertex Paint mode can edit only 8 color attributes](https://developer.blender.org/T98731)
- Hans made a patch
- Joe will check it out to sync up with his fixes

### In Review

- [⚓ T99294 Voxel Remesher text is resized based on object scale](https://developer.blender.org/T99294)
- Joe can review this
- Julien will also review on user level
- [⚓ T99203 Support Automasking For Texture Paint](https://developer.blender.org/T99203)
- Jeroen still needs to review
- [⚓ T97353 PBVH: Split PBVH Pixels into more nodes.](https://developer.blender.org/T97353)
- Jeroen still needs to update his review
- [⚓ T98382 Expose Face Sets In Python API](https://developer.blender.org/T98382)
- Still needs feedback adjustments
- [⚓ T97106 Options to convert color attributes to other domain and data types](https://developer.blender.org/T97106)
- Update from Sin Ran / Jeroen?

### In Progress

- [⚓ T96225 PBVH image texture painting implementation](https://developer.blender.org/T96225)
- Seam bleeding is still priority of Jereoen
- [⚓ T97352 3D texturing: Fix seam bleeding](https://developer.blender.org/T97352)
- Worked mainly on performance.
- [⚓ T98383 Sculpt Mode Cavity Masking](https://developer.blender.org/T98383)
- Still needs adjustments and fixes mentioned in last meeting
- Julien will also check it out again on a design/usability level
- [⚓ T98593 Paint Mode: Backface Occlusion](https://developer.blender.org/T98593)
- Still needs fixes and Occlusion feature

### Next To Do's

- Jeroen
- Making sure we can handle larger meshes (removing some hardcoded defaults)
- Joe
- Joe will create a task for the remaining PBVH drawing issues for further discussions and to prioritise it for the 3.3 release
- High prio Bug fixes
- Cavity & Normal automasking
- Julien
- Whenever possible: Refining & commincatiing designs. Testing.

### New Tasks

- [⚙ D15368 Remove Color Attribute Limit (and associated refactors)](https://developer.blender.org/D15368)

### Other Topics

- PBVH draw might need more fixes
- 16 attribute limit in viewport
- Other issue as well when alos drawing UV atributes
- Joe and Jereoen will talk more in detail about this
- Incremental refactors would be wise for targeting the 3.3 release
- In further releases there would be a bigger rewrite with Clement

- Operators for storing Mask & Face Sets as Attributes
- Will be added for curve sculpt mode as well
- Should be consistent with mesh sculpt mode then
- Proposal
- Mask -> Generic float
- Generic attribute -> Mask
- In mask dropdown?
- Would be easy but not very discoverable
- Could add to attribte panel
- Would be more ideal
- We can add it to both locations
- Face Sets -> Integer faces attribute
- Could be a lot more tricky
- Might need to simplify face sets code before adding this operator
- Any attribute -> Face Set
- Might want to comnine the idea with exisiting operators to initiazie face sets
- Needs more desing first
- Hans will prototype Face Sets conversion

- Joe wants to port over fix from sculpt-dev to fix face set automasking for "Draw Face Set" brush
- Would fix current bug

- Ramil abandoned [⚓ T97959 Color Filter: Sliding UI](https://developer.blender.org/T97959) for now
- The task is still free for anyone to pick up

- Juan Fran has very little time to contribute. He'll be availible for small tasks and design discussions.
- Once we have final approved designs he can help with non-brush tools and operators

- Switching color attrbiute is really slow
- Could be fixed with depsgraph update

- Adjusting [⚓ T97957 Sculpt Mode Painting](https://developer.blender.org/T97957) for 3.3 release?
- Jeroen has doubts that the Canvas picker can already be used for sculpt mode
- We'll keep current solution of switching on Color Attribute drawing in sculpt mode for next release
- Jeroen suggested to hold a workshop to fix long lasting viewport drawing issues
- For current releases it would be wise then to at least add a status bar notification that the viewport color setting has been changed
- Shortcuts need to be discussed. `D` will not be availbile for brush shortcuts. `Shift X` might be a good new default for color swapping?