Skip to content

fix: update labels for OpenAI Embedding Model dropdown#18

Open
ecairol wants to merge 1 commit into
mainfrom
fix/open-ai-embedding-labels
Open

fix: update labels for OpenAI Embedding Model dropdown#18
ecairol wants to merge 1 commit into
mainfrom
fix/open-ai-embedding-labels

Conversation

@ecairol
Copy link
Copy Markdown

@ecairol ecairol commented May 30, 2026

Summary

text-embedding-ada-002 is the previous-generation embedding model ("Ada"), while text-embedding-3-<size> is the current generation and what's being used in the input "value" attr

This PR updates the label to avoid any confusion and to represent an accurate description of the chosen OpenAI model.

The attr value remains the same.

Screenshot 2026-05-30 at 2 28 42 PM

 text-embedding-ada-002 is the previous-generation embedding model (the "Ada" one), text-embedding-3-[s] is the current generation and what's being used in the input "value" attr
@ecairol ecairol changed the title fix: update model labels for consistency in class-wpvdb-models.php fix: update labels for OpenAI Embedding Model dropdown May 30, 2026
@ecairol ecairol requested a review from Copilot May 30, 2026 20:55
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates OpenAI embedding model dropdown labels to accurately reflect the actual model IDs used by WPVDB, reducing confusion with the legacy Ada model naming.

Changes:

  • Renames the text-embedding-3-small label from “Ada 3 Small” to the actual model name.
  • Renames the text-embedding-3-large label from “Ada 3 Large” to the actual model name.
  • Leaves model values and behavior unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants