Skip to content

Update main.cpp#15

Open
kindofman wants to merge 3 commits intomasterfrom
kindofman-patch-15
Open

Update main.cpp#15
kindofman wants to merge 3 commits intomasterfrom
kindofman-patch-15

Conversation

@kindofman
Copy link
Copy Markdown
Owner

Очень сложная задача :( Совсем не 5 баллов.

@kindofman kindofman requested a review from ylyubimov December 21, 2019 07:14
Copy link
Copy Markdown
Collaborator

@ylyubimov ylyubimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В этой задаче нужно что-то типа таблички с прогонами, т.к. код похож на код из задачи 14
Т.е. должно быть несколько различных примеров скажем 100 - 1000 различных распределений точек (всего 10 точек)
Ты запускаешь свое решение, а также строишь точное решение полным перебором (10 точек - всего 10!, не очень много)))) берешь отношение точного решения к приближенному
Считаешь медиану, максимум, минимум 98 персентиль и другие статистики, какие хочется
Все это в таблицу и небольшую легенду с выводами.

Comment thread 15/main.cpp
void add_edge( int u, int v );
double MST();
void self_test() const;
void __get_optimal_path( int num, vector<bool> visited, int start, double result, int prev );
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вектора лучше по ссылкам передавать. Копирование массивов дело нелегкое

Comment thread 15/main.cpp Outdated
#include <list>
#include <vector>
#include <cmath>
#define INT_MAX 2147483647
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть стандартный INT_MAX, можно свой не заводить

@kindofman kindofman requested a review from ylyubimov January 7, 2020 08:04
Copy link
Copy Markdown
Collaborator

@ylyubimov ylyubimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исследования так и нет(

@kindofman
Copy link
Copy Markdown
Owner Author

Исследования так и нет(

Как я понял исходя из ваших комментариев должно быть:

  1. табличка с прогонами.
    2)Отношение точного решения к приближенному
    3)Медиана, минимум, максимум, персентиль(98), другие статистики.
    4)Вывод.
    5)В точном решении передавать вектор по ссылке.

Все это сделано. По дефолту табличка формируется в Эксель формате в текущей папке.

@kindofman kindofman requested a review from ylyubimov January 12, 2020 12:57
@ylyubimov
Copy link
Copy Markdown
Collaborator

Нужно было просто приложить этот файлик, не нужно было, чтобы я его генерировал)

Copy link
Copy Markdown
Collaborator

@ylyubimov ylyubimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поставил 5 баллов

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.

2 participants