Bsurfaces error when cyclic cross and follow is checked in redo panel #98658

Closed
opened 2022-06-08 06:16:06 +02:00 by Dennis Crenshaw · 8 comments

System Information
Operating system: Windows 10
Graphics card: RTX 3060

Blender Version
Broken: 3.1.2
Worked: Don't Know

Short description of error
When using Cyclic Cross option the subdivide operation expects an int but receives a float from the sqrt() operation above.

Changing line 2536 to have an int conversion on segments seems to fix the problem.
bpy.ops.curve.subdivide('INVOKE_REGION_WIN', number_cuts=int(segments))

Exact steps for others to reproduce the error

  • Open .blend file
  • Enable bsurface add-on
  • perform Add surface operation (from N panel)
  • Expand redo panel
  • Check cyclic cross and cyclic follow

#98658.blend

**System Information** Operating system: Windows 10 Graphics card: RTX 3060 **Blender Version** Broken: 3.1.2 Worked: Don't Know **Short description of error** When using Cyclic Cross option the subdivide operation expects an int but receives a float from the sqrt() operation above. Changing line 2536 to have an int conversion on segments seems to fix the problem. bpy.ops.curve.subdivide('INVOKE_REGION_WIN', number_cuts=int(segments)) **Exact steps for others to reproduce the error** - Open .blend file - Enable bsurface add-on - perform Add surface operation (from N panel) - Expand redo panel - Check cyclic cross and cyclic follow [#98658.blend](https://archive.blender.org/developer/F13177536/T98658.blend)

Added subscriber: @Dennis-Crenshaw

Added subscriber: @Dennis-Crenshaw
Member

Added subscriber: @PratikPB2123

Added subscriber: @PratikPB2123
Member

Changed status from 'Needs Triage' to: 'Needs User Info'

Changed status from 'Needs Triage' to: 'Needs User Info'
Member

Hi, thanks for the report
I'm not able to reproduce this with you given steps (in 3.2, 3.3). Could you reconfirm reproducing steps?

int type expected float provided

This is likely due to python 3.10 upgrade.

Hi, thanks for the report I'm not able to reproduce this with you given steps (in 3.2, 3.3). Could you reconfirm reproducing steps? > int type expected float provided This is likely due to python 3.10 upgrade.

I'm absolutely sure the python upgrade is the cause, this isn't the only addon I've had troubles with for this expected number type case.

I think I wrote this ticket very late at night and it was Cyclic Follow that caused my heartburn:

image.png

I'm absolutely sure the python upgrade is the cause, this isn't the only addon I've had troubles with for this expected number type case. I think I wrote this ticket very late at night and it was Cyclic Follow that caused my heartburn: ![image.png](https://archive.blender.org/developer/F13138603/image.png)
Pratik Borhade changed title from bsurfaces addon "cyclic cross" error, int type expected float provided to bsurfaces addon error,when cyclic cross and follow is checked in redo panel 2022-06-17 06:38:02 +02:00
Member

Changed status from 'Needs User Info' to: 'Confirmed'

Changed status from 'Needs User Info' to: 'Confirmed'
Pratik Borhade self-assigned this 2022-06-17 06:39:55 +02:00
Pratik Borhade changed title from bsurfaces addon error,when cyclic cross and follow is checked in redo panel to Bsurfaces error when cyclic cross and follow is checked in redo panel 2022-06-17 12:36:19 +02:00

This issue was referenced by 0d1a3cc243

This issue was referenced by 0d1a3cc2430efe46cd5f4b224a29f17133947283
Member

Changed status from 'Confirmed' to: 'Resolved'

Changed status from 'Confirmed' to: 'Resolved'
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 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#98658
No description provided.