1) str.py - str как то вот пересекается с встроенным типом str - это не хорошо.
2) обилие за комментированного отладочного и/или устаревшего кода в master ветке - это не хорошо
3) python 2 - все таки считается устаревшим
4) стиль кода местами не соответствует PEP8 (например
такие длинные строки - ну это вот прям глаза режет)
5) нету единого стиля кода по самому проекту
6) вот
эта переменная итак глобальная так как определена на уровне модуля и, судя по дальнейшему использованию, вы не понимаете как в питоне работает global
7)
if new_text == "Late" or new_text == "L":
new_text = "L"
ничего не смущает? А если внимательно посмотреть? зачем делать операции вида а = а ?
8) А что если в
имени вкладки будет строка, позволяющая провести SQL Injection аттаку ? будет с вашей БД все что угодно
9) Основной файл под 3к строчек, с несколькими базовыми классами. Не кажется вам что было бы логичнее разнести код на отдельные модули?
10) судя по другим файлам в проекте, я не углублся, но могу предположить что у Вас дублируется функционал в следствии плохой архитектуры приложения в целом.
Вот вам мой комментарий, диагональным взглядом