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

Adding code run failure handling process & Update to align with the newest OpenAI package formats #34

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

cylqqqcyl
Copy link

@cylqqqcyl cylqqqcyl commented Dec 10, 2023

Description

Adding code run failure handling process:

This is done by query back to the API with the exception message and reward function code.
There is a new parameter added in the config.yaml called max_retries that limits the attempts to fix the buggy reward code and prevents infinite loop.
The current code can now at least generate 1 legit reward functions and, in most cases, generate all the num_samples to be executable.

Update to align with the newest OpenAI package formats:

According to openai/openai-python#742
OpenAI has released a new major version of its SDK, and they recommend upgrading promptly.
I reimplemented the queries and handling of responses to align with the new requirements.

Type of change

  • Bug fix
  • Feature enhancement/new feature
  • Documentation change
  • Style/UI change
  • Performance optimization
  • Refactoring/code optimization
  • Other changes

Related Issues

#4

#4 (comment)

Specific changes

  • Changed the eureka/eureka.py file's query method

  • Modulized the reward code processing by encapsulating it as a function for code reuse

  • eureka/cfg/config.yaml Changed by adding parameter max_retries and updating the model to gpt-4-1103-preview

Screenshots

Example of failed run and automatic code fix:

samples

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.

1 participant