Skip to content

Issues: Popov-Dmitriy-Ivanovich/Hackaton

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

11 Open 1 Closed
11 Open 1 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

Зачем на каждой итерации цикла создается новый датафрейм?
#32 opened Dec 16, 2023 by Popov-Dmitriy-Ivanovich
2
Это написано, чтобы при каждом импорте выполнялся код?
#31 opened Dec 16, 2023 by Popov-Dmitriy-Ivanovich
1
Почему бы не хранить эти ключи в одном большом JSON файле вместо сложной структуры из папок и csv файлов?
#30 opened Dec 16, 2023 by Popov-Dmitriy-Ivanovich
2
Хорошее возращаемое значение функции, особенно учитывая, что по её выводу будет итерироваться цикл через 15 строчек кода, при этом проверка на None отсутствует
#29 opened Dec 16, 2023 by Popov-Dmitriy-Ivanovich
2
а если пользователь не прошел форму?
#28 opened Dec 16, 2023 by Popov-Dmitriy-Ivanovich
1
Создали датафрэйм, записали его в файл, передали путь к файлу как параметр функции, внутри функции считали файл. Зачем это было сделано? Этот подход расходует больше ресурсов ПК т.к. происходят лишние обращения к постоянной памяти, засоряет хранилище сервера т.к. файл после вызова функций не удаляется. Почему бы просто не передать датафрейм как параметр функции сразу же
#26 opened Dec 16, 2023 by Popov-Dmitriy-Ivanovich
1
user_data импортируется как созданный объект из другого файла, в данном контексте это - глобальная переменная, в которую записываются данные пользователя при каждом вызове функции. Мало того, что это глобальная переменная, так она еще и не возращается к исходному значению перед вызовом функции
#25 opened Dec 15, 2023 by Popov-Dmitriy-Ivanovich
1
Абсолютный путь - очень плохая практика, особенно вхардкоженный
#24 opened Dec 15, 2023 by Popov-Dmitriy-Ivanovich
sorted - алгоритм сортировки, математически не может работать быстрее, чем за O(n*log(n)), 3 раза найти максимум работает за O(n), предлагаю сэкономить время пользователя и 3 раза найти максимум
#23 opened Dec 15, 2023 by Popov-Dmitriy-Ivanovich
1
Почему этот путь абсолютный? Почему .txt, а не .json? На сервере абсолютные пути очень плохо разворачиваются, я бы даже сказал совсем не разворачиваются
#22 opened Dec 15, 2023 by Popov-Dmitriy-Ivanovich
1
Зачем нужен output file, что в него записывается и зачем? Я не нашел ни одной функции, которая читала бы что-то из .txt файла
#21 opened Dec 15, 2023 by Popov-Dmitriy-Ivanovich
1
ProTip! Updated in the last three days: updated:>2025-02-25.