[bug] Pocket through all

[kisolre]

Recently I noticed at least twice PartDesign pocket Through all failing to produce proper pocket. This is with quite simple models. Same pocket with length works as expected:
.

PocketThroughAll.JPG (44.88 KiB) Viewed 979 times

PocketLength.JPG (33.74 KiB) Viewed 979 times

.
The two circular holes are also through all but thew work as expected.
OS: Windows 8.1 (6.3)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.21501 (Git)
Build type: Release
Branch: master
Hash: 457dba4fee90410d682f47300afda6e0a10c4fc1
Python version: 3.6.6
Qt version: 5.12.1
Coin version: 4.0.0a
OCC version: 7.3.0
Locale: Bulgarian/Bulgaria (bg_BG)

[Willem]

Checking the dependency grtaph I see "Red Lines "

Schermafdruk van 2020-06-06 14-14-30.png (251.6 KiB) Viewed 971 times

[kisolre]

Checking the dependency grtaph I see "Red Lines "

@wmayer is modifying the Origin handling but this should not affect the way pocket is created.

[wmayer]

Checking the dependency grtaph I see "Red Lines "

@wmayer is modifying the Origin handling but this should not affect the way pocket is created.

This is unrelated to my recent changes. When I load this project with v0.19 then it gets loaded correctly and the dependency graph doesn't show anything abnormal.

However, when I load it with v0.18 then it crashes straight away. I had to comment out a line "assert(0)". Afterwards I could load it and it shows the exact same problems as in Willem's screen shot. This means the tree view is messed up and the dependency graph shows the red lines.

Additionally in the output window I get these errors:

Code: Select all

Invalid Document.xml: End of document reached
No extension found of type 'PartGui::ViewProviderAttachExtension'
No extension found of type 'PartGui::ViewProviderAttachExtension'
No extension found of type 'PartGui::ViewProviderAttachExtension'
No extension found of type 'PartGui::ViewProviderAttachExtension'
No extension found of type 'PartGui::ViewProviderAttachExtension'
No extension found of type 'PartGui::ViewProviderAttachExtension'

The issue about the missing extension is probably not that dramatic but the error message "Invalid Document.xml: End of document reached" seems more serious. The crash above happened inside XMLReader::getAttributeAsInteger().

So, obviously here we have created a Document.xml with v0.19 that cannot be loaded with v0.18.

Anyway, your actual problem with the pocket is something else. Please check the bug tracker if you can find a reported issue about this problem, otherwise create a ticket.

[kisolre]

When I load this project with v0.19 then it gets loaded correctly and the dependency graph doesn't show anything abnormal.

That file was created and saved after recent origin modifications. It displays same broken dependencies graph with
OS: Windows 8.1 (6.3)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20839 (Git)
Build type: Release
Branch: master
Hash: 09005766f10539dfe32314c33c2cd3efe9141d0b
Python version: 3.6.8
Qt version: 5.12.1
Coin version: 4.0.0a
OCC version: 7.3.0
Locale: Bulgarian/Bulgaria (bg_BG)

[wmayer]

That file was created and saved after recent origin modifications. It displays same broken dependencies graph with

But this doesn't automatically mean that my recent changes are the reason for it. I have locally reverted my recent changes and I still could load your file. So, IMO the reason must be the changes between 0.19.20839 and 0.19.21501, so 662 possibilities.

Do you mind to create your model from scratch with 0.19.20839? Then I can have a look at the differences of the Document.xml files and see what are the important changes.

[papyblaise]

When I tray to open your file : Unomed is empty , so Pad isn't legal
Sketch003 and 004 are lost
But , I can't cofirm , it seam that sketch003 is coplanar whith the face

[freedman]

I see the same here in 0.19.204 when I open the file, a bunch of Unnamed origins at the bottom. I can try to recreate the model if needed....

OS: Windows 10 (10.0)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20477 (Git)
Build type: Release
Branch: master
Hash: 7d8ab175a68e476427e6e337fe3099b52da49dc8
Python version: 3.6.8
Qt version: 5.12.1
Coin version: 4.0.0a
OCC version: 7.3.0
Locale: English/United States (en_US)

[TheMarkster]

I can confirm with a new build from source. Works as expected if using length or up to face. When unchecking reverse mode it works to remove some material, but then fails. Works okay if I use a subtractive box to create a flat edge for it to detect. Problem seems to be in detecting the curved edge.

Snip macro screenshot-c38dec.png (127.92 KiB) Viewed 869 times

Snip macro screenshot-6cf5c6.png (155.18 KiB) Viewed 869 times

OS: Ubuntu 18.04.4 LTS (LXDE/LXDE)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.21505 (Git)
Build type: Debug
Branch: master
Hash: 9801519409c992fb964490abdefc35ae79e0b473
Python version: 3.6.9
Qt version: 5.9.5
Coin version: 4.0.0a
OCC version: 7.3.0
Locale: English/UnitedStates (en_US)

[wmayer]

In the meantime I did a manual git bisect and up to git commit 1521b256bcfb FreeCAD crashes when loading the file. NOTE, that this already includes the change to add group behaviour to the origin feature.

After applying git commit 457dba4fee9 FreeCAD doesn't crash any more. My intention was to avoid that the App::Origin doesn't export further properties but somehow this causes a regression and possibly invalidates the Document.xml file. So, I will revert this change.

But nevertheless the (broken?) project file is a good example that FreeCAD should also handle such input without crashing.