-
Notifications
You must be signed in to change notification settings - Fork 45
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
[BUG] Shell point and click references the wrong feature #4961
Comments
Hey @nicboone8 thanks for the report I can look into that. For your second note on multiple faces, can you elaborate? It should support 'cap' and 'wall' type of faces. We still have #4775 open that should better catch selection issues once merged (right now the flow can silently fail). |
Thanks @pierremtb! Selecting multiple faces is working perfectly for me now. I'll keep trying to replicate what I was seeing yesterday. Basically every time I selected a second face, the first face would de-select |
@nicboone8 Any chance you're pressing shift today but weren't yesterday? |
That's always possible 😝. I thought it might be because I was working on a more complex part, but maybe I only tried CTRL and ALT but not shift |
Here's something interesting. I can select multiple faces without using any keyboard buttons when the part includes a 2d mirror Zoo.Modeling.App.2025-01-08.14-21-21.mp4 |
@nicboone8 Oh that's a great find. Can confirm that with mirror2d present the selection becomes 'sticky' without any key pressed. In and out of the Shell command too |
I believe this happens in the case of an artifact graph not fully populated on the frontend. In this case the engine returns an id on select that we can't map to anything in the artifact graph. It's the same for lofts today on main, before #4898 is merged. But that PR is loft specific w.r.t the artifact graph build up |
@nicboone8 Ok I can reproduce your first issue: happens only when extrude is in the sketch pipe, not when it's broken out as a variable. See below. |
I think this might be an easy fix- the output from the shell button just needs to reference the selected feature instead of the default one.
Also I can't select multiple faces when creating a shell
Zoo.Modeling.App.2025-01-07.10-30-48.mp4
Core Dump
coredump-af3cd7bd-54c3-492f-9236-9e4bdebf455a.json
Reference ID: af3cd7bd-54c3-492f-9236-9e4bdebf455a
KCL Code
The text was updated successfully, but these errors were encountered: