Skip to content
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

fix(digest): digest title prediction #73

Merged
merged 2 commits into from
Nov 15, 2023
Merged

Conversation

quentingrchr
Copy link
Contributor

@quentingrchr quentingrchr commented Nov 14, 2023

Fix: Digest's title AI prediction

Description

This pull request addresses some unintended behavior in the digest prediction feature. The following changes have been made:

  • Reordered the digest entries from the earliest to the latest before feeding them into the prompt.
  • Explicitly set the temperature (to 0.5).
  • Added a fallback when the AI can't find a logical sequence.
  • Deleted the previously set prediction when deleting a published digest.

Update 🆕

  • Retrieve only the latest 5 digest titles. A bug was identified when users attempted to change their titles; the prompt failed to consider the new entries as they were a minority in the prompt. Ex: ["Hi 1, Hi 2,..., Hi 6, New Digest Title #001, New Digest Title #002]. This update not only resolves the bug but also contributes to cost savings by preventing unnecessary retrieval of all digest titles, and unnecessary long prompts 💸
  • Extract functionality into a separate function to enable parallel execution.

Link to Issue

Fix #70

Additional Comments

I'm using idk as a "safe word" for an empty response because it seems like GPT didn't want to send me an empty response back.

@quentingrchr quentingrchr linked an issue Nov 14, 2023 that may be closed by this pull request
Copy link

vercel bot commented Nov 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
digestclub ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 14, 2023 1:39pm

@tlenclos tlenclos merged commit e8ab47c into main Nov 15, 2023
3 checks passed
@tlenclos tlenclos deleted the fix/digest-title-prediction branch November 15, 2023 09:22
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.

Digest title prediction invalid
3 participants