sportsball 0.3.127__tar.gz → 0.4.0__tar.gz

{sportsball-0.3.127/sportsball.egg-info → sportsball-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: sportsball
-Version: 0.3.127
+Version: 0.4.0
 Summary: A library for pulling in and normalising sports stats.
 Home-page: https://github.com/8W9aG/sportsball
 Author: Will Sackfield
@@ -16,7 +16,7 @@ Requires-Dist: requests>=2.31.0
 Requires-Dist: requests-cache>=1.2.0
 Requires-Dist: python-dateutil>=1.16.0
 Requires-Dist: tqdm>=4.66.2
-Requires-Dist: beautifulsoup4>=4.13.3
+Requires-Dist: beautifulsoup4>=4.13.4
 Requires-Dist: openpyxl>=3.1.5
 Requires-Dist: joblib>=1.4.2
 Requires-Dist: pyarrow>=18.0.0
@@ -46,6 +46,7 @@ Requires-Dist: dateparser>=1.2.0
 Requires-Dist: playwright>=1.51.0
 Requires-Dist: cchardet>=2.2.0a2
 Requires-Dist: lxml>=5.3.0
+Requires-Dist: gender-guesser>=0.4.0
 Dynamic: author
 Dynamic: author-email
 Dynamic: classifier
@@ -107,6 +108,7 @@ Python 3.11.6:
 - [playwright](https://playwright.dev/)
 - [cchardet](https://github.com/PyYoshi/cChardet)
 - [lxml](https://lxml.de/)
+- [gender-guesser](https://github.com/lead-ratings/gender-guesser)
 
 ## Raison D'être :thought_balloon:
 
@@ -129,13 +131,17 @@ The supported leagues are:
 * **Game**: A game within a season.
     * **Team**: The team within the game. Note that in games with individual players a team exists as a wrapper.
         * **Player**: A player within the team.
+            * **Address**: The address information of a players birth.
+            * **Owner**: The owner of the player.
         * **Odds**: The odds for the team to win the game.
             * **Bookie**: The bookie publishing the odds.
         * **News**: News about the team the day before the game.
         * **Social**: Social posts from the team the day before the game.
+        * **Coach**: A coach for the team.
     * **Venue**: The venue the game was played in.
         * **Address**: The address information of a venue.
             * **Weather**: The weather at the address.
+    * **Dividend**: The dividends the game pays out.
 
 ### Objects
 
@@ -159,6 +165,7 @@ A representation of the game within a season.
 * **season_type**: The type of the season the game was played in.
 * **postponed**: Whether the game was postponed.
 * **playoff**: Whether the game was a playoff game.
+* **distance**: The distance the game was played over.
 
 #### Team
 
@@ -180,6 +187,29 @@ A representation of a team within a game.
 * **assists**: The number of times the player on the team made a pass that resulted in a field goal in the game.
 * **turnovers**: The number of times a player on the team loses possession of the ball in the game.
 * **marks**: The number of times a player on the team marks the ball in the game.
+* **handballs**: The number of times a player on the team handballs the ball in the game.
+* **disposals**: The number of times a player on the team disposes of the ball in the game.
+* **goals**: The number of times a player on the team scored a goal in the game.
+* **behinds**: The number of times a player on the team scored a behind in the game.
+* **hit_outs**: The number of times a player on the team hit out the ball in the game.
+* **tackles**: The number of times a player on the team tackled another player in the game.
+* **rebounds**: The number of times a player on the team gets a rebound in the game.
+* **insides**: The number of times a player on the team kicks a ball inside 50 in the game.
+* **clearances**: The number of times a player on the team performs a clearance in the game.
+* **clangers**: The number of times a player on the team performs a clanger in the game.
+* **free_kicks_for**: The number of times a player on the team was rewarded a free kick in the game.
+* **free_kicks_against**: The number of times a player on the team gave a player on the other team a free kick in the game.
+* **brownlow_votes**: The number of times a player on the team was given a brownlow vote in the game.
+* **contested_possessions**: The number of times a player on the team got a contested posession in the game.
+* **uncontested_possessions**: The number of times a player on the team got an uncontested posession in the game.
+* **contested_marks**: The number of times a player on the team got a contested mark in the game.
+* **marks_inside**: The number of times a player on the team got a mark inside 50 in the game.
+* **one_percenters**: The number of times a player on the team performs a "one-percenter" in the game.
+* **bounces**: The number of times a player on the team bounces a ball.
+* **goal_assists**: The number of times a player on the team assists another player on the team with a goal in the game.
+* **coaches**: The coaches on the team during the game.
+* **lbw**: Length behind winner, expressed in metres.
+* **dividends**: The dividends the game pays out.
 
 #### Player
 
@@ -219,6 +249,14 @@ A representation of a player within a team within a game.
 * **goal_assists**: The number of assists on goal the player had in the game.
 * **percentage_played**: The percentage of the game the player was on the field.
 * **birth_date**: The birth date of the player.
+* **species**: The species of the player.
+* **handicap_weight**: The handicap weight of the player (in KGs).
+* **father**: The player representing the father of the player.
+* **sex**: The sex of the player.
+* **age**: The age of the player in years.
+* **starting_position**: The starting position of the player.
+* **weight**: The weight of the player in (in KGs).
+* **birth_address**: The address model for the players birth location.
 
 #### Odds
 
@@ -227,6 +265,8 @@ A representation of the odds for a team to win within a game.
 * **odds**: The decimal odds offered by a bookie for the team to win in the game.
 * **bookie**: The bookie offering these odds.
 * **dt**: When the odds were posted.
+* **canonical**: Whether these odds can be treated as canonical for the purposes of backtesting.
+* **bet**: The type of bet the odds represent.
 
 #### Venue
 
@@ -280,6 +320,30 @@ Social media posts one day out from the game.
 * **views**: The number of views the post has.
 * **published**: When the post was published.
 
+#### Coach
+
+The coach on the team at the time of the game.
+
+* **identifier**: The unique identifier for the coach.
+* **name**: The name of the coach.
+* **birth_date**: The birth date of the coach.
+* **age**: The age of the coach.
+
+#### Dividend
+
+The dividend payout at the end of the game.
+
+* **pool**: The type of bet paying the dividend.
+* **combination**: The combination of team identifiers making up the dividend.
+* **dividend**: The payout of the dividend.
+
+#### Owner
+
+The owner of a player.
+
+* **name**: The name of the owner.
+* **identifier**: The unique identifier of the owner.
+
 ## Caching
 
 This library uses very aggressive caching due to the large data requirements. If the requests are about a recent game (generally in the last 7 days) the caching is bypassed. The caching is as follows:

{sportsball-0.3.127 → sportsball-0.4.0}/README.md

@@ -48,6 +48,7 @@ Python 3.11.6:
 - [playwright](https://playwright.dev/)
 - [cchardet](https://github.com/PyYoshi/cChardet)
 - [lxml](https://lxml.de/)
+- [gender-guesser](https://github.com/lead-ratings/gender-guesser)
 
 ## Raison D'être :thought_balloon:
 
@@ -70,13 +71,17 @@ The supported leagues are:
 * **Game**: A game within a season.
     * **Team**: The team within the game. Note that in games with individual players a team exists as a wrapper.
         * **Player**: A player within the team.
+            * **Address**: The address information of a players birth.
+            * **Owner**: The owner of the player.
         * **Odds**: The odds for the team to win the game.
             * **Bookie**: The bookie publishing the odds.
         * **News**: News about the team the day before the game.
         * **Social**: Social posts from the team the day before the game.
+        * **Coach**: A coach for the team.
     * **Venue**: The venue the game was played in.
         * **Address**: The address information of a venue.
             * **Weather**: The weather at the address.
+    * **Dividend**: The dividends the game pays out.
 
 ### Objects
 
@@ -100,6 +105,7 @@ A representation of the game within a season.
 * **season_type**: The type of the season the game was played in.
 * **postponed**: Whether the game was postponed.
 * **playoff**: Whether the game was a playoff game.
+* **distance**: The distance the game was played over.
 
 #### Team
 
@@ -121,6 +127,29 @@ A representation of a team within a game.
 * **assists**: The number of times the player on the team made a pass that resulted in a field goal in the game.
 * **turnovers**: The number of times a player on the team loses possession of the ball in the game.
 * **marks**: The number of times a player on the team marks the ball in the game.
+* **handballs**: The number of times a player on the team handballs the ball in the game.
+* **disposals**: The number of times a player on the team disposes of the ball in the game.
+* **goals**: The number of times a player on the team scored a goal in the game.
+* **behinds**: The number of times a player on the team scored a behind in the game.
+* **hit_outs**: The number of times a player on the team hit out the ball in the game.
+* **tackles**: The number of times a player on the team tackled another player in the game.
+* **rebounds**: The number of times a player on the team gets a rebound in the game.
+* **insides**: The number of times a player on the team kicks a ball inside 50 in the game.
+* **clearances**: The number of times a player on the team performs a clearance in the game.
+* **clangers**: The number of times a player on the team performs a clanger in the game.
+* **free_kicks_for**: The number of times a player on the team was rewarded a free kick in the game.
+* **free_kicks_against**: The number of times a player on the team gave a player on the other team a free kick in the game.
+* **brownlow_votes**: The number of times a player on the team was given a brownlow vote in the game.
+* **contested_possessions**: The number of times a player on the team got a contested posession in the game.
+* **uncontested_possessions**: The number of times a player on the team got an uncontested posession in the game.
+* **contested_marks**: The number of times a player on the team got a contested mark in the game.
+* **marks_inside**: The number of times a player on the team got a mark inside 50 in the game.
+* **one_percenters**: The number of times a player on the team performs a "one-percenter" in the game.
+* **bounces**: The number of times a player on the team bounces a ball.
+* **goal_assists**: The number of times a player on the team assists another player on the team with a goal in the game.
+* **coaches**: The coaches on the team during the game.
+* **lbw**: Length behind winner, expressed in metres.
+* **dividends**: The dividends the game pays out.
 
 #### Player
 
@@ -160,6 +189,14 @@ A representation of a player within a team within a game.
 * **goal_assists**: The number of assists on goal the player had in the game.
 * **percentage_played**: The percentage of the game the player was on the field.
 * **birth_date**: The birth date of the player.
+* **species**: The species of the player.
+* **handicap_weight**: The handicap weight of the player (in KGs).
+* **father**: The player representing the father of the player.
+* **sex**: The sex of the player.
+* **age**: The age of the player in years.
+* **starting_position**: The starting position of the player.
+* **weight**: The weight of the player in (in KGs).
+* **birth_address**: The address model for the players birth location.
 
 #### Odds
 
@@ -168,6 +205,8 @@ A representation of the odds for a team to win within a game.
 * **odds**: The decimal odds offered by a bookie for the team to win in the game.
 * **bookie**: The bookie offering these odds.
 * **dt**: When the odds were posted.
+* **canonical**: Whether these odds can be treated as canonical for the purposes of backtesting.
+* **bet**: The type of bet the odds represent.
 
 #### Venue
 
@@ -221,6 +260,30 @@ Social media posts one day out from the game.
 * **views**: The number of views the post has.
 * **published**: When the post was published.
 
+#### Coach
+
+The coach on the team at the time of the game.
+
+* **identifier**: The unique identifier for the coach.
+* **name**: The name of the coach.
+* **birth_date**: The birth date of the coach.
+* **age**: The age of the coach.
+
+#### Dividend
+
+The dividend payout at the end of the game.
+
+* **pool**: The type of bet paying the dividend.
+* **combination**: The combination of team identifiers making up the dividend.
+* **dividend**: The payout of the dividend.
+
+#### Owner
+
+The owner of a player.
+
+* **name**: The name of the owner.
+* **identifier**: The unique identifier of the owner.
+
 ## Caching
 
 This library uses very aggressive caching due to the large data requirements. If the requests are about a recent game (generally in the last 7 days) the caching is bypassed. The caching is as follows:

sportsball-0.3.127/sportsball.egg-info/requires.txt → sportsball-0.4.0/requirements.txt

@@ -3,7 +3,7 @@ requests>=2.31.0
 requests-cache>=1.2.0
 python-dateutil>=1.16.0
 tqdm>=4.66.2
-beautifulsoup4>=4.13.3
+beautifulsoup4>=4.13.4
 openpyxl>=3.1.5
 joblib>=1.4.2
 pyarrow>=18.0.0
@@ -33,3 +33,4 @@ dateparser>=1.2.0
 playwright>=1.51.0
 cchardet>=2.2.0a2
 lxml>=5.3.0
+gender-guesser>=0.4.0

{sportsball-0.3.127 → sportsball-0.4.0}/setup.py

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

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/__init__.py

@@ -4,4 +4,4 @@
 import cchardet  # type: ignore
 import lxml  # noqa: ignore
 
-__VERSION__ = "0.3.127"
+__VERSION__ = "0.4.0"

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afl/afl_afl_game_model.py

@@ -1,6 +1,6 @@
 """AFL AFL game model."""
 
-# pylint: disable=too-many-statements,protected-access,too-many-arguments,bare-except,duplicate-code
+# pylint: disable=too-many-statements,protected-access,too-many-arguments,bare-except,duplicate-code,too-many-locals
 import datetime
 import re
 
@@ -12,24 +12,32 @@ from playwright.sync_api import Playwright
 from ....playwright import ensure_install
 from ...game_model import GameModel
 from ...league import League
+from ..position import Position, position_from_str
 from .afl_afl_team_model import create_afl_afl_team_model
 from .afl_afl_venue_model import create_afl_afl_venue_model
 
 
-def parse_players_v1(div: Tag) -> list[list[tuple[str, str, str, str]]]:
+def parse_players_v1(div: Tag) -> list[list[tuple[str, str, str, str, Position]]]:
     """Parses the players from a v1 format."""
-    teams_players: list[list[tuple[str, str, str, str]]] = [[], []]
+    teams_players: list[list[tuple[str, str, str, str, Position]]] = [[], []]
     for div_positions_row in div.find_all(
         "div", {"class": re.compile(".*team-lineups__positions-row.*")}
     ):
-        row_players: list[list[tuple[str, str, str, str]]] = [[], []]
+        row_players: list[list[tuple[str, str, str, str, Position]]] = [[], []]
         for count, div_team_players in enumerate(
             div_positions_row.find_all(
                 "div",
                 {"class": re.compile(".*team-lineups__positions-players-container.*")},
             )
         ):
-            team_players: list[tuple[str, str, str, str]] = []
+            team_players: list[tuple[str, str, str, str, Position]] = []
+            position = None
+            for span in div_team_players.find_all(
+                "span", {"class": re.compile(".*team-lineups__position-meta-label.*")}
+            ):
+                position = position_from_str(span.get_text().strip())
+            if position is None:
+                raise ValueError("position is null")
             for a in div_team_players.find_all(
                 "a", {"class": re.compile(".*js-player-profile-link.*")}
             ):
@@ -49,7 +57,9 @@ def parse_players_v1(div: Tag) -> list[list[tuple[str, str, str, str]]]:
                     )
                 if player_number is None:
                     raise ValueError("player_number is null")
-                team_players.append((player_id, player_number, first_name, second_name))
+                team_players.append(
+                    (player_id, player_number, first_name, second_name, position)
+                )
             row_players[count].extend(team_players)
         for count, row_players_list in enumerate(row_players):
             teams_players[count].extend(row_players_list)
@@ -71,8 +81,8 @@ def _extract_dt(soup: BeautifulSoup) -> datetime.datetime:
 
 
 def _extract_players(
-    soup: BeautifulSoup, players: list[list[tuple[str, str, str, str]]]
-) -> list[list[tuple[str, str, str, str]]]:
+    soup: BeautifulSoup, players: list[list[tuple[str, str, str, str, Position]]]
+) -> list[list[tuple[str, str, str, str, Position]]]:
     def is_players_empty() -> bool:
         nonlocal players
         if not players:
@@ -94,15 +104,17 @@ def _extract_players(
 
 
 def _parse(
-    html: str, players: list[list[tuple[str, str, str, str]]]
-) -> tuple[list[float], datetime.datetime, list[list[tuple[str, str, str, str]]]]:
+    html: str, players: list[list[tuple[str, str, str, str, Position]]]
+) -> tuple[
+    list[float], datetime.datetime, list[list[tuple[str, str, str, str, Position]]]
+]:
     soup = BeautifulSoup(html, "lxml")
     return _extract_odds(soup), _extract_dt(soup), _extract_players(soup, players)
 
 
 def create_afl_afl_game_model(
     team_names: list[str],
-    players: list[list[tuple[str, str, str, str]]],
+    players: list[list[tuple[str, str, str, str, Position]]],
     dt: datetime.datetime | None,
     venue_name: str,
     session: requests_cache.CachedSession,
@@ -150,4 +162,7 @@ def create_afl_afl_game_model(
         season_type=None,
         postponed=None,
         play_off=None,
+        distance=None,
+        dividends=[],
+        pot=None,
     )

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afl/afl_afl_league_model.py

@@ -16,6 +16,7 @@ from ....playwright import ensure_install
 from ...game_model import GameModel
 from ...league import League
 from ...league_model import LeagueModel
+from ..position import Position, position_from_str
 from .afl_afl_game_model import create_afl_afl_game_model, parse_players_v1
 
 
@@ -71,18 +72,23 @@ def _parse_v1(
 
 def _parse_v2_soup(
     soup: BeautifulSoup, html_url: str
-) -> Iterator[tuple[list[str], list[list[tuple[str, str, str, str]]], str, str]]:
+) -> Iterator[
+    tuple[list[str], list[list[tuple[str, str, str, str, Position]]], str, str]
+]:
     for div in soup.find_all("div", {"class": "team-lineups__item"}):
         team_names = []
         for span in div.find_all(
             "span", {"class": re.compile(".*team-lineups-header__name.*")}
         ):
             team_names = [x.strip() for x in span.get_text().strip().split(" v ")]
-        team_players: list[list[tuple[str, str, str, str]]] = [[], []]
+        team_players: list[list[tuple[str, str, str, str, Position]]] = [[], []]
 
         def process_a_player(
-            a: Tag, idx: int, team_players: list[list[tuple[str, str, str, str]]]
-        ) -> list[list[tuple[str, str, str, str]]]:
+            a: Tag,
+            idx: int,
+            team_players: list[list[tuple[str, str, str, str, Position]]],
+            div: Tag,
+        ) -> list[list[tuple[str, str, str, str, Position]]]:
             href = a.get("href")
             if not isinstance(href, str):
                 raise ValueError("href is not a str")
@@ -98,17 +104,26 @@ def _parse_v2_soup(
                 jersey = div_shirt.get_text().strip()
             if jersey is None:
                 raise ValueError("jersey is null")
-            team_players[idx].append((player_id, first_name, sur_name, jersey))
+            position = None
+            for span in div.find_all(
+                "span", {"class": re.compile(".*team-lineups__position-meta-label.*")}
+            ):
+                position = position_from_str(span.get_text().strip())
+            if position is None:
+                raise ValueError("position is null")
+            team_players[idx].append(
+                (player_id, first_name, sur_name, jersey, position)
+            )
             return team_players
 
         for a in div.find_all(
             "a", {"class": re.compile(".*team-lineups__player-entry--home-team.*")}
         ):
-            team_players = process_a_player(a, 0, team_players)
+            team_players = process_a_player(a, 0, team_players, div)
         for a in div.find_all(
             "a", {"class": re.compile(".*team-lineups__player-entry--away-team.*")}
         ):
-            team_players = process_a_player(a, 1, team_players)
+            team_players = process_a_player(a, 1, team_players, div)
 
         venue_name: str | None = None
         for div_info in div.find_all("div", {"class": "team-lineups-header__info"}):

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afl/afl_afl_odds_model.py

@@ -2,6 +2,7 @@
 
 import datetime
 
+from ...bet import Bet
 from ...team_model import OddsModel
 from .afl_afl_bookie_model import create_afl_afl_bookie_model
 
@@ -10,5 +11,9 @@ def create_afl_afl_odds_model(odds: float) -> OddsModel:
     """Create an odds model based off aus sports betting."""
     bookie = create_afl_afl_bookie_model()
     return OddsModel(
-        odds=odds, bookie=bookie, dt=datetime.datetime.now(), canonical=True
+        odds=odds,
+        bookie=bookie,
+        dt=datetime.datetime.now(),
+        canonical=True,
+        bet=str(Bet.WIN),
     )  # pyright: ignore

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afl/afl_afl_player_model.py

@@ -2,12 +2,16 @@
 
 # pylint: disable=duplicate-code
 from ...player_model import PlayerModel
+from ...sex import Sex
+from ...species import Species
+from ..position import Position
 
 
 def create_afl_afl_player_model(
     identifier: str,
     player_number: str,
     name: str,
+    position: Position,
 ) -> PlayerModel:
     """Create a player model from AFL AFL."""
     return PlayerModel(
@@ -45,4 +49,13 @@ def create_afl_afl_player_model(
         goal_assists=None,
         percentage_played=None,
         birth_date=None,
+        age=None,
+        species=str(Species.HUMAN),
+        handicap_weight=None,
+        father=None,
+        sex=str(Sex.MALE),
+        starting_position=str(position),
+        weight=None,
+        birth_address=None,
+        owner=None,
     )

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afl/afl_afl_team_model.py

@@ -8,13 +8,14 @@ import requests_cache
 from ...google.google_news_model import create_google_news_models
 from ...league import League
 from ...team_model import TeamModel
+from ..position import Position
 from .afl_afl_odds_model import create_afl_afl_odds_model
 from .afl_afl_player_model import create_afl_afl_player_model
 
 
 def create_afl_afl_team_model(
     team_name: str,
-    players: list[tuple[str, str, str, str]],
+    players: list[tuple[str, str, str, str, Position]],
     session: requests_cache.CachedSession,
     dt: datetime.datetime,
     ladder: list[str],
@@ -23,9 +24,9 @@ def create_afl_afl_team_model(
     """Create a team model from AFL AFL."""
     player_models = [
         create_afl_afl_player_model(
-            identifier, player_number, " ".join([first_name, second_name])
+            identifier, player_number, " ".join([first_name, second_name]), position
         )
-        for identifier, player_number, first_name, second_name in players
+        for identifier, player_number, first_name, second_name, position in players
     ]
     odds_models = []
     if odds is not None:
@@ -48,4 +49,7 @@ def create_afl_afl_team_model(
         offensive_rebounds=None,
         assists=None,
         turnovers=None,
+        coaches=[],
+        lbw=None,
+        end_dt=None,
     )

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afl/afl_afl_venue_model.py

@@ -22,4 +22,6 @@ def create_afl_afl_venue_model(
         address=address_model,
         is_grass=None,
         is_indoor=None,
+        is_turf=None,
+        is_dirt=None,
     )

sportsball-0.4.0/sportsball/data/afl/afltables/afl_afltables_coach_model.py

@@ -0,0 +1,95 @@
+"""AFL AFLTables coach model."""
+
+# pylint: disable=too-many-locals
+import datetime
+import io
+import os
+import urllib.parse
+from urllib.parse import urlparse
+
+import pandas as pd
+import pytest_is_running
+import requests_cache
+from bs4 import BeautifulSoup
+from dateutil.parser import parse
+from dateutil.relativedelta import relativedelta
+
+from ....cache import MEMORY
+from ...coach_model import CoachModel
+
+_COACH_URL_CACHE: dict[str, str] = {}
+
+
+def _create_afl_afltables_coach_model(
+    url: str, session: requests_cache.CachedSession, year: int, dt: datetime.datetime
+) -> CoachModel:
+    html = _COACH_URL_CACHE.get(url)
+    if html is None:
+        with session.cache_disabled():
+            response = session.get(url)
+            html = response.text
+            _COACH_URL_CACHE[url] = html
+    handle = io.StringIO()
+    handle.write(html)
+    handle.seek(0)
+    dfs = pd.read_html(handle)
+    df = dfs[0]
+    df = df[df.columns.values.tolist()[:2]]
+    year_ranges = df[df.columns.values.tolist()[1]].tolist()
+    names = df[df.columns.values.tolist()[0]].tolist()
+    name = None
+    for count, year_range in enumerate(year_ranges):
+        if "-" in year_range:
+            start_year, end_year = [int(x) for x in year_range.split("-")]
+        else:
+            start_year = end_year = int(year_range)
+        if start_year <= year >= end_year:
+            name = names[count].strip()
+            break
+    if name is None:
+        raise ValueError("name is null")
+    coach_url = None
+    soup = BeautifulSoup(html, "lxml")
+    for a in soup.find_all("a", href=True):
+        a_text = a.get_text().strip()
+        if a_text == name:
+            coach_url = urllib.parse.urljoin(url, a.get("href"))
+            break
+    if coach_url is None:
+        raise ValueError("coach is null")
+    o = urlparse(coach_url)
+    last_component = o.path.split("/")[-1]
+    identifier, _ = os.path.splitext(last_component)
+
+    response = session.get(coach_url)
+    response.raise_for_status()
+    soup = BeautifulSoup(response.text, "lxml")
+    birth_date = parse(soup.get_text().split("Born:")[1].strip().split()[0])
+
+    return CoachModel(
+        identifier=identifier,
+        name=name,
+        birth_date=birth_date,
+        age=relativedelta(birth_date, dt).years,
+    )
+
+
+@MEMORY.cache(ignore=["session"])
+def _cached_create_afl_afltables_coach_model(
+    url: str, session: requests_cache.CachedSession, year: int, dt: datetime.datetime
+) -> CoachModel:
+    return _create_afl_afltables_coach_model(url=url, session=session, year=year, dt=dt)
+
+
+def create_afl_afltables_coach_model(
+    url: str, session: requests_cache.CachedSession, year: int, dt: datetime.datetime
+) -> CoachModel:
+    """Create a coach model from AFL tables."""
+    if not pytest_is_running.is_running():
+        return _cached_create_afl_afltables_coach_model(
+            url=url, session=session, year=year, dt=dt
+        )
+    with session.cache_disabled():
+        return _create_afl_afltables_coach_model(
+            url=url, session=session, year=year, dt=dt
+        )

{sportsball-0.3.127 → sportsball-0.4.0}/sportsball/data/afl/afltables/afl_afltables_game_model.py

@@ -469,6 +469,9 @@ def _create_afl_afltables_game_model(
         season_type=season_type,
         postponed=None,
         play_off=None,
+        distance=None,
+        dividends=[],
+        pot=None,
     )