Skip to content

[FIX] Fix tool calling result parsing problem in tranjectory visualizer & MCP tool name fixing#174

Merged
jeffreysijuntan merged 5 commits into
rllm-org:mainfrom
VincentXWD:main
Aug 6, 2025
Merged

[FIX] Fix tool calling result parsing problem in tranjectory visualizer & MCP tool name fixing#174
jeffreysijuntan merged 5 commits into
rllm-org:mainfrom
VincentXWD:main

Conversation

@VincentXWD
Copy link
Copy Markdown
Contributor

[UPD] update tavily tool name since tavily server returns mismatch tool names. Also had a tool calling result visualization problem on tranjectory visualizer.

Tavily MCP server running on stdio

Connected to MCP server with tools: ['tavily-search', 'tavily-extract', 'tavily-crawl', 'tavily-map']
Initialized 4 MCP tools
Available tools: ['tavily-search', 'tavily-extract', 'tavily-crawl', 'tavily-map']

Before the change list:
image

After the change list:
image

@VincentXWD VincentXWD changed the title [FIX] Update tavily tool name since tavily server returns mismatch [FIX] Fix tool calling result parsing problem in tranjectory visualizer & MCP tool name fixing Aug 5, 2025
@jeffreysijuntan
Copy link
Copy Markdown
Contributor

Thank your for your contribution. Currently pre-commit (ruff) check fails. You can run pre-commit install and pre-commit run --all-files to automatically format the files and make a commit. I will get it merged once pre-commit passes.

@VincentXWD
Copy link
Copy Markdown
Contributor Author

pre-commit run --all-files

Thanks for the reminder.

@jeffreysijuntan jeffreysijuntan merged commit 145a87b into rllm-org:main Aug 6, 2025
1 check passed
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.

2 participants