NBA-Player-Movements icon indicating copy to clipboard operation
NBA-Player-Movements copied to clipboard

Bug in y_pos of players and ball

Open dimitrisnikolaou10 opened this issue 7 years ago • 1 comments

Hey Kanstantsin,

There is a small bug in the y position of the players and the ball in the animation compared to reality. What we see as the y_pos in the animation is in fact the reflection of the y_pos. You can compare any play you have with highlights on youtube and you will understand what I mean.

A very easy fix would be to adjust the Ball.py and Player.py files as follows:

self.y = ball[3] --> self.y = Constant.Y_MAX - ball[3]

and

self.y = player[3] --> self.y = Constant.Y_MAX - player[3]

obviously, you will need to import the Constant file in both Ball and Player.

Great work otherwise, thanks for posting!

dimitrisnikolaou10 avatar Jul 05 '18 14:07 dimitrisnikolaou10

@dimitrisnikolaou10 hi, nice catch! Feel free to create a pull request.

linouk23 avatar Oct 07 '18 00:10 linouk23