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

Add an option to exclude invisible objects from floorplan generation #1069

Open
misslivirose opened this issue Jan 8, 2021 · 3 comments
Open

Comments

@misslivirose
Copy link
Contributor

Issue:
With the current behavior in Spoke, geometry that is marked as 'walkable' is included in the calculation for generating the floor plan. However, this can sometimes be undesirable behavior, since it complicates the floor plan and creators may not intentionally be including invisible meshes.

Proposed solution:
We should introduce a checkbox on the floorplan generation panel that says something like 'Include invisible meshes in floorplan'. This would decide whether or not invisible elements should be part of the calculation.

Our open question is whether or not we should change the default behavior. If we keep the default behavior the same, the check box would exclude invisible meshes. If we change the default behavior to exclude invisible meshes, the check box would be to include the invisible meshes. In the event that we change the default behavior, we might want to consider signaling that in some way so that we can let users know that the floorplan generation behavior has changed.

@brianpeiris
Copy link
Contributor

Another suggestion to avoid changing the default: Have an option or command to recursively toggle a subtree in the hierarchy as "walkable", or not.

@cvanmeurs
Copy link

I'd also add that I'd like the same to be done for collision (so that objects that are turned off are not calculated to generate the collision).

@robertlong
Copy link
Contributor

I think I've solved this with #1079

Let me know if that covers everything here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants