-
Notifications
You must be signed in to change notification settings - Fork 17
News article automated summaries generation #1906
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: develop
Are you sure you want to change the base?
Conversation
hard to tell which ones are generated and which are written by hand |
Output v2, after adding "Write summaries in an impersonal, passive voice, never attributing actions to 'author' or similar." to the prompt at the request of @vinniefalco. It's narrow just because of the PDF generation process. |
c12112d
to
8254c4f
Compare
test failures will be resolved when OPENROUTER_API_KEY is added to the envs. Do not merge before this has been completed. |
8254c4f
to
1ac4512
Compare
SUPER irritating that I have to download the PDF from GitHub instead of viewing it inline in my browser. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very cool! Couple of questions for you.
1ac4512
to
49fee93
Compare
@GregKaleka this needs your review - thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - nice work.
This PR adds triggering of news article summarization using gpt-oss-120b from openrouter.
Openrouter was chosen because it allows easy switching of models even across multiple providers like OpenAI or Anthropic, without having to set up new accounts, as well as easy access to some more niche models, like Gpt-oss-120b.
Gpt-oss-120b was chosen because for summarization it's capable enough, and is cheaper than gpt-5 nano in openrouter.
Where the content or link contents are less than 1000 characters, we will continue to use those as is on the news listing page, otherwise we use the summary.
Attached is a PDF of the final full news page with summaries where back-populated.
news.pdfSee later PDF.