langchain-ai/docs
1
0
Fork 0
mirror of https://github.com/langchain-ai/docs.git synced 2026-07-21 11:55:23 -04:00
Code Issues 490 Packages Projects Releases Wiki Activity

[PR #2940] [MERGED] OCI Generative AI docs: Quality improvements and bug fixes #3024

New Issue
Closed
opened 2026-06-05 18:20:52 -04:00 by yindo · 0 comments
yindo commented 2026-06-05 18:20:52 -04:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/2940
Author: @fede-kamel
Created: 3/5/2026
Status: ✅ Merged
Merged: 3/5/2026
Merged by: @mdrxy

Base: main ← Head: oci-docs-improvements

📝 Commits (3)

  • 9ed583f Improve documentation quality and examples
  • a40cb1d Fix Gemini multimodal format: use document_url instead of media
  • 2568d89 Simplify examples: use strings and tuples where HumanMessage not needed

📊 Changes

3 files changed (+173 additions, -140 deletions)

View changed files

📝 src/oss/python/integrations/chat/oci_generative_ai.mdx (+138 -100)
📝 src/oss/python/integrations/providers/oci.mdx (+20 -24)
📝 src/oss/python/integrations/text_embedding/oci_generative_ai.mdx (+15 -16)

📄 Description

Summary

Follow-up improvements to OCI Generative AI documentation (#2925) based on testing this morning. Apologies for the separate PR - I was working on these improvements when the original was merged.

Changes:

  1. Add example outputs - Show actual model responses so developers know what to expect
  2. Complete tool calling flow - Added full ToolMessage execution loop (was missing)
  3. Fix Gemini PDF format - Changed {"type": "media", ...} to correct {"type": "document_url", ...} format
  4. Simplify examples - Use strings/tuples where HumanMessage isn't required:
    • llm.invoke("question") instead of llm.invoke([HumanMessage(...)])
    • ("user", "..."), ("assistant", "...") tuples for multi-turn

Testing

All 13 integration tests pass against real OCI GenAI services:

  • Basic invocation, multi-turn, streaming, async
  • Tool calling with complete execution loop
  • Structured output with Pydantic
  • Vision (Llama 3.2 90B)
  • Gemini PDF processing
  • Text & image embeddings (Cohere)
  • RAG with FAISS
  • AI Agent (create_oci_agent)

Files Changed

  • src/oss/python/integrations/chat/oci_generative_ai.mdx
  • src/oss/python/integrations/providers/oci.mdx
  • src/oss/python/integrations/text_embedding/oci_generative_ai.mdx

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/docs/pull/2940 **Author:** [@fede-kamel](https://github.com/fede-kamel) **Created:** 3/5/2026 **Status:** ✅ Merged **Merged:** 3/5/2026 **Merged by:** [@mdrxy](https://github.com/mdrxy) **Base:** `main` ← **Head:** `oci-docs-improvements` --- ### 📝 Commits (3) - [`9ed583f`](https://github.com/langchain-ai/docs/commit/9ed583f834695a96c7182838a6a764ac140e8062) Improve documentation quality and examples - [`a40cb1d`](https://github.com/langchain-ai/docs/commit/a40cb1d219bf312e4e74d750883242a78c012bbb) Fix Gemini multimodal format: use document_url instead of media - [`2568d89`](https://github.com/langchain-ai/docs/commit/2568d89d789e5aedc3a4d8b5756674c98cb5d2e7) Simplify examples: use strings and tuples where HumanMessage not needed ### 📊 Changes **3 files changed** (+173 additions, -140 deletions) <details> <summary>View changed files</summary> 📝 `src/oss/python/integrations/chat/oci_generative_ai.mdx` (+138 -100) 📝 `src/oss/python/integrations/providers/oci.mdx` (+20 -24) 📝 `src/oss/python/integrations/text_embedding/oci_generative_ai.mdx` (+15 -16) </details> ### 📄 Description ## Summary Follow-up improvements to OCI Generative AI documentation (#2925) based on testing this morning. Apologies for the separate PR - I was working on these improvements when the original was merged. **Changes:** 1. **Add example outputs** - Show actual model responses so developers know what to expect 2. **Complete tool calling flow** - Added full `ToolMessage` execution loop (was missing) 3. **Fix Gemini PDF format** - Changed `{"type": "media", ...}` to correct `{"type": "document_url", ...}` format 4. **Simplify examples** - Use strings/tuples where `HumanMessage` isn't required: - `llm.invoke("question")` instead of `llm.invoke([HumanMessage(...)])` - `("user", "..."), ("assistant", "...")` tuples for multi-turn ## Testing All 13 integration tests pass against real OCI GenAI services: - Basic invocation, multi-turn, streaming, async - Tool calling with complete execution loop - Structured output with Pydantic - Vision (Llama 3.2 90B) - Gemini PDF processing - Text & image embeddings (Cohere) - RAG with FAISS - AI Agent (`create_oci_agent`) ## Files Changed - `src/oss/python/integrations/chat/oci_generative_ai.mdx` - `src/oss/python/integrations/providers/oci.mdx` - `src/oss/python/integrations/text_embedding/oci_generative_ai.mdx` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-06-05 18:20:52 -04:00
yindo closed this issue 2026-06-05 18:20:52 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
deepagents
docs-infra
external
external
help wanted
integration
internal
internal
javascript
js/ts
langchain
langgraph
langsmith
needs-title
pull-request
Mirrored from GitHub Pull Request
 python
reference
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#3024
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?