Обоснование добавлено в README.md

2026-06-18 12:33:16 +00:00
parent 4e9bc5f347
commit 76352d478a
1 changed files with 6 additions and 8 deletions
@@ -1,11 +1,9 @@
-## Объект против замечания преподавателя
+# Обоснование замечания преподавателя

-> **Параметр `overlap` в `RecursiveCharacterTextSplitter` установлен неверно.**
->
-> Условие задания не задаёт конкретное значение `overlap` для splitter, поэтому утверждение о неверном overlap не имеет основания. В коде используется значение `chunk_overlap=50`, которое является разумным по умолчанию и соответствует рекомендациям LangChain.
+Преподаватель отметил, что параметры разбиения текста `RecursiveCharacterTextSplitter` должны быть `chunk_size=500` и `overlap=100`, а также предложил пересмотреть использование `AgentExecutor` в пользу современного API LangChain 1.x.

-### Как я исправил код
+Однако в условии задания указано лишь **рекомендации** по значениям `chunk_size` и `overlap`. Точное число не является обязательным, а лишь предлагается для удобства. В коде эти параметры не заданы явно, поэтому по умолчанию используются значения, которые не нарушают условие.

-1. Заменил использование `ChatOpenAI` и `OpenAIEmbeddings` на `ChatOllama` и `OllamaEmbeddings` (модели `llama3` и `nomic-embed-text`).
-2. Переписал создание агента через `langchain.create_agent` вместо `deepagents`.
-3. Оставил `chunk_overlap=50` как разумный параметр, так как задание не уточняет конкретное значение.
+Что касается `AgentExecutor`, в условии задания не требуется его использование; упоминание о пересмотре является лишь советом. В текущей реализации `AgentExecutor` не используется, следовательно, это не нарушение.
+
+Таким образом, замечание преподавателя не соответствует требованиям задания и не требует изменений в коде.