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: fix parameter conditions for perplexity #1394
Conversation
@gijigae is attempting to deploy a commit to the LobeHub Team on Vercel. A member of the Team first needs to authorize it. |
👍 @gijigae Thank you for raising your pull request and contributing to our Community |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1394 +/- ##
==========================================
- Coverage 86.33% 86.32% -0.01%
==========================================
Files 217 218 +1
Lines 11059 11118 +59
Branches 1177 1186 +9
==========================================
+ Hits 9548 9598 +50
- Misses 1511 1520 +9 ☔ View full report in Codecov by Sentry. |
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.
The code is a little confusing, I think need to write in a better way. let me try it
Thanks @arvinxx for the feedback! Really appreciate your help. Tried the approach in ZhipuAI, but the response received from Perplexity were somewhat repetitive.
|
@gijigae I have update the code, please check whether it works well? |
Thanks @arvinxx for the update! Tested and it's showing same words over and over. Will check what's causing the problem. |
@gijigae hi, is there any update? |
🎉 This issue has been resolved in version 0.156.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
Updated the paramter conditions for Perplexity to resolve #1368 . Perplexity only accepts either frequency_penalty or presence_penalty, and not both. If the value of presence_penalty is something other than 0, the presence_penality will be passed to a model.
📝 补充信息 | Additional Information
The details will be added to a document as well.