Skip to content

Comments

Контрольная работа#1

Open
kislball wants to merge 20 commits intopython-course-matmex:mainfrom
kislball:main
Open

Контрольная работа#1
kislball wants to merge 20 commits intopython-course-matmex:mainfrom
kislball:main

Conversation

@kislball
Copy link

  1. Были добавлены файлы __init__.py, LICENSE, .python-version, .gitignore для организации репозитория
  2. Запущен линтер для исправления ошибок от CI
  3. Используется стандартное возведение в степень, поскольку оно быстрее предыдущего алгоритма
  4. Исправлен баг с алгоритмом Луна
  5. Дополнены тесты, чтобы учесть пункты 3 и 4
  6. Добавлена реализация утилиты для алгоритма Луна
  7. Не учтено ограничение для возвездения в степень, указанное в README, так как стандарнтное возведение в степень в Python умеет работать с отрицательными степенями.

Выполнил Михайленко Георгий, группа 25.Б43-мм

Built-in Python exponentiation is fast enough for most cases. It also
works on non-integers, allowing for more versatility.
In the original implementation, parity checking was incorrect.
I.e. when running the algorithm on 15 digits(+1 control), the
algorithm would double wrong numbers.

This was fixed by redoing the calculation method.
In order to launch them, since GitHub won't run CI on forks without
changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant