Laurynas Duburas laurynas
  • Joined on 2021-06-10
Laurynas Duburas commented on pull request blender/blender#119053 2024-03-04 23:21:23 +01:00
Curves: Bezier handles for new Curves

For higher lever utils I need more time and clearer head.

Regarding left, point, right, .... One benefit I see is that positions are copied in chunks not one by one. And most importantly this…

Laurynas Duburas pushed to bezier-handles-for-new-curves at laurynas/blender 2024-03-04 22:42:20 +01:00
d5c2fe500e small fixes
dd1bf10ddf make format and undo comment formatting
Compare 2 commits »
Laurynas Duburas commented on pull request blender/blender#119053 2024-03-04 22:15:01 +01:00
Curves: Bezier handles for new Curves

That blender::bke::crazyspace thing was a puzzle for me. From what I understood there are three cases:

  1. there is no extra transformation deform_mats.size() == 0
  2. transformation exits, but…
Laurynas Duburas commented on pull request blender/blender#119053 2024-03-04 21:52:03 +01:00
Curves: Bezier handles for new Curves

Shader expects "data" buffer to be and have data allocated. For this I see two possible solutions: either two separate shaders or compute shader. First solution would be difficult to support and…

Laurynas Duburas commented on pull request blender/blender#119053 2024-03-04 21:14:46 +01:00
Curves: Bezier handles for new Curves

Yes. In overlay_edit_curves_handle_vert.glsl. Actually at the moment one bit would be sufficient, but in legacy curves it is 5 and thought there could go curve type information to make lattice…

Laurynas Duburas commented on pull request blender/blender#119053 2024-03-04 21:08:22 +01:00
Curves: Bezier handles for new Curves

I was actually working on a very similar thing just now. Didn't know someone else is working on this thing specifically. I'm happy to let you finish this though :) (my initial patch: #119036)

Laurynas Duburas created pull request blender/blender#119053 2024-03-04 13:42:16 +01:00
Curves: Bezier handles for new Curves
Laurynas Duburas pushed to bezier-handles-for-new-curves at laurynas/blender 2024-03-04 13:34:27 +01:00
46814e3ded Curves: Bezier handles for new Curves
Laurynas Duburas created branch bezier-handles-for-new-curves in laurynas/blender 2024-03-04 13:33:00 +01:00
Laurynas Duburas pushed to bezier-handles-for-new-curves at laurynas/blender 2024-03-04 13:33:00 +01:00
7bf4247b09 Depsgraph: add a DEG_graph_build_from_collection convenience function
8c16b612ca Merge branch 'blender-v4.1-release'
7e9a36c1fa Cleanup: suppress undefined variable use warnings
5af4987456 Merge branch 'blender-v4.1-release'
51126fab33 BLI_tempfile: ensure the temporary directory is absolute
Compare 10 commits »
Laurynas Duburas deleted branch fix-exception-in-curves-indexbuf from laurynas/blender 2024-03-04 07:28:14 +01:00
Laurynas Duburas commented on pull request blender/blender#118951 2024-03-01 21:42:46 +01:00
Fix: GPU: Ensures length of curves GPUIndexBuf to be multiple of 4

Do I need to do something about failed checks?

Laurynas Duburas commented on pull request blender/blender#118951 2024-03-01 15:24:55 +01:00
Fix: GPU: Ensures length of curves GPUIndexBuf to be multiple of 4

Also changed solution for case with zero curves. In version with max_ii(, 4) noticed weird visual artifacts.

Laurynas Duburas pushed to fix-exception-in-curves-indexbuf at laurynas/blender 2024-03-01 15:19:13 +01:00
f564c02715 handle empty curve object case
575509cd10 use of ceil_to_multiple_u
Compare 2 commits »
Laurynas Duburas pushed to fix-exception-in-curves-indexbuf at laurynas/blender 2024-03-01 11:31:34 +01:00
fdfaeb2768 fix crash on empty curve object
Laurynas Duburas pushed to fix-exception-in-curves-indexbuf at laurynas/blender 2024-03-01 09:58:40 +01:00
961b9fda58 unused variable fix
Laurynas Duburas created pull request blender/blender#118951 2024-03-01 09:28:39 +01:00
Fix: Ensures length of curves GPUIndexBuf to be multiple of 4
Laurynas Duburas created branch fix-exception-in-curves-indexbuf in laurynas/blender 2024-03-01 09:08:12 +01:00
Laurynas Duburas pushed to fix-exception-in-curves-indexbuf at laurynas/blender 2024-03-01 09:08:12 +01:00
1f1c34677c Ensures length of curves GPUIndexBuf to be multiple of 4
bcb0d6ef9f Fix: Crash when using compositor debugger
43d93973c0 Fix #118666: VSE scopes showing garbage data in some cases
6e8272a5dd Cleanup: pass ViewContext as a const argument where possible
ca8dc11a85 Merge branch 'blender-v4.1-release'
Compare 10 commits »
Laurynas Duburas commented on pull request blender/blender#116354 2024-01-17 15:31:02 +01:00
Curves: Add extrude operator

Curve type doesn't matter, extrude treats all curves the same. At least at this stage.

From functional side or user perspective GP extrude in Blender 4.0 does different things than extrude in…