Skip to content
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

CamelAI Readme and Example Journal implemented! #538

Merged
merged 43 commits into from
Dec 16, 2024

Conversation

TKTSWalker
Copy link
Contributor

📥 Pull Request

📘 Description
Added two example journals and a dedicated readme, also added CamelAI section to opening readme.

🧪 Testing
Journals working are reflected by their end output being present in the files.

@TKTSWalker
Copy link
Contributor Author

fixes #521

@the-praxs
Copy link
Member

@TKTSWalker what's the status of this?

@TKTSWalker
Copy link
Contributor Author

@TKTSWalker
Copy link
Contributor Author

Just realized I put anthropic instead of camel with the description, fixing!

Fixed Anthropic to CamelAI at CamelAI description
Fixed "Usage Example" CamelAI link
@TKTSWalker
Copy link
Contributor Author

Fixed (as well as two minor errors on the readme end) while also syncing!

@the-praxs
Copy link
Member

is this good to merge?

@TKTSWalker
Copy link
Contributor Author

Yes, it's good to push!

Copy link
Member

@the-praxs the-praxs left a comment

Choose a reason for hiding this comment

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

Add the docs to a new file camelai.mdx in docs/v1/integrations/examples/ and add the entry to docs/v1/examples/examples.mdx

examples/camelai_examples/camelaiguide.md Outdated Show resolved Hide resolved
examples/camelai_examples/camelaiguide.md Outdated Show resolved Hide resolved
examples/camelai_examples/camelaiguide.md Outdated Show resolved Hide resolved
@the-praxs
Copy link
Member

If the spreadsheet can be fetched through a URL in the code without having to upload it in the repo then that's better.

@the-praxs the-praxs self-requested a review December 13, 2024 15:33
@TKTSWalker
Copy link
Contributor Author

i purposefully left it as a repo because i'm not sure where I could store it! For example, I often delete databases I made from my Kaggle and Google Sheets! I'd recommend making an official kaggle on the AgentOps side and dropping it in there since it'll be the least likely to randomly disappear!

@areibman
Copy link
Contributor

FYI @TKTSWalker Devin (the AI) made a PR for this already but I like your examples, so we can merge those as well!

#573

@TKTSWalker
Copy link
Contributor Author

Sounds solid to me!

Copy link
Contributor

@areibman areibman left a comment

Choose a reason for hiding this comment

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

Great!

@areibman areibman enabled auto-merge (squash) December 16, 2024 07:58
@areibman areibman merged commit 9fc6612 into AgentOps-AI:main Dec 16, 2024
2 checks passed
@TKTSWalker
Copy link
Contributor Author

TKTSWalker commented Dec 16, 2024 via email

@the-praxs
Copy link
Member

I am opening a PR for some docs fixes. Will mention you so that you can have a look.

@TKTSWalker
Copy link
Contributor Author

TKTSWalker commented Dec 16, 2024 via email

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.

4 participants