Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Issue] Extensions missing edges in some cases #8915

Closed
2 tasks done
jffmichi opened this issue Mar 16, 2023 · 1 comment · Fixed by #10033
Closed
2 tasks done

[Issue] Extensions missing edges in some cases #8915

jffmichi opened this issue Mar 16, 2023 · 1 comment · Fixed by #10033
Labels
Bug This issue or PR is related to a bug WB CAM Related to the CAM/Path Workbench

Comments

@jffmichi
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Forums discussion

https://forum.freecad.org/viewtopic.php?t=76828

Version

0.21 (Development)

Full version info

[code]
OS: Arch Linux (KDE/plasmawayland)
Word size of FreeCAD: 64-bit
Version: 0.21.0.32246 (Git)
Build type: None
Branch: master
Hash: f1ffaf3ba9a1cdf408334383e0ba5c6767919b2c
Python 3.10.10, Qt 5.15.8, Coin 4.0.1, Vtk 9.2.6, OCC 7.6.3
Locale: German/Germany (de_DE)
Installed mods: 
  * dodo 1.0.0
  * sheetmetal 0.2.59
  * fcgear 1.0.0
  * fasteners 0.4.24
  * fcgear.backup1663629032.4145591 1.0.0 (Disabled)
  * Assembly4 0.12.4
  * lattice2 1.0.0
[/code]

Subproject(s) affected?

Path

Issue description

Hello everyone,

The extension feature for Path_Pocket_Shape or Path_Adaptive misses edges in some cases (see attached screenshot and example project file). The edge should be extended all around the selected face.

This is how @Russ4262 describes the origin of the issue:
"The missing edge is Edge7. It gets removed from the wire list when the call to 'edgesMatchShape()' fails to flip Edge7 properly. I ran into this a while back and tracked it to Path.Geom code. This Path.Geom bug causes extensions to fail in other instances as well (how I found it)."

Thank you all for the great work!

Anything else?

extensions_missing_edge

extensions_missing_edge.zip

Code of Conduct

  • I agree to follow this project's Code of Conduct
@luzpaz luzpaz added Bug This issue or PR is related to a bug WB CAM Related to the CAM/Path Workbench labels Mar 18, 2023
@luzpaz luzpaz assigned mlampert and unassigned mlampert Mar 18, 2023
@luzpaz
Copy link
Contributor

luzpaz commented Aug 6, 2023

@sliptonic is this on your radar ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This issue or PR is related to a bug WB CAM Related to the CAM/Path Workbench
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants