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

Disable r_portal_fastpath in P1 maps #4083

Closed
vrad-exe opened this issue Feb 14, 2022 · 6 comments
Closed

Disable r_portal_fastpath in P1 maps #4083

vrad-exe opened this issue Feb 14, 2022 · 6 comments
Labels
Bug Something isn't working correctly Done in next release This has been addressed in the next release Portal 1 Issue only applies to Portal 1 style.

Comments

@vrad-exe
Copy link
Member

With P1 now having doorless entry mode, it's possible in rare cases for more than 4 portals to be visible onscreen - if the entrance and exit worldportals, corridor 4 obs room worldportal, and both player portals are all visible at once. This is only made worse by the world portal item UCP (and #574 if we ever make up our mind on that). We've had fastpath disabled by default for a little while in P2CE and it hasn't caused any problems, so it'd probably be good to do that in BEE P1 maps. Perhaps Old Aperture as well since it has the large obs room worldportal, and I might add a doorless entry to 70s later on.

@vrad-exe vrad-exe added Bug Something isn't working correctly Portal 1 Issue only applies to Portal 1 style. labels Feb 14, 2022
@ErinRoseWebs69
Copy link

maybe just disable it entirely via the arrival logic, and re-enable it via the departure logic? would be better, especially with the World Portals UCP.

@vrad-exe
Copy link
Member Author

Actually yeah, not sure why we need to limit it to P1. Probably won't re-enable it at the end of maps, there's no way to make sure that happens if the player quits the map early and as I said keeping it off hasn't caused any issues. It doesn't persist across game restarts, so it'll eventually get turned back on anyway unless you've got it in your autoexec.

@ErinRoseWebs69
Copy link

fair. but yeah, turning it off via the arrival elevator logic would definitely help.

@ErinRoseWebs69
Copy link

if yall are busy, i can add it to the current logics for ya. its a simple addon.

@ErinRoseWebs69
Copy link

hell, this lil thing could be added to the transition ents thats included in all maps. means i only need to edit one instance.

@vrad-exe vrad-exe added the Done in next release This has been addressed in the next release label Feb 22, 2022
@ErinRoseWebs69
Copy link

oh i just now noticed this got the Done label. nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working correctly Done in next release This has been addressed in the next release Portal 1 Issue only applies to Portal 1 style.
Projects
None yet
Development

No branches or pull requests

2 participants