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

fix: move relevant chunks before current conversations #772

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

Pouyanpi
Copy link
Collaborator

Otherwise the bot will generate based on the relevant chunks only.

@Pouyanpi
Copy link
Collaborator Author

@drazvan would you have a look? Please try out abc_v2 on develop branch to see the issue. Can it be included in the patch release?

@Pouyanpi Pouyanpi self-assigned this Oct 1, 2024
@schuellc-nvidia
Copy link
Collaborator

I think we need to better test the introduced change in #745.
Changing the prompts like that can result in unexpected behavior. E.g. I would suggest removing the markdown notation. Also, we need to take a look at the prompt templates for the GPT 4 and Llama 3 models.

@Pouyanpi
Copy link
Collaborator Author

@schuellc-nvidia, are we good to merge this PR considering #809 ?

Copy link
Collaborator

@schuellc-nvidia schuellc-nvidia left a comment

Choose a reason for hiding this comment

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

All good for me

@Pouyanpi Pouyanpi merged commit 2626bd5 into develop Oct 18, 2024
4 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.

2 participants