langchain
langchain copied to clipboard
Fixed multi input prompt for MapReduceChain
Fixed multi input prompt for MapReduceChain
Added kwargs support for inner chains of MapReduceChain via from_params method
Currently the from_method method of intialising MapReduceChain chain doesn't work if prompt has multiple inputs. It happens because it uses StuffDocumentsChain and MapReduceDocumentsChain underneath, both of them require specifying document_variable_name if prompt of their llm_chain has more than one input.
With this PR, I have added support for passing their respective kwargs via the from_params method.
Fixes https://github.com/hwchase17/langchain/issues/4752
Who can review?
@dev2049 @hwchase17 @agola11
@dev2049 @hwchase17 @agola11 Can anyone of you review this ?
could we add either an example to the map reduce notebook or an integration test where this is used
@dev2049 Let me know if this example works.