Skip to content

Conversation

@chenopis
Copy link

@chenopis chenopis commented Jan 14, 2026

Content Updates

  • Removed redundant TOC — The toc: true in frontmatter already handles this
  • Rewrote Overview — Clearer description of the package, API catalog, and NIM
  • Added Prerequisites section — Step-by-step API key setup instructions

Model Updates

  • Embedding model: nvolveqa_40k → nvidia/llama-3.2-nv-embedqa-1b-v2 (current model)
  • Generator model: nv_llama2_rlhf_70b → meta/llama-3.1-70b-instruct (current model)

Typo Fixes

  • floud → flour
  • Omlette → Omelet

Formatting Improvements

  • Consistent heading structure (## for main sections, ### for components)
  • Better code block formatting with proper spacing
  • Cleaner component descriptions using bold labels
  • Improved code example output comments

New Section

  • Added "Self-host with NVIDIA NIM" section

Code Example Cleanup

  • Removed unnecessary parameters in NvidiaGenerator example (seed, bad, stop → cleaner defaults)
  • Fixed data dict formatting in pipeline examples for readability
  • Added missing import in RAG pipeline example (from haystack import Pipeline)

Signed-off-by: Andrew Chen <andrewch@nvidia.com>
@chenopis chenopis requested a review from a team as a code owner January 14, 2026 22:05
Copy link
Contributor

@bilgeyucel bilgeyucel left a comment

Choose a reason for hiding this comment

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

Thanks @chenopis for the changes! Left some minor comments 🙌

chenopis and others added 2 commits January 20, 2026 15:58
Co-authored-by: Bilge Yücel <bilge.yucel@deepset.ai>
Signed-off-by: Andrew Chen <andrewch@nvidia.com>
@chenopis
Copy link
Author

Thanks @chenopis for the changes! Left some minor comments 🙌

@bilgeyucel Thanks for your feedback. I integrated your suggestions, kept the ToC and updated it. PTAL

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