add metadata to summarizer response

[manupomax]

currently the "summarizer" node provides context only inside the json of the response

would be nice to have metadata of original document too in the json response, similarly to the response provided by other nodes like "retriever" and "reader"

[ZanSara]

Hello @manupomax, thanks for the issue! At a first glance it sounds easy to add to Haystack: would you mind taking the time to implement it? If you decide to contribute, please tag me in your PR and I'll handle the review process. Otherwise just stay tuned, soon enough somebody else will pick this one up 🙂

[SjSnowball]

@ZanSara I would like to work on this. Currently "context" field is passed to the result from the summarizer node. Along with that, we need to pass the fields retrieved from the retriever node. Did I understand correctly?

[ZanSara]

Hi @SjSnowball, thanks for taking this! I understood the proposal slightly different: I think Summarizer nodes do not contain the metadata of the document they summarized into the response. Other nodes, like Retrievers and Readers, do include the original document's metadata in their results, and I think @manupomax would simply like the original document's metadata to show up in the results of the Summarizer as well.

However, it would be fantastic if @manupomax could confirm which one of there two interpretations are correct 🙂

[sjrl]

Hey @ZanSara is it safe to close this issue, now that PR #2179 has been merged?

[ZanSara]

Done in #2179