Add 'Transform' Python Function (Quick Hacks) #37828

Closed
opened 2013-12-16 01:11:52 +01:00 by Paolo Acampora · 8 comments
Member

Hi, I jumped into blender and chosed a quick hack from the list .

Hope the code is good (and useful) enough to make into trunk, thank you!

Paolo
Transform_Python_Functions.diff

Hi, I jumped into blender and chosed a quick hack from the [list ](http://wiki.blender.org/index.php/Dev:Doc/Quick_Hacks). Hope the code is good (and useful) enough to make into trunk, thank you! Paolo [Transform_Python_Functions.diff](https://archive.blender.org/developer/F41454/Transform_Python_Functions.diff)
Author
Member

Changed status to: 'Open'

Changed status to: 'Open'
Author
Member

Added subscriber: @PaoloAcampora

Added subscriber: @PaoloAcampora

Added subscriber: @brecht

Added subscriber: @brecht

Thanks for the patch, can you submit it here? Makes it easier to review:
http://developer.blender.org/differential/diff/create/

Can you move rna_armature_apply_transform into an ED_ function and let it share code with ED_armature_apply_transform?

DAG_id_tag_update does not need the OB_RECALC_DATA flag in this case, it's only for Objects, instead use 0.

This patch breaks compilation of blenderplayer, you need to add a few stubs in source/blenderplayer/bad_level_call_stubs/stubs.c (same as e.g. ED_mesh_transform has).

Thanks for the patch, can you submit it here? Makes it easier to review: http://developer.blender.org/differential/diff/create/ Can you move `rna_armature_apply_transform` into an `ED_` function and let it share code with `ED_armature_apply_transform`? `DAG_id_tag_update` does not need the `OB_RECALC_DATA` flag in this case, it's only for Objects, instead use `0`. This patch breaks compilation of blenderplayer, you need to add a few stubs in source/blenderplayer/bad_level_call_stubs/stubs.c (same as e.g. `ED_mesh_transform` has).

Added subscriber: @ideasman42

Added subscriber: @ideasman42
Author
Member

Hi, I used the differential now, new patch is name .

I apologize for not using it before, hope it now follows your reccomandation.

Hi, I used the differential now, new patch is [name ](here). I apologize for not using it before, hope it now follows your reccomandation.
Author
Member

so silly, link is here :)

so silly, link is [here ](http://developer.blender.org/D110) :)

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Campbell Barton self-assigned this 2013-12-20 18:05:19 +01:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#37828
No description provided.