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

Exosuit shoot pose rendering bug #305

Closed
NimmerNeko opened this issue Aug 25, 2015 · 6 comments
Closed

Exosuit shoot pose rendering bug #305

NimmerNeko opened this issue Aug 25, 2015 · 6 comments

Comments

@NimmerNeko
Copy link

How to achieve it:

  1. Put on an exosuit.
  2. Take a musket and charge a Shot
  3. Switch slot to Cancel Shot

What happens:

Arms get stuck in the shooting pose.

@elifoster elifoster changed the title Exosuit Rendering Bug Exosuit shoot pose rendering bug Aug 25, 2015
@elifoster
Copy link
Member

Bug may have been fixed in @6a13037

@elifoster elifoster added this to the 0.28.11 milestone Aug 25, 2015
@elifoster elifoster removed the Status: Unconfirmed Has not been approved label Aug 26, 2015
@elifoster
Copy link
Member

This bug is still present after the fix for the other Exosuit rendering bug.

@elifoster
Copy link
Member

What I'm thinking currently is tht the bug could be fixed by checking if the player is wearing an exosuit in ItemFirearm.onPlayerStoppedUsing. If true, force a re-render.

However this may work, special-casing is not a good way to handle things, so I will continue doing research.

@elifoster elifoster modified the milestones: 0.29.0, 0.28.11 Aug 28, 2015
@elifoster elifoster added the Status: Approved Has been approved and is ready to be implemented, or has been confirmed to be a bug label Oct 19, 2015
@elifoster
Copy link
Member

I have come up with another potential solution: on the ArrowLooseEvent, if it is a Firearm shot (or Rocket Launcher, will need to test), force a re-render. This is sort of a workaround, which isn't very good. I really should figure out the root cause of this issue. It's not the EnumActions as those are properly set by the model.

Here's what we discussed on November 12 in IRC, as noted by my personal notes: "We manually set armor rotation for the exosuit for sitting, standing, bow aiming, etc. We may need to special-case it. Look at onPlayerStoppedUsing in ItemBow for differences"

@Flaxbeard
Copy link
Contributor

I know how to fix this bug - poke me in IRC, I'll be on in like 2 hours. It has to do with caching the suit and associated issues.

Unrelated, but a lot of exo parts don't render now at least in my dev environment, including the top hat.

@elifoster
Copy link
Member

I haven't noticed that. I'll test it out at some point today

@elifoster elifoster removed the Status: Approved Has been approved and is ready to be implemented, or has been confirmed to be a bug label Nov 25, 2015
@elifoster elifoster added the Side: Client Only affects the client label Dec 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Bugs
Resolved
Development

No branches or pull requests

3 participants