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

Issues with switched NPC( 'g' key) #3397

Open
alisaad673 opened this issue Jul 28, 2022 · 3 comments
Open

Issues with switched NPC( 'g' key) #3397

alisaad673 opened this issue Jul 28, 2022 · 3 comments
Assignees

Comments

@alisaad673
Copy link
Contributor

while we press 'G' key and switch to the nearest NPC, I noticed following issues:

1- "E" activator keeps appearing over the npc, while we switch to it.

2022-07-28.18-58-47.mp4

2-The jump animation of switched npc is too fast.

2022-07-28.19-00-21.mp4

3-While we press "E" on the switched NPC, it restricts the movement of npc inside a kind of loop around the avatar.

2022-07-28.19-01-18.mp4

4-While we switch to npc from avatar, the avatar is left just in the state it was switched from.

2022-07-28.19-03-16.mp4
@avaer
Copy link
Contributor

avaer commented Aug 8, 2022

I think the bug is that avatar updates for switched-to NPCs are run twice.

@jakezira
Copy link
Contributor

jakezira commented Aug 13, 2022

Solution

  • When npc joins the party, it should be moved to avatar's app manager from scene app manager.
  • Physics tracker needs to track all app manager including scene and avatar's. Actually, manager for physics tracking is preferable ==> Physics tracker manager #3614

Avatar Switching (G)

  • You can only switch to avatar who already joined the party, not from npcs from the world.

@jakezira jakezira linked a pull request Aug 15, 2022 that will close this issue
4 tasks
@jakezira
Copy link
Contributor

jakezira commented Aug 15, 2022

Party System

Related Pull Requests

Video

Notes

  • Fixed of all 4 bugs
  • Party is up to 3 members, including the main character. When player activates the npc, it joins party.
  • Party members follows a main character in a line, instead of all members heading to main character at the same time.
  • Default avatar can't be removed from party.
    • Only npcs can be invited to and removed from the party.
    • We have fixed npc pool, and main avatar will not be added into npc pool.

Update

  • Added party manager which changes the actions and collects the members
    • Added action api to party manager. Supported features are switchcharacter, addplayer and removeplayer.
    • Player class should have no idea about parties
  • Removed dependency to avatar app on party manager

Test cases

  • Open a scene, with at least 2 npcs.
  • Activate one npc. See if it follows player.
  • Deactivate the following character. See if it stops following.
  • Activate another npc.
  • Press G to swap to new character.
  • Check if new character can interact with other character and npc.
  • Activate another remaining npc.
  • Run the player and see if all characters are following in a line.
  • Keep switching characters.
  • Do jump and switch to another character instantly. All characters should move smoothly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants
@avaer @jakezira @alisaad673 and others