Урусова Ксения

[akubara55]

No description provided.

[lgtm-com]

This pull request introduces 1 alert when merging 493d0c7 into fa45ea5 - view on LGTM.com

new alerts:

• 1 for Module imports itself

[vongostev]

Жду исправления замечаний.

[vongostev]

Написано неплохо, но тесты ничего не тестируют, нет количественного критерия правильности работы кода.

[vongostev]

Ага, нашел, что надо вводить числа в инпут. Хорошо бы сделать уведомление об этом, например

l = int(input("Введите среднее тестового распределения:"))

[vongostev]

Эти же замечания относятся и ко второму заданию.

[lgtm-com]

This pull request introduces 1 alert when merging 26482de into fa45ea5 - view on LGTM.com

new alerts:

• 1 for Module imports itself

[lgtm-com]

This pull request introduces 1 alert when merging 8377d68 into fa45ea5 - view on LGTM.com

new alerts:

• 1 for Module imports itself

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[vongostev]

Задание принимаю.

[vongostev]

В целом хорошо. Есть недочеты, но мелкие. Например, вот тут стоит использовать другой вариант итерации
list_x = [self.cosmic_bodies[i].vec_r[0] for i in range(len(self.cosmic_bodies))]
А тут использовать итерацию и zip

sun = CosmicBody(mass = 10000, vec_r = np.array([0, 0, 0]), vec_v = np.array([0, 0, 0]), t = 0, color = 'y', size = 20)
body_1 = CosmicBody(mass = data[1, 7], vec_r = data[1, :3], vec_v = data[1, 3:6], t = data[1, 6], color = 'b', size = 5)
body_2 = CosmicBody(mass = data[2, 7], vec_r = data[2, :3], vec_v = data[2, 3:6], t = data[2, 6], color = 'r', size = 2)
body_3 = CosmicBody(mass = data[3, 7], vec_r = data[3, :3], vec_v = data[3, 3:6], t = data[3, 6], color = 'm', size = 7)

Зачет ставлю