User:Filedescriptor/Reports/2021

= Weekly Reports 2021 =

January 04 - 10 (week 01)
Involved in 21 reports: ,, , , , , , , , , , , , ,
 * Confirmed: 9
 * Closed as Resolved: 2
 * Closed as Invalid: 6
 * Closed as Duplicate: 1

, ,
 * Needs Info from User: 3
 * Needs Developer to Reproduce: 0

Patches and commits

1 patch needs review:
 * Extending hint in the collision section

2 Commits:
 * Fix : Linking regular materials to gpencil
 * Fix : NURBS edit mode lines not showing

January 11 - 17 (week 02)
Involved in 30 reports: ,, , , , , , , , , , , ,
 * Confirmed: 14
 * Closed as Resolved: 0

,, , , , ,
 * Closed as Invalid: 7
 * Closed as Duplicate: 0

,, , , , , ,
 * Needs Info from User: 8
 * Needs Developer to Reproduce: 1

Patches and commits

2 patches need review: 0 Commits:
 * Fix : Wrong scale for primitives with radius
 * Fix : Reassign effect-strip ignores animdata

January 18 - 24 (week 03)
Involved in 25 reports: ,, , , , , , , , , , , ,
 * Confirmed: 7
 * Closed as Resolved: 3
 * Closed as Invalid: 6
 * Closed as Duplicate: 1

,, , , , , , ,
 * Needs Info from User: 8
 * Needs Developer to Reproduce: 0

Patches and commits

0 patches need review:

4 Commits:
 * Fix : Doversion for grease-pencil curve edit settings
 * Fix : Incorrect height using interactive add tool
 * Fix : VSE: Reassign effect-strip ignores animdata
 * GPencil: Deactivate interpolation of gpencil curves

January 25 - 31 (week 04)
Involved in 43 reports: ,, , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , ,
 * Confirmed: 10
 * Closed as Resolved: 2
 * Closed as Invalid: 13
 * Closed as Duplicate: 5
 * Needs Info from User: 11
 * Needs Developer to Reproduce: 0

,
 * Helped/Investigated/Other: 2

Patches and commits

2 patches need review:


 * Fix : Too big bisect threshold
 * Fix : GPencil: Drawing on locked axis plane

2 Commits:


 * Fix : Python API: undo/redo app handlers broken
 * Fix : GPencil: Perspective distortion while drawing

February 01 - 07 (week 05)
Involved in 30 reports: ,, , , , , ,
 * Confirmed: 8

,
 * Closed as Resolved: 2

,, , , , , , , ,
 * Closed as Invalid: 10


 * Closed as Duplicate: 1

, ,
 * Needs Info from User: 3


 * Needs Developer to Reproduce: 0

,, , , ,
 * Helped/Investigated/Other: 6

Patches and commits

0 patches need review:



2 Commits:


 * Fix : GPencil: Drawing on locked axis plane
 * Fix : Shrink/Fatten display number units

February 08 - 14 (week 06)
Involved in 30 reports: ,, , , , , , , , , ,
 * Confirmed: 12


 * Closed as Resolved: 1

,, , , , ,
 * Closed as Invalid: 7

,, ,
 * Closed as Duplicate: 4

, ,
 * Needs Info from User: 3


 * Needs Developer to Reproduce: 0

, ,
 * Helped/Investigated/Other: 3

Patches and commits

1 patch needs review:


 * Fix : Crash on switching to edit mode with uv editor open

1 Commit:


 * Fix : Display units inset operator

February 15 - 21 (week 07)
Involved in 29 reports:

, ,
 * Confirmed: 3

,
 * Closed as Resolved: 2

,, , ,
 * Closed as Invalid: 5

,, , , , , , , ,
 * Closed as Duplicate: 10

,, , , , ,
 * Needs Info from User: 7


 * Needs Developer to Reproduce: 0

,, ,
 * Helped/Investigated/Other: 2

Patches and commits

0 patches need review:



2 Commits:


 * Fix : Crash on selecting multiple frames
 * Fix : Crash on switching to edit mode with uv editor open

February 22 - 28 (week 08)
Involved in 31 reports:

,, , , , ,
 * Confirmed: 7


 * Closed as Resolved: 1

,, , , ,
 * Closed as Invalid: 6

,, , ,
 * Closed as Duplicate: 5

,, , , , ,
 * Needs Info from User: 7


 * Needs Developer to Reproduce: 0

,, , ,
 * Helped/Investigated/Other: 5

Patches and commits

0 patches need review:



1 Commit:


 * Fix : Selection when GP is parented

March 01 - 07 (week 09)
Involved in 45 reports:

,, , , , , , , , , , , , , , , ,
 * Confirmed: 18


 * Closed as Resolved: 0

,, , , , , , , , , , , , ,
 * Closed as Invalid: 15

,, ,
 * Closed as Duplicate: 4

,, , , ,
 * Needs Info from User: 6


 * Needs Developer to Reproduce: 0

,
 * Helped/Investigated/Other: 2

Patches and commits

1 patch needs review:


 * Fix : Allow geometry input in any position

0 Commits:



March 08 - 14 (week 10)
Involved in 46 reports:

,, , , , , , ,
 * Confirmed: 9


 * Closed as Resolved: 1

,, , , , , , , , , , ,
 * Closed as Archived: 13

,, , , ,
 * Closed as Duplicate: 6

,, , , , , , , , , ,
 * Needs Information from User: 12


 * Needs Information from Developer: 0

,, , ,
 * Helped/Investigated/Other: 5

Patches and commits

1 patch needs review:


 * Fix : Add bisect distance as a parameter to the mirror modifier

1 Commit:


 * Fix : GPencil: Select color for glow is inverted

March 15 - 21 (week 11)
Involved in 23 reports:

,, , , , ,
 * Confirmed: 7

,
 * Closed as Resolved: 2

,, , , ,
 * Closed as Archived: 6

,
 * Closed as Duplicate: 2

,, ,
 * Needs Information from User: 4


 * Needs Information from Developer: 0

,
 * Helped/Investigated/Other: 2

Patches and commits

1 patch needs review:


 * Fix : transform_around_single_fallback_ex not working correctly

2 Commits:


 * Fix : trace sequence keyframe offset
 * Fix : Description missing from Quick Liquid

March 22 - 28 (week 13)
Involved in 32 reports:

,, , , , , , , , , , , ,
 * Confirmed: 14


 * Closed as Resolved: 0

,, , , , ,
 * Closed as Archived: 7


 * Closed as Duplicate: 1

,, , , ,
 * Needs Information from User: 6


 * Needs Information from Developer: 0

,, ,
 * Helped/Investigated/Other: 4

Patches and commits

0 patches need review:



0 Commits:



March 29 - April 04 (week 14)
Involved in 30 reports:

,, , ,
 * Confirmed: 5

,, ,
 * Closed as Resolved: 4

,, , , ,
 * Closed as Archived: 6

, ,
 * Closed as Duplicate: 3

,, , ,
 * Needs Information from User: 5


 * Needs Information from Developer: 0

,, , , , ,
 * Helped/Investigated/Other: 7

Patches and commits

1 patch needs review:


 * Fix : Last UV factor in cyclic strokes

5 Commits:


 * GPencil Cleanup: Remove legacy key to cancel drawing
 * Fix : Drag & Drop tooltip in Scene mode
 * Fix : Add tooltip for "Paste Flipped"
 * Add documentation for paste options
 * Fix : Smooth thickness not working

April 05 - April 11 (week 15)
Note: Monday (April 05) was a national holiday

Bug Tracker
Involved in 20 reports:

,, , , , ,
 * Confirmed: 7


 * Closed as Resolved: 1

,, , , , , ,
 * Closed as Archived: 8

,
 * Closed as Duplicate: 2


 * Needs Information from User: 1


 * Needs Information from Developer: 0


 * Helped/Investigated/Other: 1

Patches and commits

2 patches needs review:


 * Fix : Last UV factor in cyclic strokes
 * Fix : Inconsistensy skin root with extrude

1 Commit:


 * : Fix : GPencil subdivide last segment

