Skip to content

Commit

Permalink
Merge pull request #345 from my-game-plan/bugfix/sportvu-coordinate-s…
Browse files Browse the repository at this point in the history
…ystem

[StatsPerform] Fix origin and vertical orientation of SportVU
  • Loading branch information
koenvo authored Aug 9, 2024
2 parents d001eb0 + e3e0b94 commit 016bd3a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions kloppy/domain/models/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -807,11 +807,11 @@ def provider(self) -> Provider:

@property
def origin(self) -> Origin:
return Origin.BOTTOM_LEFT
return Origin.TOP_LEFT

@property
def vertical_orientation(self) -> VerticalOrientation:
return VerticalOrientation.BOTTOM_TO_TOP
return VerticalOrientation.TOP_TO_BOTTOM

@property
def pitch_dimensions(self) -> PitchDimensions:
Expand Down
2 changes: 1 addition & 1 deletion kloppy/tests/test_statsperform.py
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ def test_correct_normalized_deserialization(
)

assert tracking_dataset.records[1].ball_coordinates == Point3D(
x=50.615 / 105, y=1 - 35.325 / 68, z=0.0
x=50.615 / 105, y=35.325 / 68, z=0.0
)

# Check normalised pitch dimensions
Expand Down

0 comments on commit 016bd3a

Please sign in to comment.