sportsball 0.3.55__tar.gz → 0.3.56__tar.gz

{sportsball-0.3.55/sportsball.egg-info → sportsball-0.3.56}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: sportsball
-Version: 0.3.55
+Version: 0.3.56
 Summary: A library for pulling in and normalising sports stats.
 Home-page: https://github.com/8W9aG/sportsball
 Author: Will Sackfield

{sportsball-0.3.55 → sportsball-0.3.56}/setup.py

@@ -26,7 +26,7 @@ def install_requires() -> typing.List[str]:
 
 setup(
     name='sportsball',
-    version='0.3.55',
+    version='0.3.56',
     description='A library for pulling in and normalising sports stats.',
     long_description=long_description,
     long_description_content_type='text/markdown',

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/__init__.py

@@ -1,3 +1,3 @@
 """The main module for sportsball."""
 
-__VERSION__ = "0.3.55"
+__VERSION__ = "0.3.56"

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/afl/afl/afl_afl_player_model.py

@@ -21,4 +21,5 @@ def create_afl_afl_player_model(
         assists=None,
         turnovers=None,
         name=name,
+        marks=None,
     )

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/afl/afltables/afl_afltables_game_model.py

@@ -1,6 +1,6 @@
 """AFL AFLTables game model."""
 
-# pylint: disable=too-many-arguments
+# pylint: disable=too-many-arguments,too-many-branches
 import datetime
 import urllib.parse
 from urllib.parse import urlparse
@@ -152,7 +152,7 @@ def _create_afl_afltables_game_model(
 
     def _find_teams_metadata(
         soup: BeautifulSoup, team_infos: list[tuple[str, str, int]]
-    ) -> list[tuple[str, list[tuple[str, str, int | None, str]], int]]:
+    ) -> list[tuple[str, list[tuple[str, str, int | None, str, int | None]], int]]:
         def _is_correct_table(table: Tag, name: str) -> bool:
             for th in table.find_all("th"):
                 header_text = th.get_text().strip()
@@ -160,8 +160,10 @@ def _create_afl_afltables_game_model(
                     return True
             return False
 
-        def _find_players(table: Tag) -> list[tuple[str, str, int | None, str]]:
-            players: list[tuple[str, str, int | None, str]] = []
+        def _find_players(
+            table: Tag,
+        ) -> list[tuple[str, str, int | None, str, int | None]]:
+            players: list[tuple[str, str, int | None, str, int | None]] = []
             for tr in table.find_all("tr"):
                 player_row = False
                 player_url = None
@@ -174,6 +176,7 @@ def _create_afl_afltables_game_model(
                     jersey = None
                     name = None
                     kicks = None
+                    marks = None
                     for count, td in enumerate(tr.find_all("td")):
                         if count == 0:
                             jersey = td.get_text().strip()
@@ -188,13 +191,17 @@ def _create_afl_afltables_game_model(
                             kicks_text = td.get_text().strip()
                             if kicks_text:
                                 kicks = int(kicks_text)
+                        elif count == 3:
+                            marks_text = td.get_text().strip()
+                            if marks_text:
+                                marks = int(marks_text)
 
                     if jersey is None:
                         raise ValueError("jersey is null.")
                     if name is None:
                         raise ValueError("name is null")
 
-                    players.append((player_url, jersey, kicks, name))
+                    players.append((player_url, jersey, kicks, name, marks))
             return players
 
         team_metadata = []

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/afl/afltables/afl_afltables_player_model.py

@@ -13,7 +13,7 @@ from ...player_model import PlayerModel
 
 
 def _create_afl_afltables_player_model(
-    player_url: str, jersey: str, kicks: int | None, name: str
+    player_url: str, jersey: str, kicks: int | None, name: str, marks: int | None
 ) -> PlayerModel:
     o = urlparse(player_url)
     last_component = o.path.split("/")[-1]
@@ -31,14 +31,15 @@ def _create_afl_afltables_player_model(
         assists=None,
         turnovers=None,
         name=name,
+        marks=marks,
     )
 
 
 @MEMORY.cache
 def _cached_create_afl_afltables_player_model(
-    player_url: str, jersey: str, kicks: int | None, name: str
+    player_url: str, jersey: str, kicks: int | None, name: str, marks: int | None
 ) -> PlayerModel:
-    return _create_afl_afltables_player_model(player_url, jersey, kicks, name)
+    return _create_afl_afltables_player_model(player_url, jersey, kicks, name, marks)
 
 
 def create_afl_afltables_player_model(
@@ -48,6 +49,7 @@ def create_afl_afltables_player_model(
     dt: datetime.datetime,
     session: requests_cache.CachedSession,
     name: str,
+    marks: int | None,
 ) -> PlayerModel:
     """Create a player model from AFL Tables."""
     if (
@@ -59,6 +61,9 @@ def create_afl_afltables_player_model(
             jersey,
             kicks,
             name,  # pyright: ignore
+            marks,
         )
     with session.cache_disabled():
-        return _create_afl_afltables_player_model(player_url, jersey, kicks, name)
+        return _create_afl_afltables_player_model(
+            player_url, jersey, kicks, name, marks
+        )

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/afl/afltables/afl_afltables_team_model.py

@@ -44,7 +44,7 @@ _TEAM_NAME_MAP = {
 
 def _create_afl_afltables_team_model(
     team_url: str,
-    players: list[tuple[str, str, int | None, str]],
+    players: list[tuple[str, str, int | None, str, int | None]],
     points: float,
     session: requests_cache.CachedSession,
     last_ladder_ranks: dict[str, int] | None,
@@ -72,9 +72,9 @@ def _create_afl_afltables_team_model(
         name=name,
         players=[  # pyright: ignore
             create_afl_afltables_player_model(
-                player_url, jersey, kicks, dt, session, name
+                player_url, jersey, kicks, dt, session, name, marks
             )
-            for player_url, jersey, kicks, name in players
+            for player_url, jersey, kicks, name, marks in players
         ],
         odds=[],
         points=points,
@@ -89,7 +89,7 @@ def _create_afl_afltables_team_model(
 @MEMORY.cache(ignore=["session"])
 def _cached_create_afl_afltables_team_model(
     team_url: str,
-    players: list[tuple[str, str, int | None, str]],
+    players: list[tuple[str, str, int | None, str, int | None]],
     points: float,
     session: requests_cache.CachedSession,
     last_ladder_ranks: dict[str, int] | None,
@@ -97,13 +97,19 @@ def _cached_create_afl_afltables_team_model(
     league: League,
 ) -> TeamModel:
     return _create_afl_afltables_team_model(
-        team_url, players, points, session, last_ladder_ranks, dt, league
+        team_url,
+        players,
+        points,
+        session,
+        last_ladder_ranks,
+        dt,
+        league,
     )
 
 
 def create_afl_afltables_team_model(
     team_url: str,
-    players: list[tuple[str, str, int | None, str]],
+    players: list[tuple[str, str, int | None, str, int | None]],
     points: float,
     session: requests_cache.CachedSession,
     last_ladder_ranks: dict[str, int] | None,

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/combined/combined_player_model.py

@@ -1,6 +1,6 @@
 """Combined player model."""
 
-# pylint: disable=too-many-locals
+# pylint: disable=too-many-locals,too-many-branches
 from ..player_model import PlayerModel
 
 
@@ -18,6 +18,7 @@ def create_combined_player_model(
     assists = None
     turnovers = None
     name = None
+    marks = None
     for player_model in player_models:
         player_model_jersey = player_model.jersey
         if player_model_jersey is not None:
@@ -49,6 +50,9 @@ def create_combined_player_model(
         player_model_name = player_model.name
         if player_model_name is not None:
             name = player_model_name
+        player_model_marks = player_model.marks
+        if player_model_marks is not None:
+            marks = player_model_marks
     if name is None:
         raise ValueError("name is null.")
     return PlayerModel(
@@ -63,4 +67,5 @@ def create_combined_player_model(
         assists=assists,
         turnovers=turnovers,
         name=name,
+        marks=marks,
     )

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/espn/espn_player_model.py

@@ -45,6 +45,7 @@ def _create_espn_player_model(
         assists=None,
         turnovers=None,
         name=name,
+        marks=None,
     )
 
 

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/google/google_address_model.py

@@ -12661,6 +12661,16 @@ _CACHED_GEOCODES: dict[str, Any] = {
         housenumber="",
         country="USA",
     ),
+    "Monroe, Louisiana": SportsballGeocodeTuple(
+        city="Monroe",
+        state="LA",
+        postal="",
+        lat=32.509444,
+        lng=-92.118333,
+        housenumber="",
+        country="USA",
+    ),
+    "Bison Sports Arena, Fargo, North Dakota": SCHEELS_ARENA,
 }
 
 

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/player_model.py

@@ -15,6 +15,7 @@ FIELD_GOALS_ATTEMPTED_COLUMN: Literal["field_goals_attempted"] = "field_goals_at
 OFFENSIVE_REBOUNDS_COLUMN: Literal["offensive_rebounds"] = "offensive_rebounds"
 ASSISTS_COLUMN: Literal["assists"] = "assists"
 TURNOVERS_COLUMN: Literal["turnovers"] = "turnovers"
+PLAYER_MARKS_COLUMN: Literal["marks"] = "marks"
 
 
 class PlayerModel(BaseModel):
@@ -67,3 +68,8 @@ class PlayerModel(BaseModel):
         alias=TURNOVERS_COLUMN,
     )
     name: str
+    marks: int | None = Field(
+        ...,
+        json_schema_extra={TYPE_KEY: FieldType.LOOKAHEAD},
+        alias=PLAYER_MARKS_COLUMN,
+    )

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/sportsreference/sportsreference_game_model.py

@@ -210,12 +210,15 @@ def _find_old_dt(
         dt = parse(date)
         for df in dfs:
             test_row = df.iat[0, 0]
+            if isinstance(test_row, float):
+                continue
             try:
                 if "Prev Game" in test_row:
                     _process_team_row(df)
                     break
             except TypeError as exc:
                 logging.error(test_row)
+                logging.error(response.url)
                 logging.error(response.text)
                 raise exc
 

{sportsball-0.3.55 → sportsball-0.3.56}/sportsball/data/sportsreference/sportsreference_player_model.py

@@ -78,6 +78,7 @@ def _create_sportsreference_player_model(
         assists=assists.get(name),
         turnovers=turnovers.get(name),
         name=name,
+        marks=None,
     )
 
 

{sportsball-0.3.55 → sportsball-0.3.56/sportsball.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: sportsball
-Version: 0.3.55
+Version: 0.3.56
 Summary: A library for pulling in and normalising sports stats.
 Home-page: https://github.com/8W9aG/sportsball
 Author: Will Sackfield

{sportsball-0.3.55 → sportsball-0.3.56}/tests/data/afl/afltables/afl_afltables_player_model_test.py

@@ -24,5 +24,6 @@ class TestAFLTablesPlayerModel(unittest.TestCase):
                 dt,
                 self._session,
                 "Dennis Armfield",
+                0,
             )
             self.assertEqual(player_model.identifier, "Dennis_Armfield")

{sportsball-0.3.55 → sportsball-0.3.56}/tests/data/combined/combined_player_model_test.py

@@ -30,6 +30,7 @@ class TestCombinedPlayerModel(unittest.TestCase):
                 assists=None,
                 turnovers=None,
                 name="James Jones",
+                marks=None,
             )
             combined_player_model = create_combined_player_model(
                 [player_model],

{sportsball-0.3.55 → sportsball-0.3.56}/tests/data/player_model_test.py

@@ -19,6 +19,7 @@ class TestPlayerModel(unittest.TestCase):
             assists=None,
             turnovers=None,
             name="James Jones",
+            marks=None,
         )
 
     def test_field_goals_attempted(self):

{sportsball-0.3.55 → sportsball-0.3.56}/tests/data/sportsreference/sportsreference_game_model_test.py

@@ -1005,3 +1005,17 @@ class TestSportsReferenceGameModel(unittest.TestCase):
                 datetime.datetime(2010, 10, 10, 10, 10, 0),
             )
             self.assertEqual(game_model.dt, datetime.datetime(2016, 3, 24, 0, 0))
+
+    def test_dt_old_style_20(self):
+        url = "https://www.sports-reference.com/cbb/boxscores/2016-02-23-savannah-state.html"
+        with requests_mock.Mocker() as m:
+            with open(os.path.join(self.dir, "2016-02-23-savannah-state.html"), "rb") as f:
+                m.get(url, content=f.read())
+
+            game_model = create_sportsreference_game_model(
+                self.session,
+                url,
+                League.NBA,
+                datetime.datetime(2010, 10, 10, 10, 10, 0),
+            )
+            self.assertEqual(game_model.dt, datetime.datetime(2016, 2, 23, 0, 0))