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

Problem: Toponaming fails with index error #14129

Open
2 tasks done
sliptonic opened this issue May 18, 2024 · 2 comments
Open
2 tasks done

Problem: Toponaming fails with index error #14129

sliptonic opened this issue May 18, 2024 · 2 comments
Labels
Bug This issue or PR is related to a bug Toponaming
Milestone

Comments

@sliptonic
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Problem description

testfile attached. rename tntest.zip to tntest.fcstd

This is a straightforward test case. Open the sketch 'stretchme' and drag the right edge over.

Peek 2024-05-17 18-05
tntest.zip

I see the following in the console

18:05:04  <PropertyLinks> PropertyLinks.cpp(453): tntest#Sketch002.Support missing element reference tntest#Pocket ;g11;SKT;:H1db,E;FAC;:H1db:4,F;:G0;XTR;:H1db:8,F;:H-1de,F.Face11
18:05:04  Sketch002: Shape index 11 out of bound 10
18:05:04  Sketch002: Shape index 11 out of bound 10

Full version info

OS: Linux Mint 21 (i3/i3)
Word size of FreeCAD: 64-bit
Version: 0.22.0dev.37360 (Git)
Build type: Unknown
Branch: main
Hash: 2d2977abafd93c9ab97f9ad7beb25f5311c50b06
Python 3.10.12, Qt 5.15.3, Coin 4.0.0, Vtk 7.1.1, OCC 7.5.1
Locale: English/United States (en_US)
Installed mods: 
  * OpenDark 2023.12.17
  * fasteners 0.5.15
  * BillOfMaterials 0.0.7.2
  * sheetmetal 0.4.9
  * Ondsel-Lens 2024.1.15.01
  * FeedsAndSpeeds 0.5.0

Subproject(s) affected?

None

Anything else?

Compiled with toponaming enabled.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@maxwxyz maxwxyz added the Bug This issue or PR is related to a bug label May 18, 2024
@maxwxyz
Copy link
Collaborator

maxwxyz commented May 18, 2024

@bgbsww FYI

@chennes chennes added this to the 1.0 milestone May 18, 2024
@bgbsww
Copy link
Contributor

bgbsww commented May 20, 2024

So, after cleaning up other issues, we now seem to be matching the LS3 behavior. In both cases, if you stretch the edge of the sketch a pocket is based on to outside of the supporting face, it doesn't redraw, you get console errors, and if selected, the selection shows a history name:

LS3:
16:09:29 PropertyLinks.cpp(439): tntest#Sketch002.Support missing element reference tntest#Pocket ;#60;:H-1de,F.Face11
16:09:29 6.5e-08 Document.cpp(3424): Failed to recompute tntest#Sketch002: AttachEngine3D: subshape not found Pocket.Face11
16:09:29 Recompute failed!

Main:
16:07:55 PropertyLinks.cpp(453): tntest#Sketch002.Support missing element reference tntest#Pocket ;g11;SKT;:H1db,E;FAC;:H1db:4,F;:G0;XTR;:H1db:8,F;:H-1de,F.Face11
16:07:55 Sketch002: Shape index 11 out of bound 10
16:07:55 Sketch002: Shape index 11 out of bound 10

And in both cases, moving the pocket edge back in bounds works and seems to clean the maps correctly. This is after #14178 is in place.

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 Toponaming
Projects
Development

No branches or pull requests

4 participants