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

Unique plane issue with bmodels (see tokays) #617

Open
Paril opened this issue Feb 12, 2021 · 3 comments
Open

Unique plane issue with bmodels (see tokays) #617

Paril opened this issue Feb 12, 2021 · 3 comments
Assignees

Comments

@Paril
Copy link
Collaborator

Paril commented Feb 12, 2021

https://i.imgur.com/nv39IB8.jpeg

@jdolan-chwy
Copy link

image

A workaround is in place for this with the latest tokays.map. But yes, this should be addressed, and the old .map is a good test case.

@jdolan
Copy link
Owner

jdolan commented Feb 12, 2021

Demoting to minor, as this affects just one map, and there is an easy workaround.

jdolan added a commit that referenced this issue Apr 10, 2021
This fixes movers sometimes taking world planes for a ride (Tokays quad water).
@jdolan
Copy link
Owner

jdolan commented Dec 16, 2021

I actually think the problem here is that node faces are not cleared in between ProcessModels calls, or that MakeTreePortals doesn’t limit to the brushes of the current entity or something. I don’t think that using unique planes per entity will fix this. Planes are not part of bsp_model_t. Only nodes and faces are.

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

No branches or pull requests

3 participants