Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Отправлять сообщение "Обработка запроса в процессе" при большом тайминге запроса к БЗ #104

Open
stewkk opened this issue Nov 17, 2024 · 6 comments

Comments

@stewkk
Copy link
Collaborator

stewkk commented Nov 17, 2024

@stewkk , ещё ряд предложений:

  • сделать режим рабочий, в котором выводится только один (основной) ответ, для теста скорости. Основным пока оставить тестовый, с логами и тремя вариантами (пока работаем с БЗ, он идеален для этой цели). Посмотреть по таймаутам, и если модель в среднем в рабочем режиме думает больше, чем 30 секунд, добавлять пояснение, что ответ готовится.

  • аналогично с formalize - неплохо бы предупреждать пользователя о том, что происходит (формализация, передача в интерпретатор, проверка модели в smt-решателе).

Originally posted by @TonitaN in #73

Блокируется #103

@stewkk stewkk added this to rofl-lab1 Nov 17, 2024
@BaldiSlayer
Copy link
Owner

ну оно у нас всегда теперь отправляется, так что кажется можем закрыть

@BaldiSlayer BaldiSlayer moved this from Backlog to In review in rofl-lab1 Dec 22, 2024
@BaldiSlayer
Copy link
Owner

@stewkk закроем?

@stewkk
Copy link
Collaborator Author

stewkk commented Dec 22, 2024

@stewkk закроем?

так там идея была дополнительно юзеру писать после n секунд, что его запрос еще в обработке

@BaldiSlayer
Copy link
Owner

@stewkk закроем?

так там идея была дополнительно юзеру писать после n секунд, что его запрос еще в обработке

а, ну тогда не закрываем

@BaldiSlayer
Copy link
Owner

ну ваще кстати при пустой очереди я +- посмотрел, за 4-5 секунд отвечает в быстром режиме. За 9-11 в /multimodels. Вроде как-то так..

@BaldiSlayer
Copy link
Owner

BaldiSlayer commented Dec 22, 2024

@stewkk закроем?

так там идея была дополнительно юзеру писать после n секунд, что его запрос еще в обработке

ну халява кстати, делается с помощью контекста

шлем запрос в LLM, в соседней горутине что-то типа

go func(someCtx context.Context) {
  select {
    case <-time.After(30 * time.Second):
      // редачим сообщение и говорим, что мы еще 
      // не обработали и чел еще в очереди
      // наверное есть смысл еще сюда воткнуть mutex, а то будет смешно, если мы сначала воткнем
      // ответ LLM, а потом поменяем на то, что чел еще в очереди)) 
    case <-someCtx.Done():
  }
}(someCtx)

ну и типа делаем cancel(), когда получили ответ от LLM модуля

@BaldiSlayer BaldiSlayer moved this from In review to Ready in rofl-lab1 Jan 23, 2025
@stewkk stewkk moved this from Ready to In progress in rofl-lab1 Jan 24, 2025
@stewkk stewkk moved this from In progress to Ready in rofl-lab1 Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready
Development

No branches or pull requests

2 participants