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

Changed requirements.txt #933

Merged
merged 1 commit into from
Feb 28, 2024
Merged

Changed requirements.txt #933

merged 1 commit into from
Feb 28, 2024

Conversation

Abhishek0075
Copy link
Contributor

PR Request

Features
An issue, #926, has been identified within langchain v0.0.352, currently utilized within the repository. This issue manifests as errors within prompts, adversely affecting user experience. However, it has been adddressed and rectified in the latest version of langchain, v0.1.8

Feature Docs

Influence
The introduction of langchain v0.1.8 will have a significant impact on the stability and functionality of the repository. It addresses a critical error that was affecting the prompts, potentially improving user experience and reliability.

Result
2024-02-24 12:10:49.269 | INFO | metagpt.team:invest:90 - Investment: $3.0.
2024-02-24 12:10:49.284 | INFO | metagpt.roles.role:_act:399 - Alice(Product Manager): to do PrepareDocuments(PrepareDocuments)
2024-02-24 12:10:49.593 | INFO | metagpt.utils.file_repository:save:60 - save to: C:\Users\itsab\Documents\Github\MetaGPT\workspace\20240224121049\docs\requirement.txt
2024-02-24 12:10:49.593 | INFO | metagpt.roles.role:_act:399 - Alice(Product Manager): to do WritePRD(WritePRD)
2024-02-24 12:10:49.613 | INFO | metagpt.actions.write_prd:run:86 - New requirement detected: create a 2048 game
[CONTENT]
{
"Language": "en_us",
"Programming Language": "Python",
"Original Requirements": "create a 2048 game",
"Project Name": "game_2048",
"Product Goals": [
"Create an engaging user experience",
"Improve accessibility, be responsive",
"More beautiful UI"
],
"User Stories": [

Other

Please review and merge this PR to incorporate the latest version of langchain and resolve the issues outlined in #926.

@codecov-commenter
Copy link

codecov-commenter commented Feb 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.08%. Comparing base (06d7269) to head (e12dde9).
Report is 2 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #933      +/-   ##
==========================================
+ Coverage   81.19%   82.08%   +0.88%     
==========================================
  Files         226      226              
  Lines       12477    12477              
==========================================
+ Hits        10131    10242     +111     
+ Misses       2346     2235     -111     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@LAD021
Copy link

LAD021 commented Feb 26, 2024

Vote for this pull request.
I think the Unit Test only requires a rerun

@garylin2099
Copy link
Collaborator

Full Tests suggests no problems with components (such as memory) using langchain. LGTM. @better629 Please check and give review, too.

Copy link
Owner

@geekan geekan left a comment

Choose a reason for hiding this comment

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

LGTM

@geekan geekan merged commit 19b91b4 into geekan:main Feb 28, 2024
1 of 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.

5 participants