feat(max): Make summary more actionable and self-aware #27005
Conversation
|
Size Change: 0 B Total Size: 1.11 MB ℹ️ View Unchanged
|
Twixes
force-pushed
the
assistant-self-aware
branch
from
02a7d0c to
6ab4d88
Compare
This reverts commit 705c1d6.
docker-compose.base.yml
Outdated
| ports: | ||
| - 3000:3000 |
ee/hogai/summarizer/prompts.py
Outdated
| @@ -1,17 +1,24 @@ | |||
| SUMMARIZER_SYSTEM_PROMPT = """ | |||
| Act as an expert product manager. Your task is to summarize query results in a a concise way. | |||
| Act as an expert product manager. Your task is to help the user build a successful product and business. Remember, you're a hedeghog. | |||
There was a problem hiding this comment.
Why a hedgehog role?😆 Do you want to force it to answer as a hedgehog?
There was a problem hiding this comment.
It's more setting the ground for the pure conversation skill, but we definitely will want to be aware it's Max the Hedgehog rather than "expert product manager, an OpenAI large language model"
ee/hogai/summarizer/nodes.py
Outdated
| @@ -100,5 +104,6 @@ def _construct_messages(self, state: AssistantState) -> list[tuple[str, str]]: | |||
| elif isinstance(message, AssistantMessage): | |||
| conversation.append(("assistant", message.content)) | |||
|
|
|||
| conversation.append(("human", SUMMARIZER_INSTRUCTION_PROMPT)) | |||
| conversation.append(("system", SUMMARIZER_RESULTS_PROMPT)) | |||
There was a problem hiding this comment.
Two system prompts? It's better to keep a single system prompt.
There was a problem hiding this comment.
Was playing with a different approach, but actually better to consolidate. Pushed
ee/hogai/summarizer/prompts.py
Outdated
|
|
||
| If there are interesting trends or anomalies, succintly point them out. I can see the chart, so don't just describe all of it. | ||
| The answer needs to be high-impact, no more than a few sentences. Bullets can improve clarity of action points. | ||
| Use Silicon Valley lingo. Be informal, but without fluff. NEVER USE TITLE CASE, even in headings. Our style is sentence case always. |
There was a problem hiding this comment.
Why didn't you include the language style instructions in the first system prompt? Or you can concatenate the second system prompt and the user prompt.
Makes us hit context limits!
Problem
Agent doesn't always recognize it hasn't really answered the question.
Changes
This tweaks the summarizer to be more self-aware and self-critical.
How did you test this code?
TODO!