https://github.com/FreeCAD/FreeCAD/pull/6738
0.20 bug/regression Array Pattern issue #6641
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Be nice to others! Respect the FreeCAD code of conduct!
Re: 0.20 bug/regression Array Pattern issue #6641
My latest (or last) project: B-spline Construction Project.
Re: 0.20 bug/regression Array Pattern issue #6641
Sorry for being annoying, but i have tested the PR and other stuff and i have remarks and many many question.
(except, when i write something different for testing)
First off all, for the most normal operations (rectangle, round, plane) the transform tool works, but sometimes with more complex shapes i found some funny behaviors.
I managed in an old test build before, that all fillets and transform work in the file in the linked post, filename: Kappe50-jeno_edit.FCStd.
https://forum.freecad.org/viewtopic.php ... 40#p582840
This was OCCT7.4.0 with git commit 89b8e81d5c3b11572a0288720d0b4145d2ec2a7e
Now with retesting it with the same OCCT and commit, it do not work any more and i do not know why. Unfortunately while cleaning yesterday, i deleted the original build.
When i tested the PR, originally this also do not work. Then while clicking around the properties, i found out, there is property called Support Transform for features like fillets, chamfers, draft and thickness.
When i set it to true on Fillet001, then it works, but i did never changed it before, also on the test file (when you open it, you will see it).
When i also set it true on Fillet, then the transformation succeed, but the result is unexpected, because the transformation of the Fillet is not performed, which his also for the name of the property misleading.
So my question: what does the property? That is the purpose of it? I found nothing about that. Is the property incorrect implemented or normally deprecated all all?
Also in the PR brach version, the bace face do not refine, even when Refine = true. In this version, OCCT7.4.0 with git commit 89b8e81d5c3b11572a0288720d0b4145d2ec2a7e, it does.
Just also a remark, it no regression against 0.19, there this do not work at all. It is a regression about a dev-build. I do not know (and i do not rate) if it has an impact about this evaluation.
Fun fact: with OCCT7.6.1 the transformation do not work with the testfile.
Greetings
user1234
Code: Select all
OS: Debian GNU/Linux 11 (bullseye) (X-Cinnamon/lightdm-xsession)
Word size of FreeCAD: 64-bit
Version: 0.20.28691 (Git)
Build type: Debug
Branch: wmayer_issue_6641
Hash: 2871aa3823f1b49039f83b92a2102a030f99c317
Python 3.9.2, Qt 5.15.2, Coin 4.0.0, OCC 7.4.0
Locale: English/United States (en_US)
First off all, for the most normal operations (rectangle, round, plane) the transform tool works, but sometimes with more complex shapes i found some funny behaviors.
I managed in an old test build before, that all fillets and transform work in the file in the linked post, filename: Kappe50-jeno_edit.FCStd.
https://forum.freecad.org/viewtopic.php ... 40#p582840
This was OCCT7.4.0 with git commit 89b8e81d5c3b11572a0288720d0b4145d2ec2a7e
Now with retesting it with the same OCCT and commit, it do not work any more and i do not know why. Unfortunately while cleaning yesterday, i deleted the original build.
When i tested the PR, originally this also do not work. Then while clicking around the properties, i found out, there is property called Support Transform for features like fillets, chamfers, draft and thickness.
When i set it to true on Fillet001, then it works, but i did never changed it before, also on the test file (when you open it, you will see it).
When i also set it true on Fillet, then the transformation succeed, but the result is unexpected, because the transformation of the Fillet is not performed, which his also for the name of the property misleading.
So my question: what does the property? That is the purpose of it? I found nothing about that. Is the property incorrect implemented or normally deprecated all all?
Also in the PR brach version, the bace face do not refine, even when Refine = true. In this version, OCCT7.4.0 with git commit 89b8e81d5c3b11572a0288720d0b4145d2ec2a7e, it does.
Just also a remark, it no regression against 0.19, there this do not work at all. It is a regression about a dev-build. I do not know (and i do not rate) if it has an impact about this evaluation.
Fun fact: with OCCT7.6.1 the transformation do not work with the testfile.
Greetings
user1234
Re: 0.20 bug/regression Array Pattern issue #6641
I have tested Kappe50-jeno_edit.FCStd with current master build that includes my changes from yesterday. After opening the file I made a simple copy of PolarPattern and then recomputed it.
The only thing I realized is that the recomputed feature looks a bit smoother because of more triangles which you can see visually when activating the render mode "Hidden Lines". The reason for different number of triangles is I guess the different OCC version -- mine is 7.3.0.
However, the actual shape geometry is still the same. In order to find out any differences I switched to the Inspection workbench and made a visual inspection between the copied and recomputed polar pattern. In order to see any tiny differences in the color plot I had to change the range to [-5e-6, +5e-6], i.e. 5 microns.
Tooltip says:So my question: what does the property? That is the purpose of it? I found nothing about that. Is the property incorrect implemented or normally deprecated all all?
Inside the code it's used here https://github.com/FreeCAD/FreeCAD/blob ... p.cpp#L227 but its purpose is not totally clear to me.Include the base additive/subtractive shape when used in pattern features.
If disabled, only the dressed part of the shape is used for patterning.
Re: 0.20 bug/regression Array Pattern issue #6641
Does not work here:
Code: Select all
OS: Debian GNU/Linux 11 (bullseye) (X-Cinnamon/lightdm-xsession)
Word size of FreeCAD: 64-bit
Version: 0.20.28717 (Git)
Build type: Release
Branch: master
Hash: cfe94b00d3434f3e2c8b5114d2909f628c330fa8
Python 3.9.2, Qt 5.15.2, Coin 4.0.0, OCC 7.6.1
Locale: English/United States (en_US)
The result is only a small fillet surface, the rest is vanished.
This is why i ask, because the beavior is not like it should be. It seems like a depricaded property.wmayer wrote: ↑Wed Apr 13, 2022 3:07 pm Tooltip says:
Include the base additive/subtractive shape when used in pattern features.
If disabled, only the dressed part of the shape is used for patterning.
Inside the code it's used here https://github.com/FreeCAD/FreeCAD/blob ... p.cpp#L227 but its purpose is not totally clear to me.
Greetings
user1234
Re: 0.20 bug/regression Array Pattern issue #6641
With a OCC 7.6.0 build it fails to recompute the polar pattern. The error message is: Boolean operation failedThe result is only a small fillet surface, the rest is vanished.
Re: 0.20 bug/regression Array Pattern issue #6641
If the result is different based on the property, it can't be deprecated.
My latest (or last) project: B-spline Construction Project.
Re: 0.20 bug/regression Array Pattern issue #6641
Not sure of the term but yes if there's an adverse effect by the property it can go.
My latest (or last) project: B-spline Construction Project.