Grease Pencil
I did some cleanup and more work for the bézier stroke type project this week. Also spent a good amount of time preparing a devtalk thread including a few demos (bezier pen, curve sculpting, and armature modifier with bezier strokes).


 * GPencil: Cleanup armature deform code
 * GPencil: Free poly weights when curve is updated
 * GPencil cleanup: Fix compiler warning
 * GPencil: Change UI name and icon for bezier pen
 * GPencil: Set fixed resolution bezier pen
 * GPencil: Allow bezier stroke sculpting

Bug Tracker
Involved in 14 reports:

,, ,
 * Confirmed: 4

,
 * Closed as Resolved: 2

,, ,
 * Closed as Archived: 4


 * Closed as Duplicate: 0

,
 * Needs Information from User: 2


 * Needs Information from Developer: 0

,
 * Helped/Investigated/Other: 2

Patches and commits

2 patches need review:


 * Fix : Add bisect distance as a parameter to the mirror modifier
 * Fix : Avoid uiBut update if value was same

3 Commits:


 * Fix : Stroke color when using object color mode
 * Fix compile error: Missing include paths
 * Fix : GPencil SVG export outside of camera

Grease Pencil
This week, a few meetings were scheduled. Additionally, I finished the devtalk thread and published it:
 * Finished the devtalk thread on the bezier stroke type.
 * Meeting with the Grease Pencil team on Wednesday. Subjects were: Feedback on the state of the `temp-gpencil-bezier-stroke-type` branch, current priorities, patch review sprint, drawing issues (sparked discussion and creation of ).
 * Meeting notes on the LineArt meeting on Friday.

Work on the `temp-gpencil-bezier-stroke-type` was continued. The main focus this week was to introduce the geometry update flags and start working on weights. At the moment there are many issues regarding weights, e.g. no proper transfer when converting between stroke types, not handling weights in the edit mode operators for bezier strokes. and finally no support for weight painting. I began by making sure that weights were properly created when the polyline is created from the curve data.


 * GPencil: Remove update flags for stroke and curve
 * GPencil: Hide handles in stroke mode
 * GPencil: Add flags for polyline regeneration
 * GPencil: Use geom update flags in update functions

Bug Tracker
Involved in 19 reports:

,, ,
 * Confirmed: 4

,, ,
 * Closed as Resolved: 4

,, , ,
 * Closed as Archived: 5


 * Closed as Duplicate: 0

,, , ,
 * Needs Information from User: 5


 * Needs Information from Developer: 0


 * Helped/Investigated/Other: 1

Patches and commits

0 patches need review:



4 Commits:


 * Fix : Avoid uiBut update if value was same.
 * Fix : Dragging button value cancel not working.
 * Fix : Last UV factor in cyclic strokes
 * Fix : Crash entering valid text into number field

Grease Pencil
I spent two days working on the weight data transfer from curve to polyline. This part is mostly finished now. For the rest of the week, I looked into a new approach to do curve fitting. At the moment, we do a multi-dimensional curve fitting by treating position, pressure, strength, and vertex color as different dimensions of the curve. The problem is that all of the attributes except the position are not affected by the curve handles. We do an interpolation between two control points to generate e.g. the pressure. Since the curve fitting returns a multi-dimensional curve that has handles for all the attributes, the resulting curve we get is not ideal. My new idea is to do separate curve fittings for the attributes. This has two advantages: I began by implementing a function to find stationary points in a bezier curve. This will be one of the core functions of the new fitting implementation.
 * We can choose what attributes we want to fit (e.g. there are cases where we know only specific attributes have changed so we only refit those)
 * We can treat attributes differently depending on their function (e.g. position will affect handles, pressure will not).

Bug Tracker
Involved in 30 reports:

,, , , , , ,
 * Confirmed: 8

,
 * Closed as Resolved: 2

,, , , , , , , , ,
 * Closed as Archived: 11


 * Closed as Duplicate: 1

,, ,
 * Needs Information from User: 4


 * Needs Information from Developer: 0

,, ,
 * Helped/Investigated/Other: 4

Patches and commits

0 patches need review:



2 Commits:


 * Fix : Curve Edit handle offset when snapping
 * Fix : Fill triangles wrongly calculated

