Modification of Startup addon for Generate Torus mesh : section angle and twist feature added #78948

Closed
opened 2020-07-15 14:07:31 +02:00 by Stephane Cuillerdier · 11 comments

Hello,

i have modified the Startup addon for Generate Torus mesh.

for define the section angle and section twist.

see the demo video here : https://twitter.com/aiekick/status/1283182645158064129

i think this code can be usefull, but not sure if its required by community.
this is what i wanted to do when i created a simple torus.
i found the default addon not usefull in some cases

add_mesh_torus.zip

Hello, i have modified the Startup addon for Generate Torus mesh. for define the section angle and section twist. see the demo video here : https://twitter.com/aiekick/status/1283182645158064129 i think this code can be usefull, but not sure if its required by community. this is what i wanted to do when i created a simple torus. i found the default addon not usefull in some cases [add_mesh_torus.zip](https://archive.blender.org/developer/F8696879/add_mesh_torus.zip)

Added subscriber: @AieKick

Added subscriber: @AieKick

Completed for have two types of uv map.

  • the standard uv map for normal type of geometry
  • the uv map for one ribbon only when sectiion_twist is different of minor segment count (there is only one face)

add_mesh_torus.zip

Completed for have two types of uv map. - the standard uv map for normal type of geometry - the uv map for one ribbon only when sectiion_twist is different of minor segment count (there is only one face) [add_mesh_torus.zip](https://archive.blender.org/developer/F8711227/add_mesh_torus.zip)

Added subscriber: @mont29

Added subscriber: @mont29

Changed status from 'Needs Triage' to: 'Archived'

Changed status from 'Needs Triage' to: 'Archived'
Bastien Montagne self-assigned this 2020-07-28 11:14:50 +02:00

Thanks for the patch, but please submit it through the proper review/diff process (https://developer.blender.org/differential/diff/create/)

Thanks for the patch, but please submit it through the proper review/diff process (https://developer.blender.org/differential/diff/create/)

ok, i will.

I never done that before :)

ok, i will. I never done that before :)
done here : https://developer.blender.org/differential/diff/34826/

Not sure to understand why this ticket was closed.

This modification is rejected or maybe can i do some modifications ?

i thinkc this script fill a gap of feature lack for the otrus script. but maybe there is something i not understand ?
all guys i know who are using this modified script found it usefull.

Not sure to understand why this ticket was closed. This modification is rejected or maybe can i do some modifications ? i thinkc this script fill a gap of feature lack for the otrus script. but maybe there is something i not understand ? all guys i know who are using this modified script found it usefull.

@AieKick afaict, you only created a diff, you need to create a revision from this diff now, and get it reviewed (top part of the diff page). Discussion about your changes will then happen in the diff, not in this report.

@AieKick afaict, you only created a diff, you need to create a revision from this diff now, and get it reviewed (top part of the diff page). Discussion about your changes will then happen in the diff, not in this report.

ok thanks for thsi information

ok thanks for thsi information

ok its done : https://developer.blender.org/D11761
i have abandonned the last diff (not found a button for modifiy or delete it) because i have updated the code regarding the last torus startup script i have in blender 1.93

not sure about what reviewer or subscriber to use.
same for the repositery to use

thanks

ok its done : https://developer.blender.org/D11761 i have abandonned the last diff (not found a button for modifiy or delete it) because i have updated the code regarding the last torus startup script i have in blender 1.93 not sure about what reviewer or subscriber to use. same for the repositery to use thanks
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 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#78948
No description provided.