[FIXED] Ticket #4835 - Periodic BSpline Control Polygon not Closing Anymore

Here's the place for discussion related to coding in FreeCAD, C++ or Python. Design, interfaces and structures.
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Post Reply
louisgag
Posts: 75
Joined: Fri Jan 17, 2020 9:59 am

[FIXED] Ticket #4835 - Periodic BSpline Control Polygon not Closing Anymore

Post by louisgag »

Hello,

I hope I did not mix something up in the settings, but my Periodic B-Splines are not closing properly anymore and I suspect it has to do with one of the latest daily updates (stopped working about a week ago).

Any Periodic BSpline I create leaves the last member of the control polygon out, as in the picture
The sketch also has a missing coincidence where the spline should close.
BSplineNotClosing.png
BSplineNotClosing.png (13.98 KiB) Viewed 2284 times
MissingCoincidence.png
MissingCoincidence.png (16.13 KiB) Viewed 2284 times
Here's the link the the related issue: https://tracker.freecadweb.org/view.php?id=4835 and an example FreeCAD file is attached.

OS: Ubuntu 20.04.3 LTS (ubuntu:GNOME/ubuntu)
Word size of FreeCAD: 64-bit
Version: 0.20.
Build type: Release
Branch: unknown
Hash: 4fb91e4f02636fdb69eb8471dbbbe154dc932872
Python version: 3.8.10
Qt version: 5.12.8
Coin version: 4.0.0
OCC version: 7.5.2
Locale: C/Default (C)


Can someone confirm this bug?
Attachments
periodicSplineNotClosing.FCStd
(4.49 KiB) Downloaded 31 times
Last edited by Kunda1 on Sun Feb 06, 2022 6:36 pm, edited 2 times in total.
Reason: Added [Fixed] to thread title
chrisb
Veteran
Posts: 54192
Joined: Tue Mar 17, 2015 9:14 am

Re: Periodic BSpline Control Polygon not Closing Anymore

Post by chrisb »

The missing coincidence is only indicating that there is no coincidence constraint. This was already in 0.18 so.
And the missing line may not really be an issue. If I open the file in 0.20.26155 the fourth line is shown.
A Sketcher Lecture with in-depth information is available in English, auf Deutsch, en français, en español.
User avatar
adrianinsaval
Veteran
Posts: 5551
Joined: Thu Apr 05, 2018 5:15 pm

Re: Periodic BSpline Control Polygon not Closing Anymore

Post by adrianinsaval »

perhaps abdullah's changes to viewprovider have something to do with this?
GeneFC
Veteran
Posts: 5373
Joined: Sat Mar 19, 2016 3:36 pm
Location: Punta Gorda, FL

Re: Periodic BSpline Control Polygon not Closing Anymore

Post by GeneFC »

The fourth line on the right side line shows in 0.20.26761 and does not show in 0.20.26858. I don't have any versions between those numbers.

Gene
User avatar
Kunda1
Veteran
Posts: 13434
Joined: Thu Jan 05, 2017 9:03 pm

Re: Ticket #4835 - Periodic BSpline Control Polygon not Closing Anymore

Post by Kunda1 »

abdullah wrote::bell:
CC @Abdullah
Alone you go faster. Together we go farther
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
abdullah
Veteran
Posts: 4935
Joined: Sun May 04, 2014 3:16 pm
Contact:

Re: Ticket #4835 - Periodic BSpline Control Polygon not Closing Anymore

Post by abdullah »

Kunda1 wrote: Sat Feb 05, 2022 2:04 am
abdullah wrote::bell:
CC @Abdullah
Thanks for the ping.

git commit 7aefd2366f182c09683fc696f38c6a57c1b8fa15
User avatar
Kunda1
Veteran
Posts: 13434
Joined: Thu Jan 05, 2017 9:03 pm

Re: [FIXED] Ticket #4835 - Periodic BSpline Control Polygon not Closing Anymore

Post by Kunda1 »

Nice. I've closed the ticket.

@louisgag can you retest?
Alone you go faster. Together we go farther
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
louisgag
Posts: 75
Joined: Fri Jan 17, 2020 9:59 am

Re: [FIXED] Ticket #4835 - Periodic BSpline Control Polygon not Closing Anymore

Post by louisgag »

I've just tested with the daily package and the control polygon closes properly.
(The missing coincidence remains, but it doesn't prevent the extrusion and is not an issue, as pointed above by chrisb)

Thanks for having fixed it.
Post Reply