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

549 kaizen cli make easy setup with ollama #558

Merged
merged 3 commits into from
Sep 17, 2024

Conversation

sauravpanda
Copy link
Member

@sauravpanda sauravpanda commented Sep 17, 2024

Optimize Embedding Usage in LLaMA Index Retriever

  • Purpose:
    Reduce the embedding usage in the LLaMA Index Retriever to improve performance.
  • Key Changes:
    • Set the initial embedding_usage values for prompt_tokens and total_tokens to 0.
    • Update the return value of the setup_repository method to include the updated embedding_usage instead of the embed_model.
  • Impact:
    The changes will reduce the memory and computational resources required by the LLaMA Index Retriever, potentially improving the overall performance of the system.

✨ Generated with love by Kaizen ❤️

Original Description

Copy link
Contributor

kaizen-bot bot commented Sep 17, 2024

🔍 Code Review Summary

All Clear: This commit looks good! 👍

Overview

  • Total Feedbacks: 0 (Critical: 0, Refinements: 0)
  • Files Affected: 0
  • Code Quality: [█████████████████░░░] 85% (Good)
Test Cases

3 file need updates to their tests. Run !unittest to generate create and update tests.


✨ Generated with love by Kaizen ❤️

Useful Commands
  • Feedback: Reply with !feedback [your message]
  • Ask PR: Reply with !ask-pr [your question]
  • Review: Reply with !review
  • Explain: Reply with !explain [issue number] for more details on a specific issue
  • Ignore: Reply with !ignore [issue number] to mark an issue as false positive
  • Update Tests: Reply with !unittest to create a PR with test changes

@sauravpanda sauravpanda merged commit 252efbb into main Sep 17, 2024
3 checks passed
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.

1 participant