code.py

import re
import datetime
from collections import OrderedDict


def create_racer_abbreviations_dict(file_name):
    """Retrieves {'abbreviation': (name, team)}" format dict from abbreviations.txt"""
    abbreviations_dict = {}
    with open(file_name, 'r') as fn:
        for line in fn:
            matchObj = re.match(r'^(\w+)_([a-zA-Z\s]+)_([a-zA-Z\s]+)$', line)
            # group(1) is abbreviation, i.e 'SVM'
            abbreviations_dict[matchObj.group(1)] = (
                matchObj.group(2),  # name of a pilot
                matchObj.group(3).rstrip(),  # team
            )
    return abbreviations_dict


# {'abbreviation of pilot': ('name of pilot, 'team')}
abbr_dict = create_racer_abbreviations_dict(
    'abbreviations.txt')


# returns timing log from start.log or end.log in {'abbreviation': time} format
def retrieve_timings_from_log(file_name):
    timing_log = {}
    with open(file_name, 'r') as fn:
        for line in fn:
            # matches 2 groups: abbreviation of a racer and time
            matchObj = re.match(r'^([A-Z]+).*(\d{2}:\d+:\d+\.\d+)$', line)
            # converts time from a string to datetime object
            lap_time = datetime.datetime.strptime(
                matchObj.group(2).rstrip(), '%H:%M:%S.%f')
            # adds key, value to a timing_log
            timing_log[matchObj.group(1)] = lap_time
    return timing_log


start_timings = retrieve_timings_from_log('start.log')
end_timings = retrieve_timings_from_log('end.log')


def sorted_individual_results(start_timings_, end_timings_, abbr_dict_, reverse_order=False):
    """ 
    Receives start and end timings and returns an OrderedDict with 
    {abbreviations:timedeltas}
    """
    # creating dict with best lap results
    lap_results = {key: end_timings_[key] - start_timings_.get(key, 0)
                   for key in start_timings_.keys()}
    sorted_results = dict(
        sorted(lap_results.items(), key=lambda x: x[1], reverse=reverse_order))
    return sorted_results


sorted_lap_results = sorted_individual_results(
    start_timings, end_timings, abbr_dict)

# prints result board to a console
def print_result_board(sorted_lap_results_):
    counter = 1
    for key, value in sorted_lap_results_.items():
        racer_name = abbr_dict[key][0]
        team_name = abbr_dict[key][1]
        best_time = str(value)[2:-3]
        print(("{: <3} {: <18} | {: <30}  | {}".format(
            str(counter)+'.', racer_name, team_name, best_time)))
        if counter == 15:
            print(
                '----------------------------------------------------------------------')
        counter += 1


print_result_board(sorted_lap_results)

Add if __name__ == '__main__': statement to show the entry point of the script.

Use some docstring style. For example:

• Epytext
• reST

1. Add if __name__ == '__main__': as entry point
2. Bad naming in with open(file_name, 'r') as fn:
3. Regex should be declared as constants.
4. Bad naming in variable matchObj. Should be snake_case (match_obj).
5. Should be in one line:

abbr_dict = create_racer_abbreviations_dict(
    'abbreviations.txt')

sorted_lap_results = sorted_individual_results(
    start_timings, end_timings, abbr_dict)

    sorted_results = dict(
        sorted(lap_results.items(), key=lambda x: x[1], reverse=reverse_order))

print(
                '----------------------------------------------------------------------')

6. In 67 line best practice is using f-string. Not .format() method.
7. sorted_lap_results_ bad naming - _ in the end of name
8. Developer don`t use type hints.
9. Imports should be in alphabetical order.
10. Developer use others styles for docstrings.

create_racer_abbreviations_dict:

1. Use snake-case code style for all the variables;
2. Check if string was matched to regular expression before adding items to the dictionary;

retrieve_timings_from_log:

1. Same thing here as above

sorted_individual_results:

1. abbr_dict_ variable is not used inside the function, consider removing it from arguments or check function logic for errors;
2. Consider renaming function to sort_individual_results;

Overall:

1. Split function definitions from actual logic (first all needed functions, then main logic);
2. Use if __name__ == '__main__': construct to execute main logic of the file;
3. Comments above the functions could be moved into docstrings;
4. Make file paths with input data configurable and check their existence before making any operations with them;
5. Use code-specific exceptions and add reactions to them (like if the exception is critical to workflow or not);