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

Removed deprecated llm attribute for load_chain #5343

Merged

Conversation

imeckr
Copy link
Contributor

@imeckr imeckr commented May 27, 2023

Removed deprecated llm attribute for load_chain

Currently load_chain for some chain types expect llm attribute to be present but llm is deprecated attribute for those chains and might not be persisted during their chain.save.

Fixes #5224 (issue)

Who can review?

@hwchase17
@dev2049

Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

thanks for this! one comment

langchain/chains/loading.py Show resolved Hide resolved
langchain/chains/loading.py Show resolved Hide resolved
langchain/chains/loading.py Show resolved Hide resolved
@imeckr
Copy link
Contributor Author

imeckr commented May 28, 2023

@hwchase17 thanks for the comment, I have made the changes.

Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

I guess this also isn’t backwards compatible… would be nice to maintain that

@imeckr
Copy link
Contributor Author

imeckr commented May 29, 2023

I guess this also isn’t backwards compatible… would be nice to maintain that

@hwchase17 added

Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

thanks!

@hwchase17 hwchase17 merged commit 642ae83 into langchain-ai:master May 29, 2023
12 checks passed
vowelparrot pushed a commit that referenced this pull request May 31, 2023
# Removed deprecated llm attribute for load_chain

Currently `load_chain` for some chain types expect `llm` attribute to be
present but `llm` is deprecated attribute for those chains and might not
be persisted during their `chain.save`.

Fixes #5224
[(issue)](#5224)

## Who can review?
@hwchase17
@dev2049

---------

Co-authored-by: imeckr <chandanroutray2012@gmail.com>
@danielchalef danielchalef mentioned this pull request Jun 5, 2023
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
# Removed deprecated llm attribute for load_chain

Currently `load_chain` for some chain types expect `llm` attribute to be
present but `llm` is deprecated attribute for those chains and might not
be persisted during their `chain.save`.

Fixes langchain-ai#5224
[(issue)](langchain-ai#5224)

## Who can review?
@hwchase17
@dev2049

---------

Co-authored-by: imeckr <chandanroutray2012@gmail.com>
This was referenced Jun 25, 2023
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.

PALChain loading fails
2 participants