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

Refactor prompt and Dependency Update #358

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Refactor prompt and Dependency Update #358

merged 1 commit into from
Oct 10, 2023

Conversation

Ashesh3
Copy link
Member

@Ashesh3 Ashesh3 commented Oct 10, 2023

This PR encompasses a series of modifications aimed at refining the code and updating dependencies. Changes include:

  • Updated the dependencies in requirements/base.txt.
  • Corrected typographical errors in ayushma/utils/langchain.py and ayushma/utils/openaiapi.py to enhance code readability.
  • Amended the user_msg parameter passing in get_aresponse and get_response methods in ayushma/utils/langchain.py.
  • Updated the message formatting in converse method within ayushma/utils/openaiapi.py to ensure consistency in message handling.

Note: The dependency update is required to add support for other LLM models which are not under OpenAI

@Ashesh3 Ashesh3 requested a review from skks1212 October 10, 2023 07:25
@vercel
Copy link

vercel bot commented Oct 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ayushma ✅ Ready (Inspect) Visit Preview Oct 10, 2023 7:25am

@skks1212 skks1212 merged commit dd955c9 into master Oct 10, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants