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

Feat/prompt description fix #46

Closed
wants to merge 15 commits into from
Closed

Conversation

AnniePacheco
Copy link

Summary

This PR includes the following fixes

  • Changed chat_response to be only string and not try it to be compliant to certain schema (textLong)
  • Changed source of description to be description from a persona and included in the system prompt
  • Fixed a bug, while trying to index a pdf file with more than one page

Changes

  • Updated yaml apis/paios/openapi.yaml chat_response type string
  • Updated bakend app.py to allow_origins 'http://localhost:3000'
  • Updated RagManager change assistant description to be persona description

Related Issues

N/A

Type of Change

New feature (non-breaking change which adds functionality)
Fix past bugs

Checklist

  • I have read the CONTRIBUTING documentation.
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have updated the documentation (if applicable).

Screenshots (if applicable) Example of use:

Additional Context

Tested locally using Swagger.

@AnniePacheco AnniePacheco self-assigned this Sep 12, 2024
@AnniePacheco AnniePacheco added bug Something isn't working enhancement New feature or request labels Sep 12, 2024
@samj samj added assistant pAI Assistant Application and removed enhancement New feature or request labels Oct 2, 2024
@samj samj closed this Jan 16, 2025
@samj samj deleted the feat/prompt-description-fix branch January 18, 2025 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assistant pAI Assistant Application bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants