Skip to content

🏷️ fix: Ensure modelLabel Field Usage for ModelSpecs/GPTPlugins#4224

Merged
danny-avila merged 1 commit into
mainfrom
fix/modelLabel
Sep 24, 2024
Merged

🏷️ fix: Ensure modelLabel Field Usage for ModelSpecs/GPTPlugins#4224
danny-avila merged 1 commit into
mainfrom
fix/modelLabel

Conversation

@danny-avila

@danny-avila danny-avila commented Sep 24, 2024

Copy link
Copy Markdown
Owner

Summary

Closes #4223

  • Modified OpenAIClient.js to use modelLabel as a fallback for chatGptLabel, ensuring a consistent label is always available.
  • Updated buildOptions.js in the gptPlugins service to include modelLabel in the endpoint options, allowing for more accurate model identification.
  • Refactored the getResponseSender function in parsers.ts to prioritize modelLabel when determining the response sender, improving the accuracy of sender identification.
  • Enhanced the compactPluginsSchema in schemas.ts to include the modelLabel field and added logic to remove it when null, maintaining data consistency.

Testing

To test these changes:

  1. Verify that the OpenAI client correctly uses the modelLabel as a fallback for chatGptLabel.
  2. Test the gptPlugins endpoint to ensure it properly includes the modelLabel in its options.
  3. Check various scenarios in the frontend to confirm that the correct model label is displayed for different endpoints and models.
  4. Ensure that the schema changes do not introduce any unexpected behavior when handling plugin-related data.

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • I have tested my changes to ensure they work as expected

@danny-avila danny-avila merged commit 6f498ee into main Sep 24, 2024
@danny-avila danny-avila deleted the fix/modelLabel branch September 24, 2024 11:27
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.

[Bug]: ModelSpec Name not displayed correctly

1 participant