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

Redirection of spinlokd post #672

Open
wsteffe opened this issue Jan 5, 2023 · 5 comments
Open

Redirection of spinlokd post #672

wsteffe opened this issue Jan 5, 2023 · 5 comments
Labels

Comments

@wsteffe
Copy link

wsteffe commented Jan 5, 2023

Hi RT, have you seen this problem affecting LinkDaily ?
https://forum.freecadweb.org/viewtopic.php?f=44&p=651405#p651405

@realthunder realthunder added the occ label Jan 5, 2023
@realthunder
Copy link
Owner

I trace back the problem seems to stem from the second Pad feature. You can check the geometry of that feature. Make sure to enable BOP check. It'll report some problem.

image

It is known that BOP check is sometimes over sensitive to minor problems, but it is a clue. If I extended the sketch a bit, the problem goes away. Make sure you change the sketch constraint so that the top edges remains at the exact same place to avoid topo naming complain in later feature. There will sill be complain in some, but we are only concern about feature up to feature Bottom Attachment Screw Holes.

image

After the sketch modification, I can now add a hole to Sketch007 without problem

image

BTW. I've added a new auto fix function. And I've tested it that it can correct the problem without the sketch adjustment. I'll make a release soon.

@KangZhaoKui
Copy link

Hi,Lei
I think the key point is Sketch001 height form 73mm change to 89mm;
I don't know why, maybe it can give you some inspiration;

And i think Sketch005 have some issues and it doesn't meet the specification;

Mount-1.mp4
Mount-2.mp4

@spinlokd
Copy link

spinlokd commented Jan 7, 2023

I think the issue is not about the specific constraint in Sketch001, but more about forcing a recompute by making a change.
I have started by making the constraint change first and then try to add the additional hole. That didn't work. However when I went back and modified the Sketch001 again by deleting the constraint, it made the new hole appear.
Additionally, I tried to change Ring Reinforcement Cutout pocket to do "Up To Face" instead of "Through all". That also causes an issue, even if I cancel the change. When I executed Recompute action (at 1:07 mark), additional features became broken

I have attached a screen recording. Unfortunately it doesn't capture the right-click context menu.

2023-01-07.00-57-50.mp4

@wsteffe
Copy link
Author

wsteffe commented Jan 8, 2023

Hi spinlokd, you are right. And I think that it can not be an OCC problem.

The problem is better seen on a simpler test case in which fillets are removed:
MountSimple.zip

The problem is reproduced with following steps:

  1. Add a third circle in Sketch007:
    The holes are not made and geometry is corrupted.
  2. Set "Mark To Recompute" on Sketch001
  3. Apply a "Recompute Object" to Mount Body:
    The 3 holes are done and geometry is OK.

@realthunder
Copy link
Owner

Hmm... that's indeed the case. But there is even simpler way to fix it. Just recompute Support Ring immediately when open the file. And then adding third circle will be fine. It's not the first time I see this ghostly problem. There must be some problem buries some where and needs further investigation.

realthunder added a commit that referenced this issue Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants