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

ChatGPT rewrite #192

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

goodlifefound
Copy link

Create Evaluate ChatGPT rewrite

What does this PR do?

Refactor the Code Structure:
I’ll organize the code into functions to improve readability and maintainability.
Create separate functions for loading the screenshot, evaluating it, and running test cases.
Error Handling:
I’ll add proper error handling to handle cases where the screenshot cannot be opened or evaluated.
Comments and Documentation:
I’ll add comments to explain each section of the code.
Provide clear documentation for each function.
Simplify the Evaluation Prompt:
The evaluation prompt can be simplified to make it more concise and easier to understand.

Fixes # (issue)

Requirement/Documentation

  • If there is a requirement document, please, share it here.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected. Make sure before submmiting this PR you run tests with evaluate.py

Create Evaluate ChatGPT rewrite
@goodlifefound
Copy link
Author

I am new to this, just trying to contribite! Please advise!

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