Skip to content

PR для HW3 - #6

Open
o1eg0 wants to merge 2 commits into
mainfrom
hw3
Open

PR для HW3#6
o1eg0 wants to merge 2 commits into
mainfrom
hw3

Conversation

@o1eg0

@o1eg0 o1eg0 commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@xufana

xufana commented Mar 29, 2026

Copy link
Copy Markdown

Вот где-то тут я была б признательна, если бы и соответствующий код вошёл бы в PR, потому что я , хоть убейте, не могу найти, куда и как вы отправляете трейсы.

Не хватило мониторинга и логгирования (или я не нашла об этом никаких упоминаний). Как вы определяете, как долго отвечает бот? Как вы определяете, а жив ли он вообще и шлёте алерты, если умер? Промпт, надо сказать, достаточно простой, поэтому хотелось бы видеть какую-то информацию об экспериментах с ним. Ну и замеров по latency и error-rate.

Всё ещё лайк за poetry, yet это и докер это минимум этой части дз.

4/10 + 1 балл = 5

@xufana

xufana commented Mar 31, 2026

Copy link
Copy Markdown

My bad, не увидела феникса — тогда снято замечание про latency и трейсы, но остаётся вопрос про мониторинг здоровья и алерты; так что
6/10 + 1 =7/10

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.

3 participants