Skip to content

doc: Enhance negotiation tutorial with grid & tools#288

Open
niteshver wants to merge 10 commits intomesa:mainfrom
niteshver:improve_negiation_model_tut
Open

doc: Enhance negotiation tutorial with grid & tools#288
niteshver wants to merge 10 commits intomesa:mainfrom
niteshver:improve_negiation_model_tut

Conversation

@niteshver
Copy link
Copy Markdown
Contributor

This PR improves the existing negotiation model tutorial by transforming it from a basic walkthrough into a more structured and insightful guide.

Key enhancements include:

  • Add grid & tools

The goal is to make the tutorial more useful for both beginners and advanced users, while keeping it aligned with the existing example implementation.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 5, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a686f9e1-6a47-4a78-95e9-d9a5b82a44ff

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@niteshver
Copy link
Copy Markdown
Contributor Author

niteshver commented Apr 5, 2026

Hi @colinfrisch , @jackiekazil @wang-boyu
I’ve made an update to the negotiation tutorial to by adding grid and tools
I’d really appreciate your feedback on whether this direction looks good or if you’d like me to adjust anything.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 300cb8cc77

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

self.memory = STLTMemory(
agent=self,
llm_model="ollama/llama3",
llm_model="ollama/llama3.1:latest ",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Remove trailing space from configured Ollama model

The Seller tutorial snippet sets llm_model="ollama/llama3.1:latest " (note the trailing space), and ModuleLLM forwards model names as-is, so users who copy this code can end up requesting a non-existent model identifier from Ollama (llama3.1:latest instead of llama3.1:latest). This can cause avoidable model not found failures when running the tutorial.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have solve this

@wang-boyu wang-boyu added the docs Release notes label label Apr 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Release notes label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants