-
Notifications
You must be signed in to change notification settings - Fork 51
fix: update import path for message classes #641
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
base: main
Are you sure you want to change the base?
Conversation
Hey @hefeng6500! This is intended actually -- we're thinking it's better for the end user if you're importing everything you need from one package. The only time you would need to import these from core is when you're writing integrations to avoid the dependency bloat. In light of that, going to close this. |
Hi @hntrl, thanks for the review and for keeping imports minimal for end users. from langchain.messages import HumanMessage, AIMessage, SystemMessage I get the following error: ModuleNotFoundError: No module named 'langchain.messages' It only works when I use: from langchain_core.messages import HumanMessage, AIMessage, SystemMessage Could you confirm the correct import path for the docs examples? Otherwise, users following the documentation will run into the same error. |
apologies, this was a miscomm on my part! We're currently working on adding these to |
Hi @hntrl , |
@mcavdar @hefeng6500 We can reopen this and merge, then when |
No description provided.