Вынести алгоритм обучения ДР about data-mining-tool-system

[Destrouer]

@Nikita94 этот вопрос не ко мне, а к разработчику сторонних алгоритмов. Всё что от меня зависело - я сделал, а именно описал интерфейс и сделал менеджер для работы с реализацией этого интерфейса

[Nikita94]

@Denzzel в следующем реквесте я поставлю фильтр на на сценарии и ты не сможешь обучаться. Для того, чтобы создать сценарий, тебе нужно поддержать свое view и общий интерфейс обучения. Советую поторопиться.

[michael-smirnov]

@Nikita94 , рано поддерживать интерфейс обучения. По моему скромному мнению он недоделан. @Destrouer , поправь, если не так, и ответь пожалуйста на несколько вопросов по поводу этого интерфейса:

1. Интерфейс для обучения называется ILearningAlgo?
2. Управление всеми созданными АО осуществляется через LearningAlgoManager?
3. В интерфейсе ILearningAlgo есть метод получения значений параметров алгоритма (getParams), а где метод инициализации этих параметров?
4. С самого начала этого года мы, насколько я помню, договорились, что может быть несколько выходных атрибутов. Почему startLearn принимает только один выходной атрибут?
5. Будет ли автоматически перерисовываться формочка с параметрами алгоритма после того, как я создам свой АО, наследуя его от ILearningAlgo, регистрируя в LearningAlgoManager и выбирая соответствующий АО в выпадающем списке при создании нового сценария обучения?

[Destrouer]

@michael-smirnov , да вроде всё доделано.

1. Да.
2. Да.
3. Он возвращает ссылку на твои объекты, так что инициализация происходит через менеджер.
4. Потому что в него передаётся один, по этому вопросу ещё висит задача, если я сейчас поменяю на массив то система вальнётся.
5. Увы - нет, эту фичу оставим на след год, а сейчас тебе надо свою вьюшку делать под алгоритм

[michael-smirnov]

@Destrouer , мои комментарии:

Он возвращает ссылку на твои объекты, так что инициализация происходит через менеджер

3. Это не очевидно, особенно когда работаешь и с шарпом, и с плюсами. Хотя бы хомментарий где-нибудь поставил что ли.

Потому что в него передаётся один, по этому вопросу ещё висит задача, если я сейчас поменяю на массив то система вальнётся.

4. Я не могу интегрировать обучение Кохонена в систему пока это так, потому что там явно нужно передавать двумерный массив y. Поэтому нужно исправлять

Увы - нет, эту фичу оставим на след год, а сейчас тебе надо свою вьюшку делать под алгоритм

5. Куча вопросов:
   a. Какой-нибудь сопровождающий код для поддержки моих вьюшек и вьюмоделей есть? Или сейчас там жестко генетик?
   b. Как система будет работать с разными вьюшками через LearningAlgoManager?
   c. Тебе под каждый свой АО тоже придется свою вьюшку делать?

Ну и в качестве педантства и занудства: не стоит выносить ILearningAlgo в отдельное пространство имен, нет в этом необходимости. В шарпе принято public методы и поля писать с заглавной буквы, а private с маленькой, а не наоборот. Это очень дезориентирует в твоем коде. И еще - "алгоритм" пишется algorithm, у тебя без h 😈

[Destrouer]

@michael-smirnov , один ответ на всё - не моя идея работать разным людям с АО, так что я ничего не делал для сторонних разработчиков.
Надо выносить. Так не надо код то читать, да и читать там 50 строк, а надо работать с интерфейсами доступными объектам. И да, будет круто получить ссылочку на стандарт + прописать где-то какой стандарт мы юзаем. НАСРАТЬ))

[michael-smirnov]

@Destrouer, единого, общепринятого стандарта кодирования на C#, очевидно, нет. Однако все более-менее популярные стандарты (можно посмотреть здесь и здесь) основаны на стиле кодирования всех сущностей из стандартной библиотеки .NET, который ты и нарушил. Это тебе просто для ознакомления, на будущее. Все-таки код пишется не для себя, а для других людей.

На счет что "не твоя идея": идея эта была с сентября, о чем я тебе тогда говорил, и ты с этим согласился (на то, чтобы обучение деревьев отдать @Denzzel). После этого я и начал создавать GUI. Ну и раз мы работаем в команде, результат работы будет также командный, как и ответственность за принятые решения.

Мне сейчас лишь нужно понять, как мне интегрировать мою часть обучения, а не обсуждать кто виноват. Поэтому еще раз спрашиваю: ожидать ли мне фикса п. 4 и как конкретно дела обстоят в п.5 (a-c).

[Destrouer]

@michael-smirnov , нельзя нарушить то чего нет! я не c# инженер и не знаю "стандартов", которые у кого-то в голове, есть требования - милости прошу их сформировать в виде документа, в ином случае - это лишь твоё мнение\догадки. На будущее - нет стандарта = нет нарушения, а к чему ты где-то привык - лишь твои проблемы.
Я про идею слияния меня и вас, это появилось после написания кода. Вот о чём я говорю - пришло поздно и поэтому нет ни доков ни подсказок, ничего! и делать сейчас я это не стану, тупо не до этого. Смотрите код и повторяйте что уже написано, не думаю что тут нужно больше знаний чем уметь читать и клацать по клаве.
СМ #93 . Не я занимался общей вьюшкой, но на сколько я вижу(и сам мог посмотреть, дабы время не тратить) там всё жестко завязано на вьюшку генетика, что надо сделать:

1. Написать свою вьюшку
2. Добавить байдинги на LearningAlgoManager
3. Прицепить её к CreateLearningScenarioPage
4. Может что ещё, я хз, этот вопрос вообще мимо кассы. Пробуй\смотри, там кода 10 строк.
   P.S. больше инфы в ближайшую неделю\две от меня не будет, т.к. у меня есть другие дела - сессия например, а то что кто-то только сейчас спохватился, спустя ДВЕ НЕДЕЛИ! меня мало волнует, было время когда я мог - кто-то не мог, теперь кто-то может - я не могу.

[michael-smirnov]

@Destrouer , чудненько.
Тогда в ближайшее время я разбираюсь с вьюшками / вьюмодельками для параметров АО и пытаюсь включить Кохонена в текущую версию ILearningAlgo. Жду от тебя переход на много выходных, это важно.
Если у меня будет дополнительное время между кривым подключением Кохонена и твоим изменением ILearningAlgo, я бы хотел протестировать Кохонена на генетическом алгоритме. Для этого мне нужно твое разрешение на модификацию его кода - для Кохонена перед startGenetic нужно будет вызывать одну функцию дополнительно.

И еще, все должны понимать, что решение #93 со стороны @Destrouer влечет каскадные изменения со стороны обучения Кохонена и ДР. Поэтому после правок @Destrouer и до моих, когда я поправлю Кохонена, система, вероятно, не будет компилироваться.