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

Don't leave bots behind in SE #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jmerdich
Copy link

Space Exploration has a navigation mode that creates a fake
player on other surfaces to view them from a 'satellite'. In the
existing code, any time that there isn't a character body, the robots
don't follow. Unfortunately, there also isn't a body for some cutscenes
that happen on transitions to other worlds.

Use the sanctioned interface to check if we're using the sat view vs
actually arriving somewhere else in a cutscene.

There are more sat view bugs, but not leaving the bots behind makes
them less likely to occur.

Space Exploration has a navigation mode that creates a fake
player on other surfaces to view them from a 'satellite'. In the
existing code, any time that there isn't a character body, the robots
don't follow. Unfortunately, there also isn't a body for some cutscenes
that happen on transitions to other worlds.

Use the sanctioned interface to check if we're using the sat view vs
actually arriving somewhere else in a cutscene.

There are more sat view bugs, but not leaving the bots behind makes
them less likely to occur.
@jmerdich jmerdich changed the title Add a fallback for players moving surfaces Don't leave bots behind in SE Jul 15, 2022
@jmerdich
Copy link
Author

jmerdich commented Aug 3, 2022

So.... this fixes one issue, but there seem to be a number of other SE-related issues with the navigation views which will likely require larger changes-- either to grab the actual character location regardless of view or otherwise disable the bots while in that state so they don't sync position on their surface with what the player is viewing on another.

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

Successfully merging this pull request may close these issues.

1 participant