Skip to content

doc: Camel MCP cookbook #2310

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

doc: Camel MCP cookbook #2310

wants to merge 2 commits into from

Conversation

MuggleJinx
Copy link
Collaborator

Description

Close #2092.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@MuggleJinx MuggleJinx requested a review from Wendong-Fan May 2, 2025 08:11
Copy link

review-notebook-app bot commented May 2, 2025

View / edit / reply to this conversation on ReviewNB

Wendong-Fan commented on 2025-05-02T08:42:18Z
----------------------------------------------------------------

the install output could be cleaned


Copy link

review-notebook-app bot commented May 2, 2025

View / edit / reply to this conversation on ReviewNB

Wendong-Fan commented on 2025-05-02T08:42:19Z
----------------------------------------------------------------

should we remove this?


Copy link

review-notebook-app bot commented May 2, 2025

View / edit / reply to this conversation on ReviewNB

Wendong-Fan commented on 2025-05-02T08:42:20Z
----------------------------------------------------------------

for the image we could reuse from https://www.camel-ai.org/blogs/owl-mcp-toolkit-practice with camel logo


Copy link

review-notebook-app bot commented May 2, 2025

View / edit / reply to this conversation on ReviewNB

Wendong-Fan commented on 2025-05-02T08:42:20Z
----------------------------------------------------------------

I think it should be ACI.dev instead of ACPI.dev,also we could put ACI.dev as first one


Copy link

View / edit / reply to this conversation on ReviewNB

Wendong-Fan commented on 2025-05-02T08:42:21Z
----------------------------------------------------------------

use CAMEL instead of Camel


Copy link
Member

@Wendong-Fan Wendong-Fan left a comment

Choose a reason for hiding this comment

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

thanks @MuggleJinx , left some comments above

Copy link

View / edit / reply to this conversation on ReviewNB

lightaime commented on 2025-05-04T00:47:46Z
----------------------------------------------------------------

Line #2.    from camel.toolkits.mcp_toolkit import MCPToolkit, MCPClient
MCPClient import but not used

@MuggleJinx MuggleJinx requested a review from Wendong-Fan May 5, 2025 06:12
@MuggleJinx
Copy link
Collaborator Author

Thanks @Wendong-Fan @lightaime , updated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[Feature Request] MCP tool and CAMEL tool cookbook and doc
2 participants