Amélie Fondevilla amelief
  • https://www.afonde.com/
  • Research Engineer at the animation studio Les Fées Spéciales. Occasional contributor to Grease Pencil.

  • Joined on 2022-01-17
Amélie Fondevilla pushed to gpv3-extrude-point-operator at amelief/blender 2024-05-02 12:48:32 +02:00
c37f795093 Remove unused variable
Amélie Fondevilla pushed to gpv3-extrude-point-operator at amelief/blender 2024-05-02 12:34:56 +02:00
0b369be623 Remove unused property
bdce8dd0ab Change return type of the function
394d988d15 Minor fixes
6a5e197c51 Minor fixes
59b8532906 Move function to grease_pencil_edit
Compare 112 commits »
Amélie Fondevilla commented on pull request blender/blender#121249 2024-04-30 13:08:10 +02:00
GPv3: Extrude operator

Thanks for spotting this @PratikPB2123 ! I will move it to grease_pencil_edit.cc 🙂

Amélie Fondevilla commented on pull request blender/blender#121249 2024-04-30 13:05:21 +02:00
GPv3: Extrude operator

Hey @filedescriptor thanks for the review, I'll edit the code. I was wondering it we could optimize this by separating the case of the extrude containing endpoints or not. If the selection does…

Amélie Fondevilla commented on pull request blender/blender#121249 2024-04-30 12:56:22 +02:00
GPv3: Extrude operator

I suspected that the intersection was not efficient yes 😅 For point_to_curve_map, I think I recall Hans saying that it was better to loop over the curves and use points_by_curve, but ofc…

Amélie Fondevilla created pull request blender/blender#121249 2024-04-30 10:25:45 +02:00
GPv3: Extrude operator
Amélie Fondevilla pushed to gpv3-extrude-point-operator at amelief/blender 2024-04-30 10:02:00 +02:00
7310379437 Deal with cyclic attribute
00fa4af305 Extrude both inner and end points
dbed4e66eb Extrude points, inner point behavior only
05b7cf4283 Set up extrude function in curves edit
0684e187c2 Set up the operator and include it in the UI
Compare 10 commits »
Amélie Fondevilla created branch gpv3-extrude-point-operator in amelief/blender 2024-04-30 10:02:00 +02:00
Amélie Fondevilla approved blender/blender#120606 2024-04-16 15:01:38 +02:00
GPv3: Layer properties in dopesheet n-panel

Looks good to me. This is not a part of the code that I am familiar with, so I left just one comment on the code.

Amélie Fondevilla commented on pull request blender/blender#120606 2024-04-16 15:01:37 +02:00
GPv3: Layer properties in dopesheet n-panel

Why is there two transform panels ? Maybe the one with a capital T is a duplicate ?

Amélie Fondevilla commented on pull request blender/blender#120605 2024-04-16 14:29:07 +02:00
GPv3: Display layers in dopesheet and timeline

hi @PratikPB2123 , thanks for the PR.

I think this integration should also come with the support of basic operators, at least to avoid crashes. For example, right now, if I try to move a…

Amélie Fondevilla created pull request blender/blender#119337 2024-03-11 18:14:39 +01:00
Fix #118509 : Grease Pencil frames not longer disappearing while being transformed in the timeline
Amélie Fondevilla created branch gpv3-fix-disappearing-frames in amelief/blender 2024-03-11 17:57:58 +01:00
Amélie Fondevilla pushed to gpv3-fix-disappearing-frames at amelief/blender 2024-03-11 17:57:58 +01:00
1665d6e569 Use the operator flag for duplicate in the init function instead of the update function
02375a513d Minor change: Vector to Span in function call
6d06713129 Fix disappearing frames when moving multiple frames
f04bf1694f Fix #118509: Make sure to use valid indices in frame maps
51bcaad457 GPv3: Set dash segment active
Compare 10 commits »
Amélie Fondevilla approved blender/blender#119207 2024-03-11 12:41:38 +01:00
Fix #118509: Make sure to use valid indices in frame maps

Thank you for fixing this, I cant believe I did not notice this before..

Amélie Fondevilla approved blender/blender#117628 2024-01-31 09:51:45 +01:00
GPv3: Dopesheet UI template

Looks good to me !

Amélie Fondevilla commented on pull request blender/blender#117628 2024-01-31 09:44:46 +01:00
GPv3: Dopesheet UI template

For isolate layer, guess I need to create a new operator for that (should that be in this PR or a separate?).

I guess it's best to have a…