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

Enhance Program Performance and Conciseness #1

Closed
wants to merge 16 commits into from
Closed

Enhance Program Performance and Conciseness #1

wants to merge 16 commits into from

Conversation

bassel27
Copy link

@bassel27 bassel27 commented Apr 20, 2023

This pull request includes several updates to make the program faster and make the code more concise. The following changes were made:

  • Removed unnecessary time.sleep() statements to reduce wait times and make the program faster.
  • Implemented explicit waits to further improve program speed.
  • Added an exception to check if request limits have been exceeded on Chat GPT.
  • Removed all redundant methods and attributes after implementing these changes.
  • Added deleteCurrentConversation method to delete conversations after having them.

@ugorsahin
Copy link
Owner

ugorsahin commented May 1, 2023

Hello Basel, first of all thank you for your efforts to make it better, unfortunately you've changed everything in a direction which I would prefer to avoid. However, I liked some of the parts like deleting all conversations and checking if limit exceeded. I am happy to accept them if you can send pull requests in a modular basis.

@bassel27
Copy link
Author

bassel27 commented May 5, 2023

Alright, no problem. I'll do it

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.

None yet

2 participants