Grease Pencil
This week I took a bit of a break from implementing the new fitting algorithm and worked on various areas regarding the `temp-gpencil-bezier-stroke-type` branch. The most notable changes are:
 * Bézier Mode in Draw Mode: This change introduces a way to draw freehand curves. In the future, the plan is to have all drawing tools create bézier strokes in this mode.
 * Curve smoothing algorithm: This was one of the remaining operators (`GPENCIL_OT_stroke_smooth`) that were unclear as to how they would work for bézier strokes. I came up with a solution that is fast as well as consistent with the poly stroke smoothing that we have already.
 * Vertex paint for bézier strokes: Added initial support for vertex painting. At the moment this implements the `Tint` brush.
 * Refactor of the dissolve operator: No functional changes. Introduces `BKE_gpencil_editcurve_dissolve` and moves all the logic into that function. This not only makes the code cleaner, but was also done in preparation for the `GPENCIL_OT_stroke_merge_by_distance` operator.

Bug Tracker
Involved in 20 reports:

, ,
 * Confirmed: 3

,
 * Closed as Resolved: 2

,, , , ,
 * Closed as Archived: 6


 * Closed as Duplicate: 0

,, , , , ,
 * Needs Information from User: 7


 * Needs Information from Developer: 0

,
 * Helped/Investigated/Other: 2

Patches and commits

1 patch needs review:


 * Fix : GPencil pressure changes not handled

2 Commits:


 * Fix : Motion tracker button tooltips identical
 * Fix : Hover+return doesn't accept 0 as input

Grease Pencil
This week I did more work on the new fitting algorithm, although progress was slow and I ended up reverting some of the work I did. I worked on some of the other edit mode operators: 'Merge by Distance` was finished, the new smooth algorithm is now used in the smooth modifier, and I worked on the 'Sample' operator that will re-sample the bézier stroke to have curve points at the same distance. This last algorithm is not fully implemented yet.


 * Implement the smooth modifier for bézier strokes
 * Implement 'Merge By Distance' for bézier strokes

May 10 - May 16 (week 20)
Note: Thursday this week was a national holiday.

Bug Tracker
Involved in 22 reports:

,, , , , ,
 * Confirmed: 7
 * Closed as Resolved: 0

,, , , ,
 * Closed as Archived: 6
 * Closed as Duplicate: 0

,
 * Needs Information from User: 2
 * Needs Information from Developer: 0

,, , , , ,
 * Helped/Investigated/Other: 7

Patches and commits

0 patch needs review:



0 Commits:



Grease Pencil
This week I was working on the sample operator for curves as well as a bug report. I created a patch that allows disabling grease pencil masks on a layer during render. This is especially useful for when you need to do the compositing after and just want the layers as they are with no masking.


 * GPencil: Add option to disable masks in view layer

May 17 - May 23 (week 21)
I was on holiday this week.

May 24 - May 30 (week 22)
Note: Monday this week was a national holiday.

Bug Tracker
Involved in 9 reports:

, ,
 * Confirmed: 3


 * Closed as Resolved: 1

, ,
 * Closed as Archived: 3


 * Closed as Duplicate: 0


 * Needs Information from User: 0


 * Needs Information from Developer: 0

,
 * Helped/Investigated/Other: 2

Patches and commits

1 patch needs review:


 * Fix T88651: svg rect not using id as name

1 Commit:
 * Fix: GPencil mask shows in view layer render

Grease Pencil
This week I looked into conversion between curve objects and grease pencil. With the new stroke type, it makes sense to convert curves into the bézier stroke type. Eventually I would like the code to handle the other way (so from grease pencil to curve) as well as import/export of SVG. I also worked on an issue and added an option to disable masks during rendering of grease pencil layers in a view layer (see https://wiki.blender.org/wiki/Reference/Release_Notes/3.0/Grease_Pencil#Compositing for the release note of this feature).


 * GPencil: Add option to disable masks in view layer
 * GPencil: Convert curve objects to bezier strokes

Other
I handled some reports on the bug tracker this week that ended up not being bugs, but rather small feature requests. I still worked on two that mades sense to me:


 * UI: Add light count to viewport statistics
 * Python: Add init/cancel app handlers for save/load