Skip to content

Conversation

@njhale
Copy link
Member

@njhale njhale commented Oct 14, 2024

  • Add gpt-4o-mini to smoke test github workflow
  • Remove outdated gpt-4o version and gpt-4-turbo entirely from smoke test
  • Focus comparison on matching event types to reduce false negatives
  • Drop "ignore callProgress" rule (we're eliding them from the event
    stream before sending them to the judge now)

Reminder: The workflow from main is what runs for PRs, so you won't see smoke tests run for gpt-4o-mini on this PR.

- Focus comparison on matching event types to reduce false negatives
- Drop "ignore callProgress" rule (we're eliding them from the event
  stream before sending them to the judge now)

Signed-off-by: Nick Hale <4175918+njhale@users.noreply.github.com>
@njhale njhale changed the title test/smoke 4o mini Add gpt-4o-mini to smoke test github workflow and make smoke test judge more reliable Oct 14, 2024
- Add gpt-4o-mini to smoke test github workflow
- Add gpt-4o-mini smoke test golden files
- Remove outdated gpt-4o model and 4-turbo smoke test config and golden
files
- Add golden files for gpt-4o-2024-08-06
- Regenerate golden files for existing models to drop callProgress
  events (we weren't comparing these anyway)

Signed-off-by: Nick Hale <4175918+njhale@users.noreply.github.com>
@njhale njhale force-pushed the test/smoke-4o-mini branch from 2380ce1 to ef6c6fc Compare October 14, 2024 16:33
@njhale njhale merged commit eefe829 into gptscript-ai:main Oct 14, 2024
10 checks passed
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