Nathan Vegdahl nathanvegdahl
  • Amsterdam, Netherlands
  • https://cessen.com
  • Animator, rigger, and software developer. Currently working at the Blender Institute as a developer on Blender's animation system.

    Been using Blender since 1998, and worked on Big Buck Bunny and Sintel (two of Blender's open movie projects).

  • Joined on 2003-03-21
Nathan Vegdahl commented on pull request blender/blender#129052 2024-10-17 10:33:00 +02:00
Fix #125816: Clear constraint while action baking not doing visual keying

Okay, thanks for looking into it.

Just to double-check: is the challenge the layout (making something a sub-item) or the greying-out/disabling? Or both? If it's only the layout, then I think…

Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 17:51:53 +02:00
7deaac1e11 Update reports/2024.md
Nathan Vegdahl approved blender/blender#129007 2024-10-15 17:51:30 +02:00
Fix #125451: Duplicating bone doesn't copy pose bone colors

Code looks good and works great in testing.

Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 17:40:52 +02:00
12e9f99bda Update reports/2024.md
Nathan Vegdahl commented on pull request blender/blender#129052 2024-10-15 17:40:15 +02:00
Fix #125816: Clear constraint while action baking not doing visual keying

Another possibility would be to make "Clear Constraints" a sub-item under "Visual Keying" and grey it out (and internally disable it) when "Visual Keying" isn't checked. That would make the…

Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 17:33:51 +02:00
467ca6eb32 Update reports/2024.md
Nathan Vegdahl approved blender/blender#129057 2024-10-15 17:33:31 +02:00
Fix: Error when baking custom properties of unavailable addon

Looks good, and works great in testing.

Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 17:23:55 +02:00
3998dfa5db Update reports/2024.md
Nathan Vegdahl approved blender/blender#128984 2024-10-15 17:23:27 +02:00
Fix #128805: Missing Depsgraph Update when removing FCurve modifiers

Code looks good and works great in testing.

Nathan Vegdahl commented on pull request blender/blender#128984 2024-10-15 17:11:02 +02:00
Fix #128805: Missing Depsgraph Update when removing FCurve modifiers

The code seems good... except for some reason this isn't actually fixing the issue for me (following the steps with the example blend file in the linked issue).

Nathan Vegdahl commented on issue blender/blender#128519 2024-10-15 16:54:05 +02:00
Regression: Pasting pose on liboverride armature produces warning about 'removing all override data'

That triggers an assert when saving:

BLI_assert failed: /home/guest/Projects/blender/blender/source/blender/blenkernel/intern/main_idmap.cc:152, BKE_main_idmap_insert_id(), at 'existing_key…
Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 16:40:56 +02:00
4a83511fde Update reports/2024.md
Nathan Vegdahl approved blender/blender#129004 2024-10-15 16:40:34 +02:00
Fix #128445: Symmetrize pose bone settings

Code looks good, and works great in testing. Just one small note that can be addressed while landing.

Nathan Vegdahl commented on pull request blender/blender#129004 2024-10-15 16:40:33 +02:00
Fix #128445: Symmetrize pose bone settings

I would add a note here that properties are intended to be added on a case-by-case basis, to avoid mirroring things that shouldn't be mirrored. Just to ward off someone trying to do something "clever".

Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 15:35:01 +02:00
197ad17cce Update reports/2024.md
Nathan Vegdahl commented on issue blender/blender#128519 2024-10-15 15:25:29 +02:00
Regression: Pasting pose on liboverride armature produces warning about 'removing all override data'

I've spent some time looking into this, but it seems to be the intersection of two systems that I only have a rough high-level understanding of: partial blend file writing/reading and library…

Nathan Vegdahl deleted branch fix_action_upgrade_for_embedded_ids from nathanvegdahl/blender 2024-10-15 11:04:26 +02:00
Nathan Vegdahl merged pull request blender/blender#129002 2024-10-15 11:04:23 +02:00
Fix: handle embedded IDs when upgrading to slotted actions
Nathan Vegdahl pushed to main at blender/blender 2024-10-15 11:04:21 +02:00
989634c0a1 Fix: handle embedded IDs when upgrading to slotted actions
Nathan Vegdahl pushed to main at nathanvegdahl/.profile 2024-10-15 11:02:46 +02:00
02724fe898 Update reports/2024.md