Bug: Update method signature in single-user raceguess page

Dockerfile

@@ -1,10 +1,17 @@
 # syntax=docker/dockerfile:1
 
 FROM public.ecr.aws/docker/library/python:3.11.8-slim-bookworm
+
 RUN apt-get update -y
+
 WORKDIR /app
 COPY requirements.txt requirements.txt
+
 RUN pip3 install -r requirements.txt
+
 COPY . .
+
 EXPOSE 5000
+
+ENV FASTF1_CACHE="/cache"
 CMD ["python3", "-u", "-m", "flask", "--app", "formula10", "run", "--host", "0.0.0.0"]

flake.nix

@@ -5,64 +5,143 @@
   inputs.flake-utils.url = "github:numtide/flake-utils";
   inputs.devshell.url = "github:numtide/devshell";
 
-  outputs = { self, nixpkgs, flake-utils, devshell }:
-    flake-utils.lib.eachDefaultSystem (system:
-      let
-        pkgs = import nixpkgs {
-          inherit system;
-          config.allowUnfree = true;
-          overlays = [ devshell.overlays.default ];
+  outputs = {
+    self,
+    nixpkgs,
+    flake-utils,
+    devshell,
+  }:
+    flake-utils.lib.eachDefaultSystem (system: let
+      pkgs = import nixpkgs {
+        inherit system;
+        config.allowUnfree = true;
+        overlays = [devshell.overlays.default];
+      };
+
+      timple = pkgs.python311Packages.buildPythonPackage rec {
+        pname = "timple";
+        version = "0.1.8";
+
+        src = pkgs.python311Packages.fetchPypi {
+          inherit pname version;
+          hash = "sha256-u8EgMA8BA6OpPlSg0ASRxLcIcv5psRIEcBpIicagXw8=";
         };
 
-        myPython = pkgs.python311.withPackages (p: with p; [
+        doCheck = false;
+        pyproject = false;
+
+        # Build time deps
+        nativeBuildInputs = with pkgs.python311Packages; [
+          setuptools
+        ];
+
+        # Run time deps
+        dependencies = with pkgs.python311Packages; [
+          matplotlib
+          numpy
+        ];
+      };
+
+      fastf1 = pkgs.python311Packages.buildPythonPackage rec {
+        pname = "fastf1";
+        version = "3.4.0";
+
+        src = pkgs.python311Packages.fetchPypi {
+          inherit pname version;
+          hash = "sha256-rCjJaM0W2m9Yk3dhHkMOdIqPiKtVqoXuELBasmA9ybA=";
+        };
+
+        doCheck = false;
+        pyproject = true;
+
+        # Build time deps
+        nativeBuildInputs = with pkgs.python311Packages; [
+          hatchling
+          hatch-vcs
+        ];
+
+        # Run time deps
+        dependencies = with pkgs.python311Packages; [
+          matplotlib
+          numpy
+          pandas
+          python-dateutil
+          requests
+          requests-cache
+          scipy
+          rapidfuzz
+          websockets
+
+          timple
+        ];
+      };
+
+      myPython = pkgs.python311.withPackages (p:
+        with p; [
           # Basic
           rich
-          numpy
 
           # Web
           flask
           flask-sqlalchemy
+          flask-caching
           sqlalchemy
           requests
 
+          # Test
           pytest
+
+          fastf1
+
+          # TODO: For some reason, listing those under fastf1.dependencies doesn't work???
+          matplotlib
+          numpy
+          pandas
+          python-dateutil
+          requests-cache
+          scipy
+          rapidfuzz
+          websockets
+          timple
         ]);
-      in {
-        devShell = pkgs.devshell.mkShell {
-          name = "Formula10";
+    in {
+      devShell = pkgs.devshell.mkShell {
+        name = "Formula10";
 
-          packages = with pkgs; [
-            myPython
+        packages = with pkgs; [
+          myPython
 
-            nodejs_21
-            nodePackages.sass
-            nodePackages.postcss-cli
-            nodePackages.autoprefixer
-          ];
+          sqlitebrowser
 
-          # Use $1 for positional args
-          commands = [
-            {
-              name = "vscode";
-              help = "Launch VSCode";
-              command = "code . &>/dev/null &";
-            }
-            {
-              name = "pycharm";
-              help = "Launch PyCharm Professional";
-              command = "pycharm-professional . &>/dev/null &";
-            }
-            {
-              name = "db";
-              help = "Launch SQLiteBrowser";
-              command = "sqlitebrowser ./instance/formula10.db &>/dev/null &";
-            }
-            {
-              name = "api";
-              help = "Launch Hoppscotch in Google Chrome";
-              command = "google-chrome-stable https://hoppscotch.io &>/dev/null &";
-            }
-          ];
-        };
-      });
+          nodejs_21
+          nodePackages.sass
+          nodePackages.postcss-cli
+          nodePackages.autoprefixer
+        ];
+
+        # Use $1 for positional args
+        commands = [
+          {
+            name = "vscode";
+            help = "Launch VSCode";
+            command = "code . &>/dev/null &";
+          }
+          {
+            name = "pycharm";
+            help = "Launch PyCharm Professional";
+            command = "pycharm-professional . &>/dev/null &";
+          }
+          {
+            name = "db";
+            help = "Launch SQLiteBrowser";
+            command = "sqlitebrowser ./instance/formula10.db &>/dev/null &";
+          }
+          {
+            name = "api";
+            help = "Launch Hoppscotch in Google Chrome";
+            command = "google-chrome-stable https://hoppscotch.io &>/dev/null &";
+          }
+        ];
+      };
+    });
 }

formula10/__init__.py

@@ -1,6 +1,8 @@
 import os
 from flask import Flask
 from flask_sqlalchemy import SQLAlchemy
+from flask_caching import Cache
+from werkzeug.middleware.profiler import ProfilerMiddleware
 
 # Load local ENV variables (can be set when calling the executable)
 ENABLE_TIMING: bool = False if os.getenv("DISABLE_TIMING") == "True" else True
@@ -21,8 +23,13 @@ app.url_map.strict_slashes = False
 db: SQLAlchemy = SQLAlchemy()
 db.init_app(app)
 
+cache: Cache = Cache(config={"CACHE_TYPE": "SimpleCache"})
+cache.init_app(app)
+
+# app.wsgi_app = ProfilerMiddleware(app.wsgi_app, restrictions=("/formula10/*",), sort_by=("cumtime",))
+
 # NOTE: These imports are required to register the routes. They need to be imported after "app" is declared
-import formula10.controller.race_controller  # type: ignore
+import formula10.controller.race_controller
 import formula10.controller.season_controller
 import formula10.controller.leaderboard_controller
 import formula10.controller.statistics_controller

formula10/controller/admin_controller.py

@@ -5,12 +5,13 @@ from werkzeug import Response
 
 from formula10.controller.error_controller import error_redirect
 from formula10.database.update_queries import update_race_result, update_user
+from formula10.domain.cache_invalidator import cache_invalidate_user_updated, cache_invalidate_race_result_updated
 from formula10.domain.domain_model import Model
 from formula10.domain.template_model import TemplateModel
 from formula10 import app
-from formula10.openf1.model.api_session import ApiSession
+from formula10.openf1.model.openf1_session import OpenF1Session
 from formula10.openf1.openf1_definitions import OPENF1_SESSION_NAME_RACE
-from formula10.openf1.openf1_fetcher import fetch_openf1_driver, fetch_openf1_position, fetch_openf1_session
+from formula10.openf1.openf1_fetcher import openf1_fetch_driver, openf1_fetch_position, openf1_fetch_session
 
 
 @app.route("/result")
@@ -43,18 +44,20 @@ def result_enter_post(race_name: str) -> Response:
     if fastest_lap is None:
         return error_redirect("Data was not saved, because fastest lap was not set.")
 
+    cache_invalidate_race_result_updated()
     race_id: int = Model().race_by(race_name=race_name).id
     return update_race_result(race_id, pxxs, first_dnfs, dnfs, excluded, int(fastest_lap), sprint_pxxs, sprint_dnf_drivers)
 
 
 @app.route("/result-fetch/<race_name>", methods=["POST"])
 def result_fetch_post(race_name: str) -> Response:
-    session: ApiSession = fetch_openf1_session(OPENF1_SESSION_NAME_RACE, "KSA")
-    fetch_openf1_driver(session.session_key, "VER")
-    fetch_openf1_position(session.session_key, 1)
+    session: OpenF1Session = openf1_fetch_session(OPENF1_SESSION_NAME_RACE, "KSA")
+    openf1_fetch_driver(session.session_key, "VER")
+    openf1_fetch_position(session.session_key, 1)
 
     # @todo Fetch stuff and build the race_result using update_race_result(...)
 
+    cache_invalidate_race_result_updated()
     return redirect("/result")
 
 
@@ -68,11 +71,13 @@ def user_root() -> str:
 
 @app.route("/user-add", methods=["POST"])
 def user_add_post() -> Response:
+    cache_invalidate_user_updated()
     username: str | None = request.form.get("select-add-user")
     return update_user(username, add=True)
 
 
 @app.route("/user-delete", methods=["POST"])
 def user_delete_post() -> Response:
+    cache_invalidate_user_updated()
     username: str | None = request.form.get("select-delete-user")
     return update_user(username, delete=True)

formula10/controller/race_controller.py

@@ -1,8 +1,10 @@
+from typing import List
 from urllib.parse import unquote
 from flask import redirect, render_template, request
 from werkzeug import Response
 
 from formula10.database.update_queries import delete_race_guess, update_race_guess
+from formula10.domain.cache_invalidator import cache_invalidate_race_guess_updated
 from formula10.domain.domain_model import Model
 from formula10.domain.points_model import PointsModel
 from formula10.domain.template_model import TemplateModel
@@ -37,6 +39,7 @@ def race_guess_post(race_name: str, user_name: str) -> Response:
     pxx: str | None = request.form.get("pxxselect")
     dnf: str | None = request.form.get("dnfselect")
 
+    cache_invalidate_race_guess_updated()
     race_id: int = Model().race_by(race_name=race_name).id
     user_id: int = Model().user_by(user_name=user_name).id
     return update_race_guess(race_id, user_id,
@@ -49,6 +52,7 @@ def race_guess_delete_post(race_name: str, user_name: str) -> Response:
     race_name = unquote(race_name)
     user_name = unquote(user_name)
 
+    cache_invalidate_race_guess_updated()
     race_id: int = Model().race_by(race_name=race_name).id
     user_id: int = Model().user_by(user_name=user_name).id
     return delete_race_guess(race_id, user_id)

formula10/controller/season_controller.py

@@ -5,6 +5,7 @@ from werkzeug import Response
 
 from formula10.database.model.db_team import DbTeam
 from formula10.database.update_queries import update_season_guess
+from formula10.domain.cache_invalidator import cache_invalidate_season_guess_updated
 from formula10.domain.domain_model import Model
 from formula10.domain.model.team import NONE_TEAM
 from formula10.domain.points_model import PointsModel
@@ -44,5 +45,6 @@ def season_guess_post(user_name: str) -> Response:
     ]
     podium_driver_guesses: List[str] = request.form.getlist("podiumdrivers")
 
+    cache_invalidate_season_guess_updated()
     user_id: int = Model().user_by(user_name=user_name).id
     return update_season_guess(user_id, guesses, team_winner_guesses, podium_driver_guesses)

formula10/domain/cache_invalidator.py

@@ -0,0 +1,98 @@
+from typing import Callable, List
+
+from formula10 import cache
+from formula10.domain.domain_model import Model
+from formula10.domain.points_model import PointsModel
+
+
+def cache_invalidate_user_updated() -> None:
+    caches: List[str] = [
+        "domain_all_users",
+        "domain_all_race_guesses",
+        "domain_all_season_guesses",
+        "points_points_per_step",
+        "points_user_standing",
+    ]
+
+    memoized_caches: List[Callable] = [
+        PointsModel.points_by,
+        PointsModel.race_guesses_by,
+        PointsModel.season_guesses_by,
+    ]
+
+    for c in caches:
+        cache.delete(c)
+
+    for c in memoized_caches:
+        cache.delete_memoized(c)
+
+
+def cache_invalidate_race_result_updated() -> None:
+    caches: List[str] = [
+        "domain_all_race_results",
+        "points_points_per_step",
+        "points_team_points_per_step",
+        "points_dnfs",
+        "points_driver_points_per_step_cumulative",
+        "points_wdc_standing_by_position",
+        "points_wdc_standing_by_driver",
+        "points_most_dnf_names",
+        "points_most_gained_names",
+        "points_most_lost_names",
+        "points_team_points_per_step_cumulative",
+        "points_teams_sorted_by_points",
+        "points_wcc_standing_by_position",
+        "points_wcc_standing_by_team",
+        "points_user_standing",
+        "template_first_race_without_result",
+    ]
+
+    memoized_caches: List[Callable] = [
+        PointsModel.driver_points_per_step,
+        PointsModel.driver_points_by,
+        PointsModel.total_driver_points_by,
+        PointsModel.drivers_sorted_by_points,
+        PointsModel.total_team_points_by,
+        PointsModel.points_by,
+        PointsModel.is_team_winner,
+        PointsModel.has_podium,
+        PointsModel.picks_with_points_count,
+    ]
+
+    for c in caches:
+        cache.delete(c)
+
+    for c in memoized_caches:
+        cache.delete_memoized(c)
+
+
+def cache_invalidate_race_guess_updated() -> None:
+    caches: List[str] = [
+        "domain_all_race_guesses",
+    ]
+
+    memoized_caches: List[Callable] = [
+        Model.race_guesses_by,
+    ]
+
+    for c in caches:
+        cache.delete(c)
+
+    for c in memoized_caches:
+        cache.delete_memoized(c)
+
+
+def cache_invalidate_season_guess_updated() -> None:
+    caches: List[str] = [
+        "domain_all_season_guesses",
+    ]
+
+    memoized_caches: List[Callable] = [
+        Model.season_guesses_by,
+    ]
+
+    for c in caches:
+        cache.delete(c)
+
+    for c in memoized_caches:
+        cache.delete_memoized(c)

formula10/domain/domain_model.py

@@ -18,133 +18,97 @@ from formula10.domain.model.season_guess import SeasonGuess
 from formula10.domain.model.season_guess_result import SeasonGuessResult
 from formula10.domain.model.team import NONE_TEAM, Team
 from formula10.domain.model.user import User
-from formula10 import db
+from formula10 import db, cache
 
 
 class Model:
-    _all_users: List[User] | None = None
-    _all_race_results: List[RaceResult] | None = None
-    _all_race_guesses: List[RaceGuess] | None = None
-    _all_season_guesses: List[SeasonGuess] | None = None
-    _all_season_guess_results: List[SeasonGuessResult] | None = None
-    _all_races: List[Race] | None = None
-    _all_drivers: List[Driver] | None = None
-    _all_active_drivers: List[Driver] | None = None
-    _all_teams: List[Team] | None = None
-
-    def all_users(self) -> List[User]:
+    @staticmethod
+    @cache.cached(timeout=None, key_prefix="domain_all_users") # Clear when adding/deleting users
+    def all_users() -> List[User]:
         """
         Returns a list of all enabled users.
         """
-        if self._all_users is None:
-            self._all_users = [
-                User.from_db_user(db_user)
-                for db_user in db.session.query(DbUser).filter_by(enabled=True).all()
-            ]
+        db_users = db.session.query(DbUser).filter_by(enabled=True).all()
+        return [User.from_db_user(db_user) for db_user in db_users]
 
-        return self._all_users
-
-    def all_race_results(self) -> List[RaceResult]:
+    @staticmethod
+    @cache.cached(timeout=None, key_prefix="domain_all_race_results") # Clear when adding/updating results
+    def all_race_results() -> List[RaceResult]:
         """
         Returns a list of all race results, in descending order (most recent first).
         """
-        if self._all_race_results is None:
-            self._all_race_results = [
-                RaceResult.from_db_race_result(db_race_result)
-                for db_race_result in db.session.query(DbRaceResult).join(DbRaceResult.race).order_by(desc(DbRace.number)).all()
-            ]
+        db_race_results = db.session.query(DbRaceResult).join(DbRaceResult.race).order_by(desc("number")).all()
+        return [RaceResult.from_db_race_result(db_race_result) for db_race_result in db_race_results]
 
-        return self._all_race_results
-
-    def all_race_guesses(self) -> List[RaceGuess]:
+    @staticmethod
+    @cache.cached(timeout=None, key_prefix="domain_all_race_guesses") # Clear when adding/updating race guesses or users
+    def all_race_guesses() -> List[RaceGuess]:
         """
         Returns a list of all race guesses (of enabled users).
         """
-        if self._all_race_guesses is None:
-            self._all_race_guesses = [
-                RaceGuess.from_db_race_guess(db_race_guess)
-                for db_race_guess in db.session.query(DbRaceGuess).join(DbRaceGuess.user).filter_by(enabled=True).all()  # Ignore disabled users
-            ]
+        db_race_guesses = db.session.query(DbRaceGuess).join(DbRaceGuess.user).filter_by(enabled=True).all()
+        return [RaceGuess.from_db_race_guess(db_race_guess) for db_race_guess in db_race_guesses]
 
-        return self._all_race_guesses
-
-    def all_season_guesses(self) -> List[SeasonGuess]:
+    @staticmethod
+    @cache.cached(timeout=None, key_prefix="domain_all_season_guesses") # Clear when adding/updating season guesses or users
+    def all_season_guesses() -> List[SeasonGuess]:
         """
         Returns a list of all season guesses (of enabled users).
         """
-        if self._all_season_guesses is None:
-            self._all_season_guesses = [
-                SeasonGuess.from_db_season_guess(db_season_guess)
-                for db_season_guess in db.session.query(DbSeasonGuess).join(DbSeasonGuess.user).filter_by(enabled=True).all()  # Ignore disabled users
-            ]
+        db_season_guesses = db.session.query(DbSeasonGuess).join(DbSeasonGuess.user).filter_by(enabled=True).all()
+        return [SeasonGuess.from_db_season_guess(db_season_guess) for db_season_guess in db_season_guesses]
 
-        return self._all_season_guesses
+    @staticmethod
+    @cache.cached(timeout=None, key_prefix="domain_all_season_guess_results") # No cleanup, bc entered manually
+    def all_season_guess_results() -> List[SeasonGuessResult]:
+        """
+        Returns a list of all season guess results (of enabled users).
+        """
+        db_season_guess_results = db.session.query(DbSeasonGuessResult).join(DbSeasonGuessResult.user).filter_by(enabled=True).all()
+        return [SeasonGuessResult.from_db_season_guess_result(db_season_guess_result) for db_season_guess_result in db_season_guess_results]
 
-    def all_season_guess_results(self) -> List[SeasonGuessResult]:
-        if self._all_season_guess_results is None:
-            self._all_season_guess_results = [
-                SeasonGuessResult.from_db_season_guess_result(db_season_guess_result)
-                for db_season_guess_result in db.session.query(DbSeasonGuessResult).join(DbSeasonGuessResult.user).filter_by(enabled=True).all()  # Ignore disabled users
-            ]
-
-        return self._all_season_guess_results
-
-    def all_races(self) -> List[Race]:
+    @staticmethod
+    @cache.cached(timeout=None, key_prefix="domain_all_races") # No cleanup, bc entered manually
+    def all_races() -> List[Race]:
         """
         Returns a list of all races, in descending order (last race first).
         """
-        if self._all_races is None:
-            self._all_races = [
-                Race.from_db_race(db_race)
-                for db_race in db.session.query(DbRace).order_by(desc(DbRace.number)).all()
-            ]
+        db_races = db.session.query(DbRace).order_by(desc("number")).all()
+        return [Race.from_db_race(db_race) for db_race in db_races]
 
-        return self._all_races
-
-    def all_drivers(self, *, include_none: bool, include_inactive: bool) -> List[Driver]:
+    @staticmethod
+    @cache.memoize(timeout=None) # No cleanup, bc entered manually
+    def all_drivers(*, include_none: bool, include_inactive: bool) -> List[Driver]:
         """
         Returns a list of all active drivers.
         """
-        if include_inactive:
-            if self._all_drivers is None:
-                self._all_drivers = [
-                    Driver.from_db_driver(db_driver)
-                    for db_driver in db.session.query(DbDriver).all()
-                ]
+        db_drivers = db.session.query(DbDriver).all()
+        drivers = [Driver.from_db_driver(db_driver) for db_driver in db_drivers]
 
-            if include_none:
-                return self._all_drivers
-            else:
-                predicate: Callable[[Driver], bool] = lambda driver: driver != NONE_DRIVER
-                return find_multiple_strict(predicate, self._all_drivers)
-        else:
-            if self._all_active_drivers is None:
-                self._all_active_drivers = [
-                    Driver.from_db_driver(db_driver)
-                    for db_driver in db.session.query(DbDriver).filter_by(active=True).all()
-                ]
+        if not include_inactive:
+            predicate: Callable[[Driver], bool] = lambda driver: driver.active
+            drivers = find_multiple_strict(predicate, drivers)
 
-            if include_none:
-                return self._all_active_drivers
-            else:
-                predicate: Callable[[Driver], bool] = lambda driver: driver != NONE_DRIVER
-                return find_multiple_strict(predicate, self._all_active_drivers)
+        if not include_none:
+            predicate: Callable[[Driver], bool] = lambda driver: driver != NONE_DRIVER
+            drivers = find_multiple_strict(predicate, drivers)
 
-    def all_teams(self, *, include_none: bool) -> List[Team]:
+        return drivers
+
+    @staticmethod
+    @cache.memoize(timeout=None) # No cleanup, bc entered manually
+    def all_teams(*, include_none: bool) -> List[Team]:
         """
         Returns a list of all teams.
         """
-        if self._all_teams is None:
-            self._all_teams = [
-                Team.from_db_team(db_team)
-                for db_team in db.session.query(DbTeam).all()
-            ]
+        db_teams = db.session.query(DbTeam).all()
+        teams = [Team.from_db_team(db_team) for db_team in db_teams]
 
-        if include_none:
-            return self._all_teams
-        else:
+        if not include_none:
             predicate: Callable[[Team], bool] = lambda team: team != NONE_TEAM
-            return find_multiple_strict(predicate, self._all_teams)
+            return find_multiple_strict(predicate, teams)
+
+        return teams
 
     #
     # User queries
@@ -217,6 +181,7 @@ class Model:
         """
         return self.race_guesses_by()
 
+    @cache.memoize(timeout=None, args_to_ignore=["self"]) # Cleanup when adding/updating race guesses or users
     def race_guesses_by(self, *, user_name: str | None = None, race_name: str | None = None) -> RaceGuess | List[RaceGuess] | Dict[str, Dict[str, RaceGuess]] | None:
         # List of all guesses by a single user
         if user_name is not None and race_name is None:
@@ -266,6 +231,7 @@ class Model:
         """
         return self.season_guesses_by()
 
+    @cache.memoize(timeout=None, args_to_ignore=["self"]) # Cleanup when adding/updating season guesses or users
     def season_guesses_by(self, *, user_name: str | None = None) -> SeasonGuess | Dict[str, SeasonGuess] | None:
         if user_name is not None:
             predicate: Callable[[SeasonGuess], bool] = lambda guess: guess.user.name == user_name
@@ -320,6 +286,7 @@ class Model:
         """
         return self.drivers_by(include_inactive=include_inactive)
 
+    @cache.memoize(timeout=None, args_to_ignore=["self"]) # No Cleanup, data added manually
     def drivers_by(self, *, team_name: str | None = None, include_inactive: bool) -> List[Driver] | Dict[str, List[Driver]]:
         if team_name is not None:
             predicate: Callable[[Driver], bool] = lambda driver: driver.team.name == team_name

formula10/domain/model/driver.py

@@ -34,6 +34,11 @@ class Driver:
     def __hash__(self) -> int:
         return hash(self.id)
 
+    # This is important to memoize functions getting a Driver as input.
+    # The repr() will be appended to the cache key.
+    def __repr__(self) -> str:
+        return f"Driver(id={self.id}, name={self.name})"
+
     id: int
     name: str
     abbr: str
@@ -52,4 +57,4 @@ NONE_DRIVER.name = "None"
 NONE_DRIVER.abbr = "None"
 NONE_DRIVER.country = "NO"
 NONE_DRIVER.team = NONE_TEAM
-NONE_DRIVER.active = False
+NONE_DRIVER.active = True

formula10/domain/model/race.py

@@ -36,6 +36,9 @@ class Race:
     def __hash__(self) -> int:
         return hash(self.id)
 
+    def __repr__(self) -> str:
+        return f"race(\n\tid={self.id}, name={self.name}, number={self.number},\n\tdate={self.date}, quali_date={self.quali_date},\n\thas_sprint={self.has_sprint}, place_to_guess={self.place_to_guess}\n)"
+
     id: int
     name: str
     number: int

formula10/domain/points_model.py

@@ -1,7 +1,8 @@
 import json
-from typing import Any, Callable, Dict, List, overload
+from typing import Any, Callable, Dict, List, overload, Tuple
 import numpy as np
 
+from formula10 import cache
 from formula10.domain.domain_model import Model
 from formula10.domain.model.driver import NONE_DRIVER, Driver
 from formula10.domain.model.race_guess import RaceGuess
@@ -10,15 +11,11 @@ from formula10.domain.model.season_guess import SeasonGuess
 from formula10.domain.model.season_guess_result import SeasonGuessResult
 from formula10.domain.model.team import Team
 from formula10.domain.model.user import User
+from formula10.database.validation import find_single_or_none_strict
 
 # Guess points
 
-RACE_GUESS_OFFSET_POINTS: Dict[int, int] = {
-    3: 1,
-    2: 3,
-    1: 6,
-    0: 10
-}
+RACE_GUESS_OFFSET_POINTS: Dict[int, int] = {3: 1, 2: 3, 1: 6, 0: 10}
 RACE_GUESS_DNF_POINTS: int = 10
 SEASON_GUESS_HOT_TAKE_POINTS: int = 10
 SEASON_GUESS_P2_POINTS: int = 10
@@ -43,18 +40,9 @@ DRIVER_RACE_POINTS: Dict[int, int] = {
     7: 6,
     8: 4,
     9: 2,
-    10: 1
-}
-DRIVER_SPRINT_POINTS: Dict[int, int] = {
-    1: 8,
-    2: 7,
-    3: 6,
-    4: 5,
-    5: 4,
-    6: 3,
-    7: 2,
-    8: 1
+    10: 1,
 }
+DRIVER_SPRINT_POINTS: Dict[int, int] = {1: 8, 2: 7, 3: 6, 4: 5, 5: 4, 6: 3, 7: 2, 8: 1}
 DRIVER_FASTEST_LAP_POINTS: int = 1
 
 # Last season results
@@ -79,8 +67,9 @@ WDC_STANDING_2023: Dict[str, int] = {
     "Daniel Ricciardo": 17,
     "Zhou Guanyu": 18,
     "Kevin Magnussen": 19,
-    "Logan Sargeant": 21
+    "Logan Sargeant": 21,
 }
+
 WCC_STANDING_2023: Dict[str, int] = {
     "Red Bull": 1,
     "Mercedes": 2,
@@ -91,11 +80,47 @@ WCC_STANDING_2023: Dict[str, int] = {
     "Williams": 7,
     "VCARB": 8,
     "Sauber": 9,
-    "Haas": 10
+    "Haas": 10,
+}
+
+# In case a substitute driver is driving, those points have to be subtracted from the actual driver
+# (Driver_ID, Race_ID, Points)
+WDC_SUBSTITUTE_POINTS: List[Tuple[int, int, int]] = [
+    (15, 2, 6), # Bearman raced for Sainz in Saudi Arabia
+    (8, 17, 1), # Bearman raced for Magnussen in Azerbaijan
+]
+
+WDC_STANDING_2024: Dict[str, int] = {
+    "Max Verstappen": 1,
+    "Lando Norris": 2,
+    "Charles Leclerc": 3,
+    "Oscar Piastri": 4,
+    "Carlos Sainz": 5,
+    "George Russell": 6,
+    "Lewis Hamilton": 7,
+    "Sergio Perez": 8,
+    "Fernando Alonso": 9,
+    "Pierre Gasly": 10,
+    "Nico Hulkenberg": 11,
+    "Yuki Tsunoda": 12,
+    "Lance Stroll": 13,
+    "Esteban Ocon": 14,
+    "Kevin Magnussen": 15,
+    "Alexander Albon": 16,
+    "Daniel Ricciardo": 17,
+    "Oliver Bearman": 18,
+    "Franco Colapinto": 19,
+    "Zhou Guanyu": 20,
+    "Liam Lawson": 21,
+    "Valtteri Bottas": 22,
+    "Logan Sargeant": 23,
+    "Jack Doohan": 24
 }
 
 def standing_points(race_guess: RaceGuess, race_result: RaceResult) -> int:
-    guessed_driver_position: int | None = race_result.driver_standing_position(driver=race_guess.pxx_guess)
+    guessed_driver_position: int | None = race_result.driver_standing_position(
+        driver=race_guess.pxx_guess
+    )
     if guessed_driver_position is None:
         return 0
 
@@ -105,6 +130,7 @@ def standing_points(race_guess: RaceGuess, race_result: RaceResult) -> int:
 
     return RACE_GUESS_OFFSET_POINTS[position_offset]
 
+
 def dnf_points(race_guess: RaceGuess, race_result: RaceResult) -> int:
     if race_guess.dnf_guess in race_result.initial_dnf:
         return RACE_GUESS_DNF_POINTS
@@ -114,163 +140,213 @@ def dnf_points(race_guess: RaceGuess, race_result: RaceResult) -> int:
 
     return 0
 
+
+def substitute_points(driver: Driver, race_number: int) -> int:
+    predicate: Callable[[Tuple[int, int, int]], bool] = lambda substitution: driver.id == substitution[0] and race_number == substitution[1]
+    substitution: Tuple[int, int, int] = find_single_or_none_strict(predicate, WDC_SUBSTITUTE_POINTS)
+
+    if substitution is not None:
+        return substitution[2]
+    else:
+        return 0
+
+
 class PointsModel(Model):
     """
     This class bundles all data + functionality required to do points calculations.
     """
 
-    _points_per_step: Dict[str, List[int]] | None = None
-    _driver_points_per_step: Dict[str, List[int]] | None = None
-    _active_driver_points_per_step: Dict[str, List[int]] | None = None
-    _team_points_per_step: Dict[str, List[int]] | None = None
-    _dnfs: Dict[str, int] | None = None
-
     def __init__(self):
         Model.__init__(self)
 
+    @cache.cached(
+        timeout=None, key_prefix="points_points_per_step"
+    )  # Clear when adding/updating race results or users
     def points_per_step(self) -> Dict[str, List[int]]:
         """
         Returns a dictionary of lists, containing points per race for each user.
         """
-        if self._points_per_step is None:
-            self._points_per_step = dict()
-            for user in self.all_users():
-                self._points_per_step[user.name] = [0] * (len(self.all_races()) + 1)  # Start at index 1, like the race numbers
+        points_per_step = dict()
+        for user in self.all_users():
+            points_per_step[user.name] = [0] * (
+                    len(self.all_races()) + 1
+            )  # Start at index 1, like the race numbers
 
-            for race_guess in self.all_race_guesses():
-                user_name: str = race_guess.user.name
-                race_number: int = race_guess.race.number
-                race_result: RaceResult | None = self.race_result_by(race_name=race_guess.race.name)
+        for race_guess in self.all_race_guesses():
+            user_name: str = race_guess.user.name
+            race_number: int = race_guess.race.number
+            race_result: RaceResult | None = self.race_result_by(
+                race_name=race_guess.race.name
+            )
 
-                if race_result is None:
-                    continue
+            if race_result is None:
+                continue
 
-                self._points_per_step[user_name][race_number] = standing_points(race_guess, race_result) + dnf_points(race_guess, race_result)
+            points_per_step[user_name][race_number] = standing_points(
+                race_guess, race_result
+            ) + dnf_points(race_guess, race_result)
 
-        return self._points_per_step
+        return points_per_step
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Clear when adding/updating race results
     def driver_points_per_step(self, *, include_inactive: bool) -> Dict[str, List[int]]:
         """
         Returns a dictionary of lists, containing points per race for each driver.
         """
-        if include_inactive:
-            if self._driver_points_per_step is None:
-                self._driver_points_per_step = dict()
-                for driver in self.all_drivers(include_none=False, include_inactive=True):
-                    self._driver_points_per_step[driver.name] = [0] * (len(self.all_races()) + 1)  # Start at index 1, like the race numbers
+        driver_points_per_step = dict()
+        for driver in self.all_drivers(
+                include_none=False, include_inactive=include_inactive
+        ):
+            driver_points_per_step[driver.name] = [0] * (
+                    len(self.all_races()) + 1
+            )  # Start at index 1, like the race numbers
 
-                for race_result in self.all_race_results():
-                    race_number: int = race_result.race.number
+        for race_result in self.all_race_results():
+            race_number: int = race_result.race.number
 
-                    for position, driver in race_result.standing.items():
-                        self._driver_points_per_step[driver.name][race_number] = DRIVER_RACE_POINTS[int(position)] if int(position) in DRIVER_RACE_POINTS else 0
-                        self._driver_points_per_step[driver.name][race_number] += DRIVER_FASTEST_LAP_POINTS if race_result.fastest_lap_driver == driver else 0
+            for position, driver in race_result.standing.items():
+                driver_points_per_step[driver.name][race_number] = (
+                    DRIVER_RACE_POINTS[int(position)]
+                    if int(position) in DRIVER_RACE_POINTS
+                    else 0
+                )
+                driver_points_per_step[driver.name][race_number] += (
+                    DRIVER_FASTEST_LAP_POINTS
+                    if race_result.fastest_lap_driver == driver
+                    and int(position) <= 10
+                    else 0
+                )
+                driver_points_per_step[driver.name][race_number] -= substitute_points(driver, race_number)
 
-                    for position, driver in race_result.sprint_standing.items():
-                        driver_name: str = driver.name
+            for position, driver in race_result.sprint_standing.items():
+                driver_name: str = driver.name
 
-                        self._driver_points_per_step[driver_name][race_number] += DRIVER_SPRINT_POINTS[int(position)] if int(position) in DRIVER_SPRINT_POINTS else 0
-
-            return self._driver_points_per_step
-        else:
-            if self._active_driver_points_per_step is None:
-                self._active_driver_points_per_step = dict()
-                for driver in self.all_drivers(include_none=False, include_inactive=False):
-                    self._active_driver_points_per_step[driver.name] = [0] * (len(self.all_races()) + 1)  # Start at index 1, like the race numbers
-
-                for race_result in self.all_race_results():
-                    race_number: int = race_result.race.number
-
-                    for position, driver in race_result.standing.items():
-                        self._active_driver_points_per_step[driver.name][race_number] = DRIVER_RACE_POINTS[int(position)] if int(position) in DRIVER_RACE_POINTS else 0
-                        self._active_driver_points_per_step[driver.name][race_number] += DRIVER_FASTEST_LAP_POINTS if race_result.fastest_lap_driver == driver else 0
-
-                    for position, driver in race_result.sprint_standing.items():
-                        driver_name: str = driver.name
-
-                        self._active_driver_points_per_step[driver_name][race_number] += DRIVER_SPRINT_POINTS[int(position)] if int(position) in DRIVER_SPRINT_POINTS else 0
-
-            return self._active_driver_points_per_step
+                driver_points_per_step[driver_name][race_number] += (
+                    DRIVER_SPRINT_POINTS[int(position)]
+                    if int(position) in DRIVER_SPRINT_POINTS
+                    else 0
+                )
 
+        return driver_points_per_step
 
+    @cache.cached(timeout=None, key_prefix="points_team_points_per_step")
     def team_points_per_step(self) -> Dict[str, List[int]]:
         """
         Returns a dictionary of lists, containing points per race for each team.
         """
-        if self._team_points_per_step is None:
-            self._team_points_per_step = dict()
-            for team in self.all_teams(include_none=False):
-                self._team_points_per_step[team.name] = [0] * (len(self.all_races()) + 1)  # Start at index 1, like the race numbers
+        team_points_per_step = dict()
+        for team in self.all_teams(include_none=False):
+            team_points_per_step[team.name] = [0] * (
+                    len(self.all_races()) + 1
+            )  # Start at index 1, like the race numbers
 
-            for race_result in self.all_race_results():
-                for driver in race_result.standing.values():
-                    team_name: str = driver.team.name
-                    race_number: int = race_result.race.number
+        for race_result in self.all_race_results():
+            for driver in race_result.standing.values():
+                team_name: str = driver.team.name
+                race_number: int = race_result.race.number
 
-                    self._team_points_per_step[team_name][race_number] += self.driver_points_per_step(include_inactive=True)[driver.name][race_number]
+                team_points_per_step[team_name][
+                    race_number
+                ] += self.driver_points_per_step(include_inactive=True)[driver.name][
+                    race_number
+                ]
 
-        return self._team_points_per_step
+        return team_points_per_step
 
+    @cache.cached(timeout=None, key_prefix="points_dnfs")
     def dnfs(self) -> Dict[str, int]:
-        if self._dnfs is None:
-            self._dnfs = dict()
+        dnfs = dict()
 
-            for driver in self.all_drivers(include_none=False, include_inactive=True):
-                self._dnfs[driver.name] = 0
+        for driver in self.all_drivers(include_none=False, include_inactive=True):
+            dnfs[driver.name] = 0
 
-            for race_result in self.all_race_results():
-                for driver in race_result.all_dnfs:
-                    self._dnfs[driver.name] += 1
+        for race_result in self.all_race_results():
+            for driver in race_result.all_dnfs:
+                dnfs[driver.name] += 1
 
-                for driver in race_result.sprint_dnfs:
-                    self._dnfs[driver.name] += 1
+            for driver in race_result.sprint_dnfs:
+                dnfs[driver.name] += 1
 
-        return self._dnfs
+        return dnfs
 
     #
     # Driver stats
     #
 
+    @cache.cached(
+        timeout=None, key_prefix="points_driver_points_per_step_cumulative"
+    )  # Cleanup when adding/updating race results
     def driver_points_per_step_cumulative(self) -> Dict[str, List[int]]:
         """
         Returns a dictionary of lists, containing cumulative points per race for each driver.
         """
         points_per_step_cumulative: Dict[str, List[int]] = dict()
-        for driver_name, points in self.driver_points_per_step(include_inactive=True).items():
+        for driver_name, points in self.driver_points_per_step(
+                include_inactive=True
+        ).items():
             points_per_step_cumulative[driver_name] = np.cumsum(points).tolist()
 
         return points_per_step_cumulative
 
     @overload
-    def driver_points_by(self, *, driver_name: str, include_inactive: bool) -> List[int]:
+    def driver_points_by(
+            self, *, driver_name: str, include_inactive: bool
+    ) -> List[int]:
         """
         Returns a list of points per race for a specific driver.
         """
-        return self.driver_points_by(driver_name=driver_name, include_inactive=include_inactive)
+        return self.driver_points_by(
+            driver_name=driver_name, include_inactive=include_inactive
+        )
 
     @overload
-    def driver_points_by(self, *, race_name: str, include_inactive: bool) -> Dict[str, int]:
+    def driver_points_by(
+            self, *, race_name: str, include_inactive: bool
+    ) -> Dict[str, int]:
         """
         Returns a dictionary of points per driver for a specific race.
         """
-        return self.driver_points_by(race_name=race_name, include_inactive=include_inactive)
+        return self.driver_points_by(
+            race_name=race_name, include_inactive=include_inactive
+        )
 
     @overload
-    def driver_points_by(self, *, driver_name: str, race_name: str, include_inactive: bool) -> int:
+    def driver_points_by(
+            self, *, driver_name: str, race_name: str, include_inactive: bool
+    ) -> int:
         """
         Returns the points for a specific race for a specific driver.
         """
-        return self.driver_points_by(driver_name=driver_name, race_name=race_name, include_inactive=include_inactive)
+        return self.driver_points_by(
+            driver_name=driver_name,
+            race_name=race_name,
+            include_inactive=include_inactive,
+        )
 
-    def driver_points_by(self, *, driver_name: str | None = None, race_name: str | None = None, include_inactive: bool) -> List[int] | Dict[str, int] | int:
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
+    def driver_points_by(
+            self,
+            *,
+            driver_name: str | None = None,
+            race_name: str | None = None,
+            include_inactive: bool
+    ) -> List[int] | Dict[str, int] | int:
         if driver_name is not None and race_name is None:
-            return self.driver_points_per_step(include_inactive=include_inactive)[driver_name]
+            return self.driver_points_per_step(include_inactive=include_inactive)[
+                driver_name
+            ]
 
         if driver_name is None and race_name is not None:
             race_number: int = self.race_by(race_name=race_name).number
             points_by_race: Dict[str, int] = dict()
 
-            for _driver_name, points in self.driver_points_per_step(include_inactive=include_inactive).items():
+            for _driver_name, points in self.driver_points_per_step(
+                    include_inactive=include_inactive
+            ).items():
                 points_by_race[_driver_name] = points[race_number]
 
             return points_by_race
@@ -278,58 +354,108 @@ class PointsModel(Model):
         if driver_name is not None and race_name is not None:
             race_number: int = self.race_by(race_name=race_name).number
 
-            return self.driver_points_per_step(include_inactive=include_inactive)[driver_name][race_number]
+            return self.driver_points_per_step(include_inactive=include_inactive)[
+                driver_name
+            ][race_number]
 
         raise Exception("driver_points_by received an illegal combination of arguments")
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
     def total_driver_points_by(self, driver_name: str) -> int:
-        return sum(self.driver_points_by(driver_name=driver_name, include_inactive=True))
+        return sum(
+            self.driver_points_by(driver_name=driver_name, include_inactive=True)
+        )
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
     def drivers_sorted_by_points(self, *, include_inactive: bool) -> List[Driver]:
-        comparator: Callable[[Driver], int] = lambda driver: self.total_driver_points_by(driver.name)
-        return sorted(self.all_drivers(include_none=False, include_inactive=include_inactive), key=comparator, reverse=True)
+        comparator: Callable[[Driver], int] = (
+            lambda driver: self.total_driver_points_by(driver.name)
+        )
+        return sorted(
+            self.all_drivers(include_none=False, include_inactive=include_inactive),
+            key=comparator,
+            reverse=True,
+        )
 
+    @cache.cached(
+        timeout=None, key_prefix="points_wdc_standing_by_position"
+    )  # Cleanup when adding/updating race results
     def wdc_standing_by_position(self) -> Dict[int, List[str]]:
         standing: Dict[int, List[str]] = dict()
 
-        for position in range(1, len(self.all_drivers(include_none=False, include_inactive=True)) + 1):
-            standing[position] = list()
+        if WDC_STANDING_2024 is None:
+            for position in range(
+                    1, len(self.all_drivers(include_none=False, include_inactive=True)) + 1
+            ):
+                standing[position] = list()
 
-        position: int = 1
-        last_points: int = 0
+            position: int = 1
+            last_points: int = 0
 
-        for driver in self.drivers_sorted_by_points(include_inactive=True):
-            points: int = self.total_driver_points_by(driver.name)
-            if points < last_points:
-                position += 1
+            for driver in self.drivers_sorted_by_points(include_inactive=True):
+                points: int = self.total_driver_points_by(driver.name)
+                if points < last_points:
+                    # If multiple drivers have equal points, a place is shared.
+                    # In this case, the next driver does not occupy the immediate next position.
+                    position += len(standing[position])
 
-            standing[position].append(driver.name)
-            last_points = points
+                standing[position].append(driver.name)
+                last_points = points
+
+        if WDC_STANDING_2024 is not None:
+            for position in range(1, len(WDC_STANDING_2024) + 1):
+                standing[position] = list()
+
+            for driver, position in WDC_STANDING_2024.items():
+                standing[position] += [driver]
 
         return standing
 
+    @cache.cached(
+        timeout=None, key_prefix="points_wdc_standing_by_driver"
+    )  # Cleanup when adding/updating race results
     def wdc_standing_by_driver(self) -> Dict[str, int]:
         standing: Dict[str, int] = dict()
 
-        position: int = 1
-        last_points: int = 0
+        if WDC_STANDING_2024 is None:
+            position: int = 1
+            last_points: int = 0
 
-        for driver in self.drivers_sorted_by_points(include_inactive=True):
-            points: int = self.total_driver_points_by(driver.name)
-            if points < last_points:
-                position += 1
+            for driver in self.drivers_sorted_by_points(include_inactive=True):
+                points: int = self.total_driver_points_by(driver.name)
+                if points < last_points:
+                    drivers_with_this_position = 0
+                    for _driver, _position in standing.items():
+                        if _position == position:
+                            drivers_with_this_position += 1
 
-            standing[driver.name] = position
-            last_points = points
+                    # If multiple drivers have equal points, a place is shared.
+                    # In this case, the next driver does not occupy the immediate next position.
+                    position += drivers_with_this_position
 
-        return standing
+                standing[driver.name] = position
+                last_points = points
+
+            return standing
+
+        if WDC_STANDING_2024 is not None:
+            return WDC_STANDING_2024
 
     def wdc_diff_2023_by(self, driver_name: str) -> int:
         if not driver_name in WDC_STANDING_2023:
             return 0
 
-        return WDC_STANDING_2023[driver_name] - self.wdc_standing_by_driver()[driver_name]
+        return (
+                WDC_STANDING_2023[driver_name] - self.wdc_standing_by_driver()[driver_name]
+        )
 
+    @cache.cached(
+        timeout=None, key_prefix="points_most_dnf_names"
+    )  # Cleanup when adding/updating race results
     def most_dnf_names(self) -> List[str]:
         dnf_names: List[str] = list()
         most_dnfs: int = 0
@@ -344,6 +470,9 @@ class PointsModel(Model):
 
         return dnf_names
 
+    @cache.cached(
+        timeout=None, key_prefix="points_most_gained_names"
+    )  # Cleanup when adding/updating race results
     def most_gained_names(self) -> List[str]:
         most_gained_names: List[str] = list()
         most_gained: int = 0
@@ -362,6 +491,9 @@ class PointsModel(Model):
 
         return most_gained_names
 
+    @cache.cached(
+        timeout=None, key_prefix="points_most_lost_names"
+    )  # Cleanup when adding/updating race results
     def most_lost_names(self) -> List[str]:
         most_lost_names: List[str] = list()
         most_lost: int = 100
@@ -384,6 +516,9 @@ class PointsModel(Model):
     # Team points
     #
 
+    @cache.cached(
+        timeout=None, key_prefix="points_team_points_per_step_cumulative"
+    )  # Cleanup when adding/updating race results
     def team_points_per_step_cumulative(self) -> Dict[str, List[int]]:
         """
         Returns a dictionary of lists, containing cumulative points per race for each team.
@@ -394,18 +529,34 @@ class PointsModel(Model):
 
         return points_per_step_cumulative
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
     def total_team_points_by(self, team_name: str) -> int:
-        teammates: List[Driver] = self.drivers_by(team_name=team_name, include_inactive=True)
-        return sum(sum(self.driver_points_by(driver_name=teammate.name, include_inactive=True)) for teammate in teammates)
+        teammates: List[Driver] = self.drivers_by(
+            team_name=team_name, include_inactive=True
+        )
+        return sum(
+            sum(self.driver_points_by(driver_name=teammate.name, include_inactive=True))
+            for teammate in teammates
+        )
 
+    @cache.cached(
+        timeout=None, key_prefix="points_teams_sorted_by_points"
+    )  # Cleanup when adding/updating race results
     def teams_sorted_by_points(self) -> List[Team]:
-        comparator: Callable[[Team], int] = lambda team: self.total_team_points_by(team.name)
+        comparator: Callable[[Team], int] = lambda team: self.total_team_points_by(
+            team.name
+        )
         return sorted(self.all_teams(include_none=False), key=comparator, reverse=True)
 
+    @cache.cached(
+        timeout=None, key_prefix="points_wcc_standing_by_position"
+    )  # Cleanup when adding/updating race results
     def wcc_standing_by_position(self) -> Dict[int, List[str]]:
         standing: Dict[int, List[str]] = dict()
 
-        for position in range (1, len(self.all_teams(include_none=False)) + 1):
+        for position in range(1, len(self.all_teams(include_none=False)) + 1):
             standing[position] = list()
 
         position: int = 1
@@ -414,13 +565,18 @@ class PointsModel(Model):
         for team in self.teams_sorted_by_points():
             points: int = self.total_team_points_by(team.name)
             if points < last_points:
-                position += 1
+                # If multiple teams have equal points, a place is shared.
+                # In this case, the next team does not occupy the immediate next position.
+                position += len(standing[position])
 
             standing[position].append(team.name)
             last_points = points
 
         return standing
 
+    @cache.cached(
+        timeout=None, key_prefix="points_wcc_standing_by_team"
+    )  # Cleanup when adding/updating race results
     def wcc_standing_by_team(self) -> Dict[str, int]:
         standing: Dict[str, int] = dict()
 
@@ -430,7 +586,14 @@ class PointsModel(Model):
         for team in self.teams_sorted_by_points():
             points: int = self.total_team_points_by(team.name)
             if points < last_points:
-                position += 1
+                teams_with_this_position = 0
+                for _team, _position in standing.items():
+                    if _position == position:
+                        teams_with_this_position += 1
+
+                # If multiple teams have equal points, a place is shared.
+                # In this case, the next team does not occupy the immediate next position.
+                position += teams_with_this_position
 
             standing[team.name] = position
             last_points = points
@@ -475,7 +638,12 @@ class PointsModel(Model):
         """
         return self.points_by(user_name=user_name, race_name=race_name)
 
-    def points_by(self, *, user_name: str | None = None, race_name: str | None = None) -> List[int] | Dict[str, int] | int:
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results or users
+    def points_by(
+            self, *, user_name: str | None = None, race_name: str | None = None
+    ) -> List[int] | Dict[str, int] | int:
         if user_name is not None and race_name is None:
             return self.points_per_step()[user_name]
 
@@ -495,31 +663,76 @@ class PointsModel(Model):
 
         raise Exception("points_by received an illegal combination of arguments")
 
-    def total_points_by(self, user_name: str) -> int:
+    def season_points_by(self, *, user_name: str) -> int:
+        """
+        Returns the number of points from seasonguesses for a specific user.
+        """
+        big_picks = (int(self.hot_take_correct(user_name=user_name)) * 10
+                     + int(self.p2_constructor_correct(user_name=user_name)) * 10
+                     + int(self.overtakes_correct(user_name=user_name)) * 10
+                     + int(self.dnfs_correct(user_name=user_name)) * 10
+                     + int(self.most_gained_correct(user_name=user_name)) * 10
+                     + int(self.most_lost_correct(user_name=user_name)) * 10)
+
+        small_picks = 0
+        guess: SeasonGuess = self.season_guesses_by(user_name=user_name)
+
+        for driver in guess.team_winners:
+            if self.is_team_winner(driver):
+                small_picks += 3
+            else:
+                small_picks -= 3
+
+        # NOTE: Not picked drivers that had a podium are also wrong
+        for driver in self.all_drivers(include_none=False, include_inactive=True):
+            if driver in guess.podiums and self.has_podium(driver):
+                small_picks += 3
+            elif driver in guess.podiums and not self.has_podium(driver):
+                small_picks -=2
+            elif driver not in guess.podiums and self.has_podium(driver):
+                small_picks -=2
+
+        return big_picks + small_picks
+
+    def total_points_by(self, *, user_name: str, include_season: bool) -> int:
         """
         Returns the total number of points for a specific user.
         """
-        return sum(self.points_by(user_name=user_name))
+        if include_season:
+            return sum(self.points_by(user_name=user_name)) + self.season_points_by(user_name=user_name)
+        else:
+            return sum(self.points_by(user_name=user_name))
 
-    def users_sorted_by_points(self) -> List[User]:
+    def users_sorted_by_points(self, *, include_season: bool) -> List[User]:
         """
         Returns the list of users, sorted by their points from race guesses (in descending order).
         """
-        comparator: Callable[[User], int] = lambda user: self.total_points_by(user.name)
+        comparator: Callable[[User], int] = lambda user: self.total_points_by(user_name=user.name, include_season=include_season)
         return sorted(self.all_users(), key=comparator, reverse=True)
 
-    def user_standing(self) -> Dict[str, int]:
+    @cache.cached(
+        timeout=None, key_prefix="points_user_standing"
+    )  # Cleanup when adding/updating race results or users
+    def user_standing(self, *, include_season: bool) -> Dict[str, int]:
         standing: Dict[str, int] = dict()
 
         position: int = 1
         last_points: int = 0
-        for user in self.users_sorted_by_points():
-            if self.total_points_by(user.name) < last_points:
-                position += 1
+
+        for user in self.users_sorted_by_points(include_season=include_season):
+            if self.total_points_by(user_name=user.name, include_season=include_season) < last_points:
+                users_with_this_position = 0
+                for _user, _position in standing.items():
+                    if _position == position:
+                        users_with_this_position += 1
+
+                # If multiple users have equal points, a place is shared.
+                # In this case, the next user does not occupy the immediate next position.
+                position += users_with_this_position
 
             standing[user.name] = position
 
-            last_points = self.total_points_by(user.name)
+            last_points = self.total_points_by(user_name=user.name, include_season=include_season)
 
         return standing
 
@@ -527,11 +740,16 @@ class PointsModel(Model):
         # Treat standing + dnf picks separately
         return len(self.race_guesses_by(user_name=user_name)) * 2
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
     def picks_with_points_count(self, user_name: str) -> int:
         count: int = 0
 
         for race_guess in self.race_guesses_by(user_name=user_name):
-            race_result: RaceResult | None = self.race_result_by(race_name=race_guess.race.name)
+            race_result: RaceResult | None = self.race_result_by(
+                race_name=race_guess.race.name
+            )
             if race_result is None:
                 continue
 
@@ -546,16 +764,22 @@ class PointsModel(Model):
         if self.picks_count(user_name) == 0:
             return 0.0
 
-        return self.total_points_by(user_name) / self.picks_count(user_name)
+        return self.total_points_by(user_name=user_name, include_season=False) / self.picks_count(user_name)
 
     #
     # Season guess evaluation
     #
 
     def hot_take_correct(self, user_name: str) -> bool:
-        season_guess_result: SeasonGuessResult | None = self.season_guess_result_by(user_name=user_name)
+        season_guess_result: SeasonGuessResult | None = self.season_guess_result_by(
+            user_name=user_name
+        )
 
-        return season_guess_result.hot_take_correct if season_guess_result is not None else False
+        return (
+            season_guess_result.hot_take_correct
+            if season_guess_result is not None
+            else False
+        )
 
     def p2_constructor_correct(self, user_name: str) -> bool:
         season_guess: SeasonGuess | None = self.season_guesses_by(user_name=user_name)
@@ -566,9 +790,15 @@ class PointsModel(Model):
         return season_guess.p2_wcc.name in self.wcc_standing_by_position()[2]
 
     def overtakes_correct(self, user_name: str) -> bool:
-        season_guess_result: SeasonGuessResult | None = self.season_guess_result_by(user_name=user_name)
+        season_guess_result: SeasonGuessResult | None = self.season_guess_result_by(
+            user_name=user_name
+        )
 
-        return season_guess_result.overtakes_correct if season_guess_result is not None else False
+        return (
+            season_guess_result.overtakes_correct
+            if season_guess_result is not None
+            else False
+        )
 
     def dnfs_correct(self, user_name: str) -> bool:
         season_guess: SeasonGuess | None = self.season_guesses_by(user_name=user_name)
@@ -594,14 +824,22 @@ class PointsModel(Model):
 
         return season_guess.most_wdc_lost.name in self.most_lost_names()
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
     def is_team_winner(self, driver: Driver) -> bool:
-        teammates: List[Driver] = self.drivers_by(team_name=driver.team.name, include_inactive=True)
-        teammate: Driver = teammates[0] if teammates[1] == driver else teammates[1]
+        teammates: List[Driver] = self.drivers_by(
+            team_name=driver.team.name, include_inactive=True
+        )
 
-        print(f"{driver.name} standing: {self.wdc_standing_by_driver()[driver.name]}, {teammate.name} standing: {self.wdc_standing_by_driver()[teammate.name]}")
+        # Min - Highest position is the lowest place number
+        winner: Driver = min(teammates, key=lambda driver: self.wdc_standing_by_driver()[driver.name])
 
-        return self.wdc_standing_by_driver()[driver.name] <= self.wdc_standing_by_driver()[teammate.name]
+        return driver == winner
 
+    @cache.memoize(
+        timeout=None, args_to_ignore=["self"]
+    )  # Cleanup when adding/updating race results
     def has_podium(self, driver: Driver) -> bool:
         for race_result in self.all_race_results():
             position: int | None = race_result.driver_standing_position(driver)
@@ -619,13 +857,13 @@ class PointsModel(Model):
 
         data["labels"] = [0] + [
             race.name for race in sorted(self.all_races(), key=lambda race: race.number)
-        ]
+        ] # + ["Season"]
 
         data["datasets"] = [
             {
-                "data": self.points_per_step_cumulative()[user.name],
+                "data": self.points_per_step_cumulative()[user.name], # + [self.total_points_by(user_name=user.name, include_season=True)],
                 "label": user.name,
-                "fill": False
+                "fill": False,
             }
             for user in self.all_users()
         ]
@@ -643,7 +881,7 @@ class PointsModel(Model):
             {
                 "data": self.driver_points_per_step_cumulative()[driver.name],
                 "label": driver.abbr,
-                "fill": False
+                "fill": False,
             }
             for driver in self.all_drivers(include_none=False, include_inactive=True)
         ]
@@ -661,7 +899,7 @@ class PointsModel(Model):
             {
                 "data": self.team_points_per_step_cumulative()[team.name],
                 "label": team.name,
-                "fill": False
+                "fill": False,
             }
             for team in self.all_teams(include_none=False)
         ]

formula10/domain/template_model.py

@@ -1,5 +1,5 @@
 from typing import List, Callable
-from formula10 import ENABLE_TIMING
+from formula10 import ENABLE_TIMING, cache
 
 from formula10.domain.domain_model import Model
 from formula10.domain.model.driver import Driver
@@ -54,6 +54,7 @@ class TemplateModel(Model):
 
         return self.all_users()
 
+    @cache.cached(timeout=None, key_prefix="template_first_race_without_result") # Cleanup when adding/updating race results
     def first_race_without_result(self) -> Race | None:
         """
         Returns the first race-object with no associated race result.
@@ -103,4 +104,4 @@ class TemplateModel(Model):
 
     def active_drivers_for_wdc_gained(self) -> List[Driver]:
         predicate: Callable[[Driver], bool] = lambda driver: driver.abbr not in self._wdc_gained_excluded_abbrs
-        return find_multiple_strict(predicate, self.all_drivers(include_none=False, include_inactive=False))
+        return find_multiple_strict(predicate, self.all_drivers(include_none=False, include_inactive=True))

formula10/fastf1/fastf1_definitions.py

@@ -0,0 +1,364 @@
+# https://docs.fastf1.dev/events.html#event-formats
+from typing import Optional
+
+from fastf1.core import Lap, DriverResult, Session
+from fastf1.events import Event
+from pandas import Timestamp, Timedelta
+
+FASTF1_SESSIONTYPE_NORMAL: str = "conventional"
+FASTF1_SESSIONTYPE_SPRINT: str = "sprint_qualifying"
+
+FASTF1_SPRINT_QUALIFYING_SESSION: int = 2
+FASTF1_SPRINT_SESSION: int = 3
+FASTF1_QUALIFYING_SESSION: int = 4
+FASTF1_RACE_SESSION: int = 5
+
+class EventDataHelper:
+    """
+    Helper class that provides easy access to EventDataFrame columns.
+    """
+
+    def __init__(self, event: Event):
+        self.__event = event
+
+    __event: Event
+
+    @property
+    def round_number(self) -> int:
+        return self.__event["RoundNumber"]
+
+    @property
+    def country(self) -> str:
+        return self.__event["Country"]
+
+    @property
+    def location(self) -> str:
+        return self.__event["Location"]
+
+    @property
+    def official_event_name(self) -> str:
+        """
+        Get the event name including sponsor names etc.
+        """
+        return self.__event["OfficialEventName"]
+
+    @property
+    def event_name(self) -> str:
+        return self.__event["EventName"]
+
+    @property
+    def event_date(self) -> Timestamp:
+        return self.__event["EventDate"]
+
+    @property
+    def event_format(self) -> str:
+        """
+        Get the event format.
+        @return: Either conventional, sprint, sprint_shootout, sprint_qualifying or testing (sprint_qualifying is the 2024 sprint format).
+        """
+        return self.__event["EventFormat"]
+
+    def session(self, session_number: int) -> str:
+        """
+        Get the format of a session belonging to this event.
+        @param session_number: Either 1, 2, 3, 4 or 5
+        @return: Either Pactice 1, Practice 2, Practice 3, Qualifying, Sprint, Sprint Shootout or Race
+        """
+        return self.__event[f"Session{session_number}"]
+
+    def session_date(self, session_number: int) -> Timestamp:
+        """
+        Get the date of a session belonging to this event.
+        @param session_number: Either 1, 2, 3, 4 or 5
+        """
+        return self.__event[f"Session{session_number}Date"]
+
+    def session_date_utc(self, session_number: int) -> Timestamp:
+        """
+        Get the date a session belonging to this event in coordinated universal time.
+        @param session_number: Either 1, 2, 3, 4 or 5
+        """
+        return self.__event[f"Session{session_number}DateUtc"]
+
+    @property
+    def f1_api_support(self) -> bool:
+        return self.__event["F1ApiSupport"]
+
+    # The following doesn't correspond to DataFrame columns
+
+    @property
+    def sprint_qualifying(self):
+        return self.__event.get_sprint_qualifying()
+
+    @property
+    def sprint(self):
+        return self.__event.get_sprint()
+
+    @property
+    def qualifying(self) -> Session:
+        return self.__event.get_qualifying()
+
+    @property
+    def race(self) -> Session:
+        return self.__event.get_race()
+
+
+class LapDataHelper:
+    """
+    Helper class that provides easy access to Lap DataFrame columns.
+    """
+
+    def __init__(self, lap: Lap):
+        self.__lap = lap
+
+    __lap: Lap
+
+    @property
+    def time(self) -> Timedelta:
+        return self.__lap["Time"]
+
+    @property
+    def driver(self) -> str:
+        return self.__lap["Driver"]
+
+    @property
+    def driver_number(self) -> str:
+        return self.__lap["DriverNumber"]
+
+    @property
+    def lap_time(self) -> Timedelta:
+        return self.__lap["LapTime"]
+
+    @property
+    def lap_number(self) -> int:
+        return self.__lap["LapNumber"]
+
+    @property
+    def stint(self) -> int:
+        return self.__lap["Stint"]
+
+    @property
+    def pit_out_time(self) -> Timedelta:
+        """
+        Get the session time when the car left the pit.
+        """
+        return self.__lap["PitOutTime"]
+
+    @property
+    def pit_in_time(self) -> Timedelta:
+        """
+        Get the session time when the car entered the pit.
+        """
+        return self.__lap["PitInTime"]
+
+    def sector_time(self, sector_number: int) -> Timedelta:
+        """
+        Get the sector times set in this lap.
+        @param sector_number: Either 1, 2 or 3.
+        """
+        return self.__lap[f"Sector{sector_number}Time"]
+
+    def sector_session_time(self, sector_number: int) -> Timedelta:
+        """
+        Get the session time when the sector time was set.
+        @param sector_number: Either 1, 2 or 3.
+        """
+        return self.__lap[f"Sector{sector_number}SessionTime"]
+
+    def speed(self, speedtrap: str) -> float:
+        """
+        Get car speed at measure point in this lap.
+        @param speedtrap: Either I1, I2, FL or ST (sector 1, sector 2, finish line or longest straight)
+        """
+        return self.__lap[f"Speed{speedtrap}"]
+
+    @property
+    def is_personal_best(self) -> bool:
+        return self.__lap["IsPersonalBest"]
+
+    @property
+    def compound(self) -> str:
+        """
+        Get compound used in this lap.
+        @return: Either SOFT, MEDIUM, HARD, INTERMEDIATE or WET
+        """
+        return self.__lap["Compound"]
+
+    @property
+    def tyre_life(self) -> int:
+        """
+        Get laps driven on the current tyre (includes laps from other sessions).
+        """
+        return self.__lap["TyreLife"]
+
+    @property
+    def fresh_tyre(self) -> bool:
+        return self.__lap["FreshTyre"]
+
+    @property
+    def team(self) -> str:
+        return self.__lap["Team"]
+
+    @property
+    def lap_start_time(self) -> Timedelta:
+        return self.__lap["LapStartTime"]
+
+    @property
+    def lap_start_date(self) -> Timestamp:
+        return self.__lap["LapStartDate"]
+
+    @property
+    def track_status(self) -> str:
+        """
+        Get the track status in this lap.
+        @return: Either 1, 2, 3, 4, 5, 6 or 7 (clear, yellow flag, ?, SC, red flag, VSC and VSC ending).
+        """
+        return self.__lap["TrackStatus"]
+
+    @property
+    def position(self) -> int:
+        return self.__lap["Position"]
+
+    @property
+    def deleted(self) -> Optional[bool]:
+        """
+        Determine if the lap was deleted (only available if race control messages are loaded).
+        """
+        return self.__lap["Deleted"]
+
+    @property
+    def deleted_reason(self) -> str:
+        return self.__lap["DeletedReason"]
+
+    @property
+    def fast_f1_generated(self) -> bool:
+        """
+        Determine if the lap was generated by FastF1 (information is interpolated).
+        """
+        return self.__lap["FastF1Generated"]
+
+    @property
+    def is_accurate(self) -> bool:
+        """
+        Determine if lap start and end match with other laps before and after.
+        """
+        return self.__lap["IsAccurate"]
+
+
+class DriverResultDataHelper:
+    """
+    Helper class that provides easy access to DriverResult DataFrame columns.
+    """
+
+    def __init__(self, driver_result: DriverResult):
+        self.__driver_result = driver_result
+
+    __driver_result: DriverResult
+
+    @property
+    def driver_number(self) -> str:
+        return self.__driver_result["DriverNumber"]
+
+    @property
+    def broadcast_name(self) -> str:
+        """
+        Get this driver's broadcast name.
+        @return: For example P GASLY
+        """
+        return self.__driver_result["BroadcastName"]
+
+    @property
+    def full_name(self) -> str:
+        return self.__driver_result["FullName"]
+
+    @property
+    def abbreviation(self) -> str:
+        return self.__driver_result["Abbreviation"]
+
+    # @property
+    # def driver_id(self) -> str:
+    #     """
+    #     Get the driverId used by the Ergast API.
+    #     """
+    #     return self.__driver_result["DriverID"]
+    
+    @property
+    def team_name(self) -> str:
+        return self.__driver_result["TeamName"]
+
+    @property
+    def team_color(self) -> str:
+        return self.__driver_result["TeamColor"]
+
+    # @property
+    # def team_id(self) -> str:
+    #     """
+    #     Get the constructorId used by the Ergast API.
+    #     @return:
+    #     """
+    #     return self.__driver_result["TeamID"]
+
+    @property
+    def first_name(self) -> str:
+        return self.__driver_result["FirstName"]
+
+    @property
+    def last_name(self) -> str:
+        return self.__driver_result["LastName"]
+
+    @property
+    def headshot_url(self) -> str:
+        return self.__driver_result["HeadshotUrl"]
+
+    @property
+    def country_code(self) -> str:
+        """
+        Get a driver's three-letter country code.
+        @return: For example FRA
+        """
+        return self.__driver_result["CountryCode"]
+
+    @property
+    def position(self) -> int:
+        return self.__driver_result["Position"]
+
+    @property
+    def classified_position(self) -> str:
+        """
+        Get the classification result for this driver.
+        @return: Either R, D, E, W, F, N (retired, disqualified, excluded, withdrawn, failed to qualify and not classified)
+        """
+        return self.__driver_result["ClassifiedPosition"]
+
+    @property
+    def grid_position(self) -> int:
+        """
+        Get the driver's starting position.
+        """
+        return self.__driver_result["GridPosition"]
+
+    def qualifying_time(self, qualifying_number: int) -> Timedelta:
+        """
+        Get the driver's best qualifying time.
+        @param qualifying_number: Either 1, 2 or 3 (for Q1, Q2 and Q3)
+        """
+        return self.__driver_result[f"Q{qualifying_number}"]
+
+    @property
+    def time(self) -> Timedelta:
+        """
+        Get the driver's total race time.
+        """
+        return self.__driver_result["Time"]
+
+    @property
+    def status(self) -> str:
+        """
+        Get the driver's finishing status.
+        @return: For example Finished, +1 Lap, Crash, Gearbox...
+        """
+        return self.__driver_result["Status"]
+
+    @property
+    def points(self) -> int:
+        return self.__driver_result["Points"]

formula10/openf1/model/openf1_driver.py

@@ -1,7 +1,7 @@
 from typing import Any, Callable, Dict
 
 
-class ApiDriver:
+class OpenF1Driver:
     __type_conversion_map__: Dict[str, Callable[[Any], Any]] = {
         "session_key": int,
         "meeting_key": int,
@@ -30,7 +30,7 @@ class ApiDriver:
 
             setattr(self, key, self.__type_conversion_map__[key](response[key]))
 
-        print("ApiDriver:", self.__dict__)
+        print("OpenF1Driver:", self.__dict__)
 
     def to_params(self) -> Dict[str, str]:
         params: Dict[str, str] = dict()

formula10/openf1/model/openf1_position.py

@@ -2,7 +2,7 @@ from datetime import datetime
 from typing import Any, Callable, Dict
 
 
-class ApiPosition:
+class OpenF1Position:
     __type_conversion_map__: Dict[str, Callable[[Any], Any]] = {
         "session_key": int,
         "meeting_key": int,
@@ -24,7 +24,7 @@ class ApiPosition:
 
             setattr(self, key, self.__type_conversion_map__[key](response[key]))
 
-        print("ApiPosition:", self.__dict__)
+        print("OpenF1Position:", self.__dict__)
 
     def to_params(self) -> Dict[str, str]:
         params: Dict[str, str] = dict()

formula10/openf1/model/openf1_session.py

@@ -2,7 +2,7 @@ from datetime import datetime, time
 from typing import Any, Callable, Dict
 
 
-class ApiSession:
+class OpenF1Session:
     __type_conversion_map__: Dict[str, Callable[[Any], Any]] = {
         "location": str,
         "country_key": int,
@@ -35,7 +35,7 @@ class ApiSession:
 
             setattr(self, key, self.__type_conversion_map__[key](response[key]))
 
-        print("ApiSession:", self.__dict__)
+        print("OpenF1Session:", self.__dict__)
 
     def to_params(self) -> Dict[str, str]:
         params: Dict[str, str] = dict()

formula10/openf1/openf1_fetcher.py

@@ -3,12 +3,12 @@ import json
 from typing import Any, Callable, Dict, List, cast
 from requests import Response, get
 
-from formula10.openf1.model.api_driver import ApiDriver
-from formula10.openf1.model.api_position import ApiPosition
-from formula10.openf1.model.api_session import ApiSession
+from formula10.openf1.model.openf1_driver import OpenF1Driver
+from formula10.openf1.model.openf1_position import OpenF1Position
+from formula10.openf1.model.openf1_session import OpenF1Session
 from formula10.openf1.openf1_definitions import OPENF1_DRIVER_ENDPOINT, OPENF1_POSITION_ENDPOINT, OPENF1_SESSION_ENDPOINT, OPENF1_SESSION_NAME_RACE, OPENF1_SESSION_NAME_SPRINT, OPENF1_SESSION_TYPE_RACE
 
-def request_helper(endpoint: str, params: Dict[str, str]) -> List[Dict[str, str]]:
+def openf1_request_helper(endpoint: str, params: Dict[str, str]) -> List[Dict[str, str]]:
     response: Response = get(endpoint, params=params)
     if not response.ok:
         raise Exception(f"OpenF1 request to {response.request.url} failed")
@@ -23,54 +23,54 @@ def request_helper(endpoint: str, params: Dict[str, str]) -> List[Dict[str, str]
         raise Exception(f"Unexpected OpenF1 response from {response.request.url}: {obj}")
 
 
-def fetch_openf1_latest_session(session_name: str) -> ApiSession:
+def openf1_fetch_latest_session(session_name: str) -> OpenF1Session:
     # ApiSession object only supports integer session_keys
-    response: List[Dict[str, str]] = request_helper(OPENF1_SESSION_ENDPOINT, {
+    response: List[Dict[str, str]] = openf1_request_helper(OPENF1_SESSION_ENDPOINT, {
         "session_key": "latest",
         "session_type": OPENF1_SESSION_TYPE_RACE,
         "session_name": session_name
     })
 
-    return ApiSession(response[0])
+    return OpenF1Session(response[0])
 
 
-def fetch_openf1_latest_race_session_key() -> int:
-    return fetch_openf1_latest_session(OPENF1_SESSION_NAME_RACE).session_key
+def openf1_fetch_latest_race_session_key() -> int:
+    return openf1_fetch_latest_session(OPENF1_SESSION_NAME_RACE).session_key
 
 
-def fetch_openf1_latest_sprint_session_key() -> int:
-    return fetch_openf1_latest_session(OPENF1_SESSION_NAME_SPRINT).session_key
+def openf1_fetch_latest_sprint_session_key() -> int:
+    return openf1_fetch_latest_session(OPENF1_SESSION_NAME_SPRINT).session_key
 
 
-def fetch_openf1_session(session_name: str, country_code: str) -> ApiSession:
-    _session: ApiSession = ApiSession(None)
+def openf1_fetch_session(session_name: str, country_code: str) -> OpenF1Session:
+    _session: OpenF1Session = OpenF1Session(None)
     _session.session_type = OPENF1_SESSION_TYPE_RACE  # includes races + sprints
     _session.year = 2024
     _session.country_code = country_code
     _session.session_name = session_name
 
-    response: List[Dict[str, str]] = request_helper(OPENF1_SESSION_ENDPOINT, _session.to_params())
+    response: List[Dict[str, str]] = openf1_request_helper(OPENF1_SESSION_ENDPOINT, _session.to_params())
 
-    return ApiSession(response[0])
+    return OpenF1Session(response[0])
 
 
-def fetch_openf1_driver(session_key: int, name_acronym: str) -> ApiDriver:
-    _driver: ApiDriver = ApiDriver(None)
+def openf1_fetch_driver(session_key: int, name_acronym: str) -> OpenF1Driver:
+    _driver: OpenF1Driver = OpenF1Driver(None)
     _driver.name_acronym = name_acronym
     _driver.session_key = session_key
 
-    response: List[Dict[str, str]] = request_helper(OPENF1_DRIVER_ENDPOINT, _driver.to_params())
+    response: List[Dict[str, str]] = openf1_request_helper(OPENF1_DRIVER_ENDPOINT, _driver.to_params())
 
-    return ApiDriver(response[0])
+    return OpenF1Driver(response[0])
 
 
-def fetch_openf1_position(session_key: int, position: int):
-    _position: ApiPosition = ApiPosition(None)
+def openf1_fetch_position(session_key: int, position: int):
+    _position: OpenF1Position = OpenF1Position(None)
     _position.session_key = session_key
     _position.position = position
 
-    response: List[Dict[str, str]] = request_helper(OPENF1_POSITION_ENDPOINT, _position.to_params())
+    response: List[Dict[str, str]] = openf1_request_helper(OPENF1_POSITION_ENDPOINT, _position.to_params())
 
     # Find the last driver that was on this position at last
     predicate: Callable[[Dict[str, str]], datetime] = lambda position: datetime.strptime(position["date"], "%Y-%m-%dT%H:%M:%S.%f")
-    return ApiPosition(max(response, key=predicate))
+    return OpenF1Position(max(response, key=predicate))

formula10/templates/base.jinja

@@ -24,13 +24,13 @@
 {% endmacro %}
 
 {# Simple driver select for forms #}
-{% macro driver_select(name, label, include_none, drivers=none, disabled=false, border="") %}
+{% macro driver_select(name, label, include_none, include_inactive=false, drivers=none, disabled=false, border="") %}
     <div class="form-floating">
         <select name="{{ name }}" id="{{ name }}" class="form-select {{ border }}" aria-label="{{ name }}" {% if disabled %}disabled="disabled"{% endif %}>
             <option value="" selected disabled hidden></option>
 
             {% if drivers == none %}
-                {% set drivers = model.all_drivers(include_none=include_none, include_inactive=False) %}
+                {% set drivers = model.all_drivers(include_none=include_none, include_inactive=include_inactive) %}
             {% endif %}
 
             {% for driver in drivers %}
@@ -42,14 +42,14 @@
 {% endmacro %}
 
 {# Driver select for forms where a value might be preselected #}
-{% macro driver_select_with_preselect(driver_match, name, label, include_none, drivers=none, disabled=false, border="") %}
+{% macro driver_select_with_preselect(driver_match, name, label, include_none, include_inactive=false, drivers=none, disabled=false, border="") %}
     <div class="form-floating">
         <select name="{{ name }}" id="{{ name }}" class="form-select {{ border }}" aria-label="{{ name }}" {% if disabled %}disabled="disabled"{% endif %}>
             {# Use namespace wrapper to persist scope between loop iterations #}
             {% set user_has_chosen = namespace(driverpre=false) %}
 
             {% if drivers == none %}
-                {% set drivers = model.all_drivers(include_none=include_none, include_inactive=False) %}
+                {% set drivers = model.all_drivers(include_none=include_none, include_inactive=include_inactive) %}
             {% endif %}
 
             {% for driver in drivers %}

formula10/templates/leaderboard.jinja

@@ -6,15 +6,15 @@
 
 {% block body %}
 
-    <div class="card shadow-sm mb-2">
-        <div class="card-header">
-            Note
-        </div>
-
-        <div class="card-body">
-            Points only include race picks.
-        </div>
-    </div>
+{#    <div class="card shadow-sm mb-2">#}
+{#        <div class="card-header">#}
+{#            Note#}
+{#        </div>#}
+{##}
+{#        <div class="card-body">#}
+{#            Points only include race picks.#}
+{#        </div>#}
+{#    </div>#}
 
     <div class="card shadow-sm mb-2">
         <div class="card-header">
@@ -29,6 +29,8 @@
                         <th scope="col" class="text-center" style="min-width: 50px;">Place</th>
                         <th scope="col" class="text-center" style="min-width: 50px;">User</th>
                         <th scope="col" class="text-center" style="min-width: 100px;">Points</th>
+                        <th scope="col" class="text-center" style="min-width: 100px;">Points (Race)</th>
+                        <th scope="col" class="text-center" style="min-width: 100px;">Points (Season)</th>
                         <th scope="col" class="text-center" style="min-width: 100px;">Total picks</th>
                         <th scope="col" class="text-center" style="min-width: 100px;" data-bs-toggle="tooltip"
                             title="Any points count as correct">Correct picks
@@ -38,12 +40,14 @@
                     </thead>
 
                     <tbody>
-                    {% for user in points.users_sorted_by_points() %}
-                        {% set user_standing = points.user_standing()[user.name] %}
+                    {% for user in points.users_sorted_by_points(include_season=True) %}
+                        {% set user_standing = points.user_standing(include_season=True)[user.name] %}
                         <tr class="{% if user_standing == 1 %}table-danger{% endif %}">
                             <td class="text-center text-nowrap">{{ user_standing }}</td>
                             <td class="text-center text-nowrap">{{ user.name }}</td>
-                            <td class="text-center text-nowrap">{{ points.total_points_by(user.name) }}</td>
+                            <td class="text-center text-nowrap">{{ points.total_points_by(user_name=user.name, include_season=True) }}</td>
+                            <td class="text-center text-nowrap">{{ points.total_points_by(user_name=user.name, include_season=False) }}</td>
+                            <td class="text-center text-nowrap">{{ points.season_points_by(user_name=user.name) }}</td>
                             <td class="text-center text-nowrap">{{ points.picks_count(user.name) }}</td>
                             <td class="text-center text-nowrap">{{ points.picks_with_points_count(user.name) }}</td>
                             <td class="text-center text-nowrap">{{ "%0.2f" % points.points_per_pick(user.name) }}</td>
@@ -57,7 +61,7 @@
 
     <div class="card shadow-sm mb-2">
         <div class="card-header">
-            History
+            History (Race)
         </div>
 
         <div class="card-body">

formula10/templates/race.jinja

@@ -39,13 +39,13 @@
                 {# Link should only be visible if all users are visible #}
                 {% if model.active_user is not none %}
                     <td class="text-center text-nowrap" style="min-width: 100px;">{{ model.active_user.name }}
-                        ({{ points.total_points_by(model.active_user.name) }})
+                        ({{ points.total_points_by(user_name=model.active_user.name, include_season=False) }})
                     </td>
                 {% else %}
                     {% for user in model.all_users() %}
                         <td class="text-center text-nowrap" style="min-width: 100px;">
                             <a href="/race/{{ user.name_sanitized }}" class="link-dark">{{ user.name }}
-                                ({{ points.total_points_by(user.name) }})</a>
+                                ({{ points.total_points_by(user_name=user.name, include_season=False) }})</a>
                         </td>
                     {% endfor %}
                 {% endif %}

formula10/templates/result.jinja

@@ -54,11 +54,11 @@
     <form action="{{ action_fetch_href }}" method="post">
         <div class="card shadow-sm mb-2 w-100">
             <div class="card-header">
-                OpenF1
+                Autofill
             </div>
 
             <div class="card-body">
-                <input type="submit" class="btn btn-danger mt-2 w-100" value="Fetch from OpenF1"
+                <input type="submit" class="btn btn-danger mt-2 w-100" value="Fetch using FastF1"
                        {% if race_result_open == false %}disabled="disabled"{% endif %}>
             </div>
         </div>

formula10/templates/season.jinja

@@ -10,16 +10,16 @@
 
 {% block body %}
 
-    <div class="card shadow-sm mb-2">
-        <div class="card-header">
-            Note
-        </div>
-
-        <div class="card-body">
-            Picks that match the current standings are marked in green, except for the hot-take and overtake picks, as
-            those are not evaluated automatically.<br>
-        </div>
-    </div>
+{#    <div class="card shadow-sm mb-2">#}
+{#        <div class="card-header">#}
+{#            Note#}
+{#        </div>#}
+{##}
+{#        <div class="card-body">#}
+{#            Picks that match the current standings are marked in green, except for the hot-take and overtake picks, as#}
+{#            those are not evaluated automatically.<br>#}
+{#        </div>#}
+{#    </div>#}
 
     <div class="grid card-grid">
 
@@ -50,7 +50,7 @@
                         {# Hot Take #}
                         <div class="form-floating">
                             <textarea
-                                    class="form-control {% if points.hot_take_correct(user.name) %}border-success{% endif %}"
+                                    class="form-control {% if points.hot_take_correct(user.name) %}border-success{% else %}border-danger{% endif %}"
                                     id="hot-take-input-{{ user.name }}" name="hottakeselect"
                                     style="height: 150px"
                                     {% if season_guess_open == false %}disabled="disabled"{% endif %}>
@@ -64,29 +64,29 @@
                         <div class="mt-2">
                             {{ team_select_with_preselect(team_match=user_guess.p2_wcc, name="p2select", label="P2 in WCC:",
                                    include_none=false, disabled=not season_guess_open,
-                                   border=("border-success" if points.p2_constructor_correct(user.name) else "")) }}
+                                   border=("border-success" if points.p2_constructor_correct(user.name) else "border-danger")) }}
                         </div>
 
                         {# Most Overtakes + DNFs #}
                         <div class="input-group mt-2">
                             {{ driver_select_with_preselect(driver_match=user_guess.most_overtakes, name="overtakeselect",
-                                   label="Most overtakes:", include_none=false, disabled=not season_guess_open,
-                                   border=("border-success" if points.overtakes_correct(user.name) else "")) }}
+                                   label="Most overtakes:", include_none=false, include_inactive=true, disabled=not season_guess_open,
+                                   border=("border-success" if points.overtakes_correct(user.name) else "border-danger")) }}
                             {{ driver_select_with_preselect(driver_match=user_guess.most_dnfs, name="dnfselect", label="Most DNFs:",
-                                   include_none=false, disabled=not season_guess_open,
-                                   border=("border-success" if points.dnfs_correct(user.name) else "")) }}
+                                   include_none=false, include_inactive=true, disabled=not season_guess_open,
+                                   border=("border-success" if points.dnfs_correct(user.name) else "border-danger")) }}
                         </div>
 
                         {# Most Gained + Lost #}
                         <div class="input-group mt-2" data-bs-toggle="tooltip"
                              title="Which driver will gain/lose the most places in comparison to last season's results?">
                             {{ driver_select_with_preselect(driver_match=user_guess.most_wdc_gained, name="gainedselect",
-                                   label="Most WDC pl. gained:", include_none=false, drivers=model.active_drivers_for_wdc_gained(),
-                                   disabled=not season_guess_open,
-                                   border=("border-success" if points.most_gained_correct(user.name) else "")) }}
+                                   label="Most WDC pl. gained:", include_none=false, include_inactive=true,
+                                   drivers=model.active_drivers_for_wdc_gained(), disabled=not season_guess_open,
+                                   border=("border-success" if points.most_gained_correct(user.name) else "border-danger")) }}
                             {{ driver_select_with_preselect(driver_match=user_guess.most_wdc_lost, name="lostselect",
-                                   label="Most WDC pl. lost:", include_none=false, disabled=not season_guess_open,
-                                   border=("border-success" if points.most_lost_correct(user.name) else "")) }}
+                                   label="Most WDC pl. lost:", include_none=false, include_inactive=true, disabled=not season_guess_open,
+                                   border=("border-success" if points.most_lost_correct(user.name) else "border-danger")) }}
                         </div>
 
                         {# Team-internal Winners #}
@@ -95,8 +95,10 @@
                             winners:</h6>
                         <div class="grid mt-2 container" style="row-gap: 0;">
                             {% for team in model.all_teams(include_none=false) %}
-                                {% set driver_a = model.drivers_by(team_name=team.name, include_inactive=False)[0] %}
-                                {% set driver_b = model.drivers_by(team_name=team.name, include_inactive=False)[1] %}
+                                {# HACK: Choosing 0 and 1 will chose the drivers with the lowest IDs (although there could be others). #}
+                                {#       This means the drivers chosen from at the start of the season will be visible. #}
+                                {% set driver_a = model.drivers_by(team_name=team.name, include_inactive=True)[0] %}
+                                {% set driver_b = model.drivers_by(team_name=team.name, include_inactive=True)[1] %}
 
                                 <div class="g-col-6">
                                     <div class="form-check form-check-inline">
@@ -106,7 +108,8 @@
                                                value="{{ driver_a.id }}"
                                                {% if (user_guess is not none) and (driver_a in user_guess.team_winners) %}checked="checked"{% endif %}
                                                {% if season_guess_open == false %}disabled="disabled"{% endif %}>
-                                        <label class="form-check-label {% if (user_guess is not none) and (driver_a in user_guess.team_winners) and points.is_team_winner(driver_a) %}text-success{% endif %}"
+                                        <label class="form-check-label {% if (user_guess is not none) and (driver_a in user_guess.team_winners) and points.is_team_winner(driver_a) %}text-success
+                                               {% elif (user_guess is not none) and (driver_a in user_guess.team_winners) and (not points.is_team_winner(driver_a)) %}text-danger{% endif %}"
                                                for="teamwinner-{{ team.id }}-1-{{ user.id }}">{{ driver_a.name }}</label>
                                     </div>
                                 </div>
@@ -119,7 +122,8 @@
                                                value="{{ driver_b.id }}"
                                                {% if (user_guess is not none) and (driver_b in user_guess.team_winners) %}checked="checked"{% endif %}
                                                {% if season_guess_open == false %}disabled="disabled"{% endif %}>
-                                        <label class="form-check-label {% if (user_guess is not none) and (driver_b in user_guess.team_winners) and points.is_team_winner(driver_b) %}text-success{% endif %}"
+                                        <label class="form-check-label {% if (user_guess is not none) and (driver_b in user_guess.team_winners) and points.is_team_winner(driver_b) %}text-success
+                                               {% elif (user_guess is not none) and (driver_b in user_guess.team_winners) and (not points.is_team_winner(driver_b)) %}text-danger{% endif %}"
                                                for="teamwinner-{{ team.id }}-2-{{ user.id }}">{{ driver_b.name }}</label>
                                     </div>
                                 </div>
@@ -131,8 +135,10 @@
                             title="Which driver will reach at least a single podium?">Drivers with podium(s):</h6>
                         <div class="grid mt-2 container" style="row-gap: 0;">
                             {% for team in model.all_teams(include_none=false) %}
-                                {% set driver_a = model.drivers_by(team_name=team.name, include_inactive=False)[0] %}
-                                {% set driver_b = model.drivers_by(team_name=team.name, include_inactive=False)[1] %}
+                                {# HACK: Choosing 0 and 1 will chose the drivers with the lowest IDs (although there could be others). #}
+                                {#       This means the drivers chosen from at the start of the season will be visible. #}
+                                {% set driver_a = model.drivers_by(team_name=team.name, include_inactive=True)[0] %}
+                                {% set driver_b = model.drivers_by(team_name=team.name, include_inactive=True)[1] %}
 
                                 <div class="g-col-6">
                                     <div class="form-check form-check-inline">
@@ -142,7 +148,9 @@
                                                value="{{ driver_a.id }}"
                                                {% if (user_guess is not none) and (driver_a in user_guess.podiums) %}checked="checked"{% endif %}
                                                {% if season_guess_open == false %}disabled="disabled"{% endif %}>
-                                        <label class="form-check-label {% if (user_guess is not none) and (driver_a in user_guess.podiums) and points.has_podium(driver_a) %}text-success{% endif %}"
+                                        <label class="form-check-label {% if (user_guess is not none) and (driver_a in user_guess.podiums) and points.has_podium(driver_a) %}text-success
+                                               {% elif (user_guess is not none) and (driver_a in user_guess.podiums) and (not points.has_podium(driver_a)) %}text-danger
+                                               {% elif (user_guess is not none) and (driver_a not in user_guess.podiums) and (points.has_podium(driver_a)) %}text-danger{% endif %}"
                                                for="podium-{{ driver_a.id }}-{{ user.id }}">{{ driver_a.name }}</label>
                                     </div>
                                 </div>
@@ -155,7 +163,9 @@
                                                value="{{ driver_b.id }}"
                                                {% if (user_guess is not none) and (driver_b in user_guess.podiums) %}checked="checked"{% endif %}
                                                {% if season_guess_open == false %}disabled="disabled"{% endif %}>
-                                        <label class="form-check-label {% if (user_guess is not none) and (driver_b in user_guess.podiums) and points.has_podium(driver_b) %}text-success{% endif %}"
+                                        <label class="form-check-label {% if (user_guess is not none) and (driver_b in user_guess.podiums) and points.has_podium(driver_b) %}text-success
+                                               {% elif (user_guess is not none) and (driver_b in user_guess.podiums) and (not points.has_podium(driver_b)) %}text-danger
+                                               {% elif (user_guess is not none) and (driver_b not in user_guess.podiums) and (points.has_podium(driver_b)) %}text-danger{% endif %}"
                                                for="podium-{{ driver_b.id }}-{{ user.id }}">{{ driver_b.name }}</label>
                                     </div>
                                 </div>

requirements.txt

@@ -3,7 +3,12 @@ numpy
 
 flask
 flask-sqlalchemy
+flask-caching
 sqlalchemy
 requests
+werkzeug
+
+fastf1
+pandas
 
 pytest