bitcoinsearch / summarizer

Generation of easily accessible summarization feeds for public consumption
MIT License
5 stars 5 forks source link

Sometimes, only 2 participants are mentioned in a summary #46

Closed nourou4them closed 10 months ago

nourou4them commented 1 year ago

Context: Sometimes, only 2 participants are mentioned in a summary

Solution: To group all participants or to mention them all.

image

Extheoisah commented 10 months ago

This is beyond the scope of the frontend. This issue should be moved to the mailing-list-summaries backend

urvishp80 commented 10 months ago

Can you please elaborate the issue. Are you asking to add participants in "Involving" section? or in Summary Text that we are generating above?

Extheoisah commented 10 months ago

I think @nourou4them is proposing grouping all the participants together when making reference to the authors of the conversation. For example, in the screenshot attached, only Peter Todd and Antoine Richard is mentioned instead of Peter Todd, Antoine Richards and three others .... or Peter Todd et al., .... In my opinion, I don't think the current implementation is an issue but if improvements could be made and it isn't too much effort then I'm fine with it.

urvishp80 commented 10 months ago

Thank you for an elaboration @Extheoisah. I've pushed the changes in the main branch for the same. @nourou4them