OnionGrief / Chipollino

преобразования регулярных выражений и конечных автоматов
Other
19 stars 4 forks source link

Colorize & LogTemplate & Tester (#268, #269) #292

Closed TonitaN closed 10 months ago

TonitaN commented 10 months ago

Что не нравится:

xendalm commented 10 months ago

Циклическую зависимость удалось починить предефайном FiniteAutomaton (если я правильно понял идею). С вашего позволения залью правки

TonitaN commented 10 months ago

Отлично, если удалось! Тогда передавать в set_parameter можно будет сразу объект MetaInfo, что сэкономит буквы в реализации методов (отрефакторю всё немедленно как зальете изменения).

P.S. - лог графиков улучшен + логгер переведён на unordered_map

xendalm commented 10 months ago

Тогда передавать в set_parameter можно будет сразу объект MetaInfo, что сэкономит буквы в реализации методов (отрефакторю всё немедленно как зальете изменения)

Не собирался отнимать у вас работу, просто сам отрефакторил по привычке

TonitaN commented 10 months ago

О, работы у нас ещё много, следующая на очереди - интеграция кода Эдгара и метода Normalize, так что я ничуть не в обиде)

mathhyyn commented 10 months ago

Пробую тестить графики. Так должно быть пока, что они не добавлены в шаблоны? почему то отрендеренный шаблон съедает все, включая любой текст и переносы строк, перед графиком (и после)

step для оси Y выставляется в 0, если тест слишком простой (Test {abab} {(ab)*} 1) image

и не ясно что означает столбик "кол-во итераций" (раньше им был самый первый - кол-во раскрытий итераций Клини, т.е. числа от 0 до 12 (только сейчас поняла что мы зачем то добавляли к ним 1...), а сейчас кол-во шагов по автомату (я так понимаю, максимальное)) я бы переименовала, а то пришлось гадать за что столбец отвечает

TonitaN commented 10 months ago

Пробую тестить графики. Так должно быть пока, что они не добавлены в шаблоны? почему то отрендеренный шаблон съедает все, включая любой текст и переносы строк, перед графиком (и после)

Хаха, у меня они были только в папке build/resources, а не в корне. Исправлено. График с таблицей переносятся на другую страницу, но исправление этого пока в todo (расчётная высота таблицы намного больше, чем реальная, из-за эффектов adjustbox).

TonitaN commented 10 months ago

Всякие error_refal.txt и err.txt теперь мусор: все логи ошибок рефала, во-первых, имеют имена, относящиеся к конкретным модулям, а во-вторых, расширение raux (вспомогательный файл для рефала). Поэтому старые txt-шки (вышеупомянутый мусор) я убрала из gitignore: их больше не должно появляться в принципе.

TonitaN commented 10 months ago

Подождём, может, что-нибудь ещё Аня заметит. А вы могли бы и пнуть за неконстантные аргументы, зря постеснялись)