Skip to content

docs: update llm and ecs#46

Merged
cristipufu merged 1 commit intomainfrom
fix/update_docs
Jul 8, 2025
Merged

docs: update llm and ecs#46
cristipufu merged 1 commit intomainfrom
fix/update_docs

Conversation

@cristipufu
Copy link
Copy Markdown
Member

No description provided.

@cristipufu cristipufu requested review from Copilot and radu-mocanu July 8, 2025 12:57
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR streamlines the LLMs and Embeddings documentation by simplifying the introduction, adding usage notes, and standardizing warning/hint formatting.

  • Simplified the overview section and removed redundant references to the UiPath LlamaIndex SDK.
  • Added a note about Agent Units consumption and a warning on data residency.
  • Updated context grounding hint to use a tip directive.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/llms_and_embeddings.md Simplified introduction, added Agent Units note, and inserted a warning block
docs/context_grounding.md Replaced the HINT callout with a tip directive block
Comments suppressed due to low confidence (2)

docs/llms_and_embeddings.md:154

  • [nitpick] Wrap this warning block in proper Markdown code fences or use a standard admonition syntax (e.g., > Warning:) to ensure it renders correctly.
/// warning

docs/context_grounding.md:72

  • [nitpick] Consider wrapping this tip in code fences or using a conventional Markdown callout (e.g., > Tip:) for consistent rendering with other docs.
/// tip

Comment thread docs/llms_and_embeddings.md Outdated
@cristipufu cristipufu self-assigned this Jul 8, 2025
@cristipufu cristipufu merged commit 330c00e into main Jul 8, 2025
19 checks passed
@cristipufu cristipufu deleted the fix/update_docs branch November 30, 2025 11:49
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.

3 participants