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

Chat gpt attempt questions #873

Merged
merged 5 commits into from
Aug 27, 2023
Merged

Chat gpt attempt questions #873

merged 5 commits into from
Aug 27, 2023

Conversation

tmcdonnell2
Copy link
Contributor

New and improved version of ChatGPT posting to Discord. Attempts questions posted in the question forum (attempts due to timeouts and "I'm sorry, as a LLM I can't..." if AI unable/opinion based question). Also included:

  • Break apart responses into max 2000 character limit to work with Discord's API.
  • Included test cases for testing breaking up messages.

Resolves #814,
Resolves #840

* Add ChatGPT command/service code and utils.
…ssed in to helper.

* Add parsing of question to helper.
…o the command if the service fails or an explanation that the next response is AI generated)
* Add logger.debug() to catch responses before being parsed in case of failing to parse correctly to generate new tests [AIResponseParser.java].
@tmcdonnell2 tmcdonnell2 requested review from a team as code owners August 19, 2023 23:41
@tmcdonnell2
Copy link
Contributor Author

I will close the other pull request when this one is merged to ensure all work is actually included. Sorry it is annoying but this was the best way to fix my issue with trying to pull from develop and make this branch as up to date as possible.

…stead of full file names).

* Remove ChatGPTServiceTest.java as it was an older way to generate long responses from ChatGPT for testing.
* Add another test file and include in parameters for AIResponseParserTest.java
@sonarcloud
Copy link

sonarcloud bot commented Aug 20, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@SquidXTV
Copy link
Member

ok seems like this can be shipped
dont forget to delete both branches and close/finish both pr 👍

@SquidXTV
Copy link
Member

What are you waiting for?

@tmcdonnell2 tmcdonnell2 merged commit ae006d2 into develop Aug 27, 2023
9 checks passed
@tmcdonnell2 tmcdonnell2 deleted the ChatGPT-attempt-questions2 branch August 27, 2023 00:15
@Zabuzard Zabuzard mentioned this pull request Sep 18, 2023
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.

chatgpt command limit response length ChatGPT - Answer attempt
3 participants