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

Fix: JA+ server modelscale prediction errors #387

Open
Razish opened this issue May 15, 2016 · 8 comments
Open

Fix: JA+ server modelscale prediction errors #387

Razish opened this issue May 15, 2016 · 8 comments

Comments

@Razish
Copy link
Owner

Razish commented May 15, 2016

on JA+ servers with modelscale set (e.g. jan on KR server), there are prediction errors when standing on their head

@Razish
Copy link
Owner Author

Razish commented May 16, 2016

Ping @Blackwolf1337 did you want to look at this before me :)?

@Exmirai
Copy link
Collaborator

Exmirai commented May 16, 2016

bboxes are not updated?

@ensiform
Copy link
Collaborator

I don't think they are in JA+ either.

@Blackwolf1337
Copy link
Contributor

@Razish I'll take a look at it.

@Blackwolf1337
Copy link
Contributor

Ok, I looked over the code and it seems like there is a prediction... the fault is coming from the Ja+ mod itself, since it's not scaling the Boundary Box as @EpicLoyd asked and said. So it is irrelevant to fix it, since it has to do something Serverside. Since the Size Prediction is built in, it bugs for us because the game thinks, that the model is scaled with the boundary box, which isn't. I think this can be closed @Razish ?

@Razish
Copy link
Owner Author

Razish commented May 17, 2016

If we are getting prediction errors, the bounding boxes are out of sync with the server.
Either we are updating them with the modelscale and JA+ server is not, or we are not updating them with the modelscale and JA+ is. Right?

@Exmirai
Copy link
Collaborator

Exmirai commented May 17, 2016

afaik ja+ not updating it because if resulting bbox ( after scaling ) will be too high or too wide that can block whole map or significant part.

@Blackwolf1337
Copy link
Contributor

@Razish, that's totally right. But your ja++ Plugin already has a prediction for it, when scaling models up and down you're getting the prediction. It's just Ja+.

@Razish Razish changed the title modelscale prediction errors Fix: JA+ server modelscale prediction errors May 23, 2017
@Razish Razish added this to tracking Oct 9, 2023
@Razish Razish moved this to polish in tracking Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: polish
Development

No branches or pull requests

4 participants