Add Catmull Rom as supported types to relevant node pages #104473

Merged
Hans Goudey merged 3 commits from persun/blender-manual:spline into blender-v3.6-release 2023-06-05 20:32:41 +02:00
Contributor

Add Catmull Rom as supported types to relevant node pages.

Also add some additional information.

Fixes #104434


I think the issue here is that the addition of Catmull Rom is not documented. I assume that this came from the fact that Catmull Rom type is only available in the new curves data. I imagine it's tricky to document it (adding it as one of the types here for example) since it's not available in the current (legacy) curve edit mode, etc.

So I only did minimal edit to reduce confusion like #104434. Even if this will only last for 3.6, I think it's worth it because this is an LTS version.

Spline Resolution

  • Add Catmull Rom as one of the supported types
  • According to this commit ddf956fb58, poly type splines can have resolution value other than 1. They just are not used

Set Spline Resolution

  • Add Catmull Rom as one of the supported spline types
  • Add a note that the resolution will not have an effect on segments between vector handles

Set Spline Type

  • Add a note about Bezier handles after the conversion. (I think this seems a bit out of place here. Maybe add as a note?)
  • Add Catmull Rom as one of the available types
Add Catmull Rom as supported types to relevant node pages. Also add some additional information. Fixes #104434 --- I think the issue here is that the addition of Catmull Rom is not documented. I assume that this came from the fact that Catmull Rom type is only available in the new curves data. I imagine it's tricky to document it (adding it as one of the types [here](https://docs.blender.org/manual/en/4.0/modeling/curves/structure.html) for example) since it's not available in the current (legacy) curve edit mode, etc. So I only did minimal edit to reduce confusion like #104434. Even if this will only last for 3.6, I think it's worth it because this is an LTS version. **Spline Resolution** - Add Catmull Rom as one of the supported types - According to this commit https://projects.blender.org/blender/blender/commit/ddf956fb5846da3835a1785978e0b9f1af89b45d, poly type splines can have resolution value other than 1. They just are not used **Set Spline Resolution** - Add Catmull Rom as one of the supported spline types - Add a note that the resolution will not have an effect on segments between vector handles **Set Spline Type** - Add a note about Bezier handles after the conversion. (I think this seems a bit out of place here. Maybe add as a note?) - Add Catmull Rom as one of the available types
Sun Kim added 3 commits 2023-06-03 17:45:17 +02:00
1c808817c5 Update Set Spline Type
- Add a note about Bezier handles  during conversion
- Add Catmull Rom as one of available types
350734f09a Update Set Spline Resolution
- Add Catmull Rom as one of spline types
- Add a note that the resolution will not have an effect on segements between vector handles
6cd022aa8a Update Spline Resolution
Add Catmull Rom as one of the possible types.
Sun Kim changed title from Add Catmull Rom as supported types to Add Catmull Rom as supported types to relevant node pages 2023-06-03 17:45:49 +02:00
Sun Kim requested review from Hans Goudey 2023-06-03 17:46:01 +02:00
Hans Goudey approved these changes 2023-06-05 20:32:17 +02:00
Member

Thanks!

Thanks!
Hans Goudey merged commit 14b7889c77 into blender-v3.6-release 2023-06-05 20:32:41 +02:00
Sun Kim deleted branch spline 2023-06-13 22:39:12 +02:00
Sign in to join this conversation.
No reviewers
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-manual#104473
No description provided.