Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Feat/summarize improvements#486

Merged
mahaloz merged 4 commits intomainfrom
feat/summarize_improvements
Dec 5, 2025
Merged

Feat/summarize improvements#486
mahaloz merged 4 commits intomainfrom
feat/summarize_improvements

Conversation

@LukePerezz
Copy link
Contributor

@LukePerezz LukePerezz commented Nov 19, 2025

Added gpt-5 model, and created a dialog that lets user enter their openai api key for easier usage of the summarize button.
This is 2 commits behind main, but was unsure if its an issue since everything seems good. Only files changed are summarize.py and progress_window,py.

Closes #483

@LukePerezz LukePerezz self-assigned this Nov 19, 2025
# Summarize button
self.summarize_button = QPushButton("Summarize")
self.summarize_button.clicked.connect(self.summarize)
self.summarize_button.clicked.connect(self.checkApi)
Copy link
Member

Choose a reason for hiding this comment

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

Keep this as calling summarize, but inside the function, call checkApi.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

put self.checkApi under the check for extras and above the save file section of summarize. Changed the button function to self.summarize again

@mahaloz mahaloz merged commit 9fdf3b2 into main Dec 5, 2025
1 check passed
@mahaloz mahaloz deleted the feat/summarize_improvements branch December 5, 2025 03:33
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.

Increase quality of LLM summarization

2 participants