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

feat: Have AI generate a check file for user #91

Merged
merged 82 commits into from
Nov 28, 2023
Merged

Conversation

Poiuy7312
Copy link
Collaborator

@Poiuy7312 Poiuy7312 commented Oct 17, 2023

Closes issue #33

This adds a command called:

chasten create-checks

Which will then display a textual app the code for which is in the configApp.py file:

image_2023-11-21_150448788

It will also prompt for an API key if one has not been given already just through a terminal input which it will then store an encrypted version into a text file using fernet.

The code than takes the inputs given by the user and stores it in a csv file split that information up when then the app is closed and use the write_checks function in configApp.py to format it in way to have consistently formatted inputs to send to chatGPT in an attempt to make the program more consistent.

It then displays the checks and sends the information to chatGPT and gets the results which is handled in the createchecks.py file and then displays the results in the terminal and stores it in a YAML file called checks.yml or whatever the user specifys.

image_2023-11-21_150812765

@Poiuy7312 Poiuy7312 added enhancement New feature or request in-progress Work is actively happening on this issue labels Oct 17, 2023
@Poiuy7312 Poiuy7312 changed the title WIP: feat: Have ai generate a texts file for user WIP: feat: Have AI generate a check file for user Oct 17, 2023
@gkapfham
Copy link
Collaborator

Hello @Poiuy7312 and @AlishC1 and @AlishChhetri, is this feature finished? Is there a way that you can include screenshots or other details so that people can see how it works? Finally, Alish, we talked about the strategies that you need to take to test this feature effectively. Since you cannot include an API key inside of the test suite, what approach are you going to take?

As soon as you are finished with this feature, please make sure to remove the draft label and then indicate that it is ready for review.

@AlishChhetri AlishChhetri marked this pull request as ready for review November 21, 2023 21:09
@Poiuy7312 Poiuy7312 added ready-for-review This pull request is ready for review and removed in-progress Work is actively happening on this issue labels Nov 27, 2023
@laurennevill
Copy link
Collaborator

A small thing I have noticed is that you have an analysis.md file included in this PR. If i am not mistaken, this is automatically generated by another feature of chasten and is unique to you. This file would need to be removed from the PR before it can be merged into the main branch.

@AlishChhetri
Copy link
Collaborator

A small thing I have noticed is that you have an analysis.md file included in this PR. If i am not mistaken, this is automatically generated by another feature of chasten and is unique to you. This file would need to be removed from the PR before it can be merged into the main branch.

I have removed the analysis.md file that was included in this PR

Copy link
Collaborator

@tuduun tuduun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feature is implemented and works as intended. Moreover, the code has been tested. However, @AlishChhetri told me there is an issue that chasten developers are not in control of. If chatGPT server overloads and crashes, then this feature will not work. Nonetheless, it still does what it is intended to do.

Copy link
Collaborator

@laurennevill laurennevill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this has been in-progress for a long time and it looks great! Good work, everyone

@boulais01 boulais01 self-requested a review November 28, 2023 20:24
@laurennevill laurennevill merged commit 6e8dd03 into master Nov 28, 2023
3 checks passed
@laurennevill laurennevill deleted the createchecks branch November 28, 2023 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready-for-review This pull request is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Integration with a Large Language Model (LLM) for Help with Creating XPath Expressions from XML Snippets
7 participants