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

EMBA ChatGPT integration #602

Merged
merged 234 commits into from
Jul 23, 2023
Merged

EMBA ChatGPT integration #602

merged 234 commits into from
Jul 23, 2023

Conversation

BenediktMKuehne
Copy link
Member

@BenediktMKuehne BenediktMKuehne commented Apr 27, 2023

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

  • Adds Openai checks for automated code analysis by the OpenAI chat-bot

  • Introduces a seperate container architecture for all online functionalities

  • What is the current behavior? (You can also link to an open issue here)
    No AI stuff
    Single container - host architecture

  • What is the new behavior (if this is a feature change)? If possible add a screenshot.
    Code snippets are analysed by the ChatGPT model
    image

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    Changes in architecture results in a new installation

  • Additional Info
    Please refer to the wiki page

@BenediktMKuehne
Copy link
Member Author

BenediktMKuehne commented Apr 27, 2023

  1. Would like to run some tests with different script languages etc.
  2. Module need integration with previous ones to get directories to test. @m-1-k-3 any ideas which modules should provide them?
  3. Module needs to run on the host instead of in the container...

@BenediktMKuehne BenediktMKuehne added enhancement New feature or request help wanted Extra attention is needed in progress Someone is working on this labels Apr 27, 2023
@m-1-k-3
Copy link
Member

m-1-k-3 commented Apr 27, 2023

I think we could use the results of module S20, S21, S22 and S23 for further GPT analysis.

As the GPT connector needs to run on the host (outside the docker) we can't make it as a usual module. We already have a helper for internet connections: https://github.com/e-m-b-a/emba/blob/master/helpers/helpers_emba_internet_access.sh
The helper function is started from emba main here:

emba/emba

Line 460 in 435d1b1

if [[ ${IN_DOCKER} -eq 0 ]]; then

Hope this helps.

Copy link
Member

@m-1-k-3 m-1-k-3 left a comment

Choose a reason for hiding this comment

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

I think we are done. What a grep PR! Thanks @BenediktMKuehne ... amazing work!

@m-1-k-3 m-1-k-3 merged commit c15e810 into e-m-b-a:master Jul 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker docker related things enhancement New feature or request in progress Someone is working on this reporting reporting topics (e.g., web reporter)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants