Skip to content

Conversation

@Alex-Tideman
Copy link

What was changed

Screenshot 2025-06-25 at 3 51 09 PM

Add a UI for the different samples

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@Alex-Tideman Alex-Tideman requested a review from a team as a code owner June 25, 2025 21:09
Copy link
Member

@cretz cretz Jun 26, 2025

Choose a reason for hiding this comment

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

Maybe not a big deal here, but we need to be cautious when overly increasing the size/scope of the sample to become an application. Samples are meant as code samples and are meant to be small, grok-able as a whole, and easily adapted to your use case because the code is small. We don't have a UI for running any other set of samples, not sure this one deserves to be unique here.

If there is a need for this type of larger sample/application, we have traditionally done that in separate repo owned by SA or devrel or education. Also, these samples are meant to mimic the https://github.com/openai/openai-agents-python/tree/main/examples a bit which doesn't have a UI IIUC. I think it'd actually be reasonable if we kept their directory structure too, but that's a different topic.

I wonder if we should leave these samples as-is and if we want to have an example application with a UI and such significantly different from OpenAI repo and larger in scope, we can have a new repo.

Copy link
Author

Choose a reason for hiding this comment

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

Totally fine with that approach, can take this and create a new repo with a more proper UI setup and make it a legit project.

Copy link
Contributor

Choose a reason for hiding this comment

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

The examples in this directory should track the OpenAI Agents SDK—we want to make them match as closely as possible. We could put more examples in another directory of this repo. Demos probably should go in another repo.

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.

4 participants