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

[Bug]: Import Conflict @ gemini/reasoning-engine/tutorial_langgraph_rag_agent.ipynb #1564

Open
1 task done
samjtro opened this issue Dec 19, 2024 · 4 comments
Open
1 task done
Assignees

Comments

@samjtro
Copy link

samjtro commented Dec 19, 2024

File Name

gemini/reasoning-engine/tutorial_langgraph_rag_agent.ipynb

What happened?

using the pip install command provided in the "Install and import dependencies" section of this notebook fails due to a dependency conflict between langchain-google-cloud-sql & cloud-sql-python-connector.

changed it from cloud-sql-python-connector==1.9.0 to >=1.9.0. can submit a pr if you want me to save you a line.

6ea39f0

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@koverholt
Copy link
Member

@shawn-yang-google, the mentioned notebook is returning an error on the pip install line:

ERROR: Cannot install cloud-sql-python-connector==1.9.0 and langchain-google-cloud-sql-pg==0.6.1 because these package versions have conflicting dependencies.
ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Can you take a look at bumping versions to fix? Or ideally, test with unpinned latest versions.

@shawn-yang-google
Copy link
Contributor

Sure, looking at it

@shawn-yang-google
Copy link
Contributor

shawn-yang-google commented Dec 20, 2024

I reran the colab and confirmed reproducing the same issue.

Change the cloud-sql-python-connector==1.9.0 to >=1.9.0 did solve the issue.

In the PR,
I leaved a comment: we should also change line 966 (where we define the requirements for remote agent)

@samjtro if you could, please submit the PR. Thanks in advance!

@samjtro
Copy link
Author

samjtro commented Dec 20, 2024

10/4! @shawn-yang-google just to be clear, would you like me to include the change to line 966 in my pr?

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

No branches or pull requests

3 participants