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

feat(openai): add OpenAI model constants #314

Merged
merged 2 commits into from
Feb 24, 2025

Conversation

affaan-m
Copy link
Contributor

New model provider: OpenAI Additional Models

Description

Adding constants for OpenAI's latest model offerings to the OpenAI provider in RIG.

Provider website: https://openai.com
API documentation: https://platform.openai.com/docs/models

Added constants:

  • O3_MINI: OpenAI's most efficient high-context model
  • O3_MINI_2025_01_31: Date-specific version of O3-mini
  • O1: OpenAI's base model
  • O1_2024_12_17: Date-specific version of O1

These additions make it easier for RIG users to access OpenAI's newer, more cost-efficient models directly through the framework.

Testing

  • Verified constant naming matches OpenAI's official model names
  • Confirmed models are accessible through OpenAI API
  • Tested model constants in a real application (Cainam Core)
  • Documentation comments follow RIG's format

Checklist:

  • My code follows the style guidelines of this project
  • I have commented my code with proper doc comments for each constant
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • The constants match the official model names from OpenAI
  • I've reviewed OpenAI's API documentation to ensure accuracy

Notes

This is a minimal change that only adds constants for new OpenAI models. The O3-MINI model in particular offers improved efficiency for high-context operations, making it a valuable addition to the available model options in RIG.

Contact

x.com/affaanmustafa

…atest models to the OpenAI provider: - O3_MINI: OpenAI's most efficient high-context model - O3_MINI_2025_01_31: Date-specific version of O3-mini - O1: OpenAI's flagship model - O1_2024_12_17: Date-specific version of O1 These models are compatible with OpenAI's API and can be used as drop-in replacements in applications using the RIG framework.
@joshua-mo-143
Copy link
Contributor

Hi - before I can merge this you need to make sure that the tests pass. It looks like there are some issues with incorrect syntax (you forgot a semicolon on some of the const variables) as well as formatting, which should be done locally.

@affaan-m
Copy link
Contributor Author

Hi - before I can merge this you need to make sure that the tests pass. It looks like there are some issues with incorrect syntax (you forgot a semicolon on some of the const variables) as well as formatting, which should be done locally.

Thanks for that, do we look well now? I copied the formatting from the old models exactly how the naming and comment conventions go and added the missing semicolons to the 4 new models.

@0xMochan 0xMochan merged commit 3c60638 into 0xPlaygrounds:main Feb 24, 2025
5 checks passed
@github-actions github-actions bot mentioned this pull request Feb 22, 2025
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.

3 participants