Оценка по НИР за 1-й семестр 2-го курса

[Oleg Sychev]

Originally reported on Google Code with ID 315

- три таблицы утверждены
- на основании таблиц разработаны и реализованы в коде тестовые примеры
- реализовано и оттестировано не менее 1-го правила из каждой из таблиц.

Reported by oasychev on 2014-12-24 13:47:21

[Former user]

1) реализованы правила:
==/=
acos/cos
2) добавлены тесты к ним (отдельные)
3) добавлены тесты на идентификаторы

Reported by mariabirvg on 2014-12-26 13:39:37

[Former user]

добавлены правила, названия таблиц, аргументы в таблицы

Reported by mariabirvg on 2014-12-26 14:33:58

[Former user]

Этого достаточно?

Reported by mariabirvg on 2014-12-27 10:45:03

[Oleg Sychev]

Тесты должны быть на все правила, в особенности второй и третьей таблиц. Посмотрю сегодня.

Reported by oasychev on 2014-12-28 02:13:34

[Former user]

Меня смущают тесты:

1) test_best_string_pairs_2_all_functions
2) test_best_string_pairs_21_all_functions
3) В test_best_string_pairs1, test_best_string_pairs_2 нет ассерта на вид ошибки.

Почему в них вообще нет ассертов? Я думаю, в них есть что проверять.
В best_string_pair недостает теста над добавление символа.

По идее, это мелочи, можно доделать и после зачета.

Reported by mamontov.dp on 2014-12-28 08:00:29

[Former user]

Тест test_lexical_analyzer3 не проходит из-за того, что в строке 96 lexical_analyzer.php
$pair->matches() возвращает пустой массив вместо объекта. Надо бы посмотреть, почему
таки и исправить. Да, я помню этот вопрос, поэтому и пишу причину.

Reported by mamontov.dp on 2014-12-28 08:34:56

[Former user]

1) test_best_string_pairs_2_all_functions
2) test_best_string_pairs_21_all_functions
Ассерты есть - в цикле for. На данный момент он закомментирован. Там как раз образуются
все пары с типом спец ошибка.

3) В test_best_string_pairs1, test_best_string_pairs_2 нет ассерта на вид ошибки.
Тут есть проверка на type=999999 - как раз спец.ошибка

4)В best_string_pair недостает теста над добавление символа. - это на опечатку с вставкой
символа, например, type - typpe?

Reported by mariabirvg on 2014-12-28 08:36:04

[Former user]

>1) test_best_string_pairs_2_all_functions
>2) test_best_string_pairs_21_all_functions
>Ассерты есть - в цикле for. На данный момент он закомментирован. Там как раз образуются
все пары с типом спец ошибка.

Почему они закомментированы?

>4)В best_string_pair недостает теста над добавление символа. - это на опечатку с вставкой
символа, например, type - typpe?

Да.

Reported by mamontov.dp on 2014-12-28 08:37:21

[Former user]

сейчас посмотрю

Reported by mariabirvg on 2014-12-28 08:37:25

[Former user]

Потому что код еще не реализован и они дружно будут провалены.

Добавлю, без проблем

Reported by mariabirvg on 2014-12-28 08:38:32

[Former user]

Хорошо, жду дополнительного теста. Желательно, если есть возможность - поправить ошибку
в lexical_analyzer.

Reported by mamontov.dp on 2014-12-28 08:50:30

[Former user]

Тест добавлен (замена+вставка)

Изменения по пустому набору добавлены (у меня проходит, видимо не все добавилось в
коммит)

Reported by mariabirvg on 2014-12-28 09:04:41

[Former user]

Вижу, вроде работает.

Reported by mamontov.dp on 2014-12-28 09:06:55

[Former user]

На вторую и третью таблицу добавлю после 16*00 тесты.

Reported by mariabirvg on 2014-12-28 09:11:35

[Former user]

дополнительные тесты добавлены.

по типам данных тесты не все, так как непонятно будут ли формироваться дополнительные
наборы, либо пары будут добавляться в единый набор - необходимо уточнить после разработки
идеи алгоритма

Reported by mariabirvg on 2014-12-28 14:11:16

[Former user]

Примечание: test_best_string_pairs выполняется примерно 2,5 минуты - связано с 2 тестами
по 14 лексем в каждой строке.

Reported by mariabirvg on 2014-12-28 14:12:33

[Former user]

Тест вида:

http://pastebin.ru/0GhkMFYx

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

Reported by mamontov.dp on 2015-02-01 14:43:08

[Former user]

можно номер теста? я у себя не нашла такого

Reported by mariabirvg on 2015-02-01 16:49:43

[Former user]

кстати там порог 1 - строки должны быть полностью равны - поэтому нет соответствий

Reported by mariabirvg on 2015-02-01 16:55:00

[Former user]

Тест свой, чисто для проверки того, что поле операций заполняется. Пробовал с 0,33 и
с 0,1 - не сработало. Вечером еще раз попробую, просьба внести его в тест-кейсы и проверить.

Reported by mamontov.dp on 2015-02-02 04:32:48

[Former user]

Да, выставил предел в 0,32 и это помогло. Надо бы исследовать и дополнить тесты случаями,
когда ошибки в пределах двух-трех операций и сверять в них redaction. Думаю это поможет
в последующих исследованиях. Справишься с этим? Думаю не должно занять много времени.

Reported by mamontov.dp on 2015-02-02 16:01:19

[Former user]

Как я понимаю, в redaction,

m - совпадение между буквами лексем
i - лишняя буква в ответе студента
d - буква в ответе студента пропущена
r - буква в ответе студента заменена.

Единственное что - пока не получается в голове представить процесс отображения лексемы
из corrected в correctstring, из-за того, что совпадения между ними не везде строятся.

Reported by mamontov.dp on 2015-02-02 16:06:53

[Former user]

Я подумаю еще, в принципе, это можно быстро написать.

Reported by mamontov.dp on 2015-02-02 16:07:25

[Oleg Sychev]

Там единственная операция есть которая напрямую буквами не отображается - транспозиция.

Reported by oasychev on 2015-02-04 16:58:10

[Former user]

Я её не видел, не уверен, что она есть вообще. Может быть это будет последовательность
вида iidd? Сейчас попробую сгенерировать такое у себя на сервере.

Reported by mamontov.dp on 2015-02-04 18:43:55

[Former user]

Сгенерировал такой вариант. Получилось для лексем abc и acb строка вида mimd. Я  так
понимаю, что в принципе так и можно определить транспозицию по подстроке вида imd?

Reported by mamontov.dp on 2015-02-04 19:00:26

[Former user]

Для bca и cba также сгенерировалась строка imdm, что логично. Можно ли соответствующие
тесты и еще пару дополнительных добавить в анализатор?

Reported by mamontov.dp on 2015-02-04 19:03:29

[Former user]

Не всегда imd. Посмотри случай с повторяющимися буквами, транспозицией в конце и в начале
слова.
Мне тесты написать на redaction?

Reported by mariabirvg on 2015-02-05 13:47:46

[Former user]

 test_redaction.php
тут есть тесты на redaction

Reported by mariabirvg on 2015-02-05 13:49:02

[Former user]

Да, но в интеграционных тестах им тоже надлежит быть. Особенно такой важной вещи.

Reported by mamontov.dp on 2015-02-05 14:42:19

[Former user]

Да, при анализе паттерна надо проверять какие буквы совпадают/несовпадают в строке.

Reported by mamontov.dp on 2015-02-06 11:23:55

[Former user]

Нашел довольно странную ситуацию, просто переставив символы. Такое ощущение, что исправленная
строка как-то неправильно сформирована, или вообще я чего-то непонимаю?

Reported by mamontov.dp on 2015-02-16 18:42:09

[Former user]

http://pastebin.ru/799nwCRb

Reported by mamontov.dp on 2015-02-16 18:42:31

[Former user]

Залил. В correct_mistakes был обнаружен очень странный код. Переписал с учетом своих
знаний об архитектуре. 

Сделал функции построения отображений, правда необходимо сделать тесты. Пока меня смущает
то, что в них используется correct_mistakes частично, точнее информация, которая генерируется
внутри этого алгоритма. Мне кажется, что надо как-то отрефакторить его так, чтобы эти
данные были получены сразу а не потом.

Кроме этого, думаю поля, связанные с перечислениями, добавленные в коммите, будут нужны
и Вадиму.

Reported by mamontov.dp on 2015-02-16 20:21:35

[Oleg Sychev]

Дмитрий, для обсуждения этой задачи есть #323.
Перенесите коммент туда.

Reported by oasychev on 2015-02-17 23:06:15

[Oleg Sychev]

Reported by oasychev on 2015-07-03 20:16:40 - Status changed: Done