-
Notifications
You must be signed in to change notification settings - Fork 22
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
Sandevh/stu 307 generate tests with an llm #1122
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Backend
How do we handle inference? Do we pay for it? Not sure if we can package our key with the desktop app due to security reasons. We won't be able to update the key if something happens.
Or should we let the user add their own OpenAI key? Set this key the same way as the cloud one?
Any other solutions?
Frontend:
-
The Run Test Sequence button should always be visible, in my opinion. We could move it to where the Play button is on the flowchart.
-
A toast.promise would be great so the user knows the request is being processed in the background.
-
Now that the import and all are not always present, it's easy to confuse the following:
-
For hardware, it completely hallucinate function (with tm_devices as example). We need to do some tests on how we can use other models, give it more context, fine-tune, etc. :)
- I have some idea that I want to test when the export feature is done for that :)
-
Right-click to consult code in the generated panel doesn't work.
-
I haven't went through all the frontend code wet :) But the UI is clean !!
from openai import OpenAI | ||
from captain.utils.test_sequencer.data_stream import _with_error_report | ||
|
||
key = "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" # TODO have this from .env, Joey said to hard code for now until his PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
key should be as an ENV variable, never hard coded since it can easily be forgotten
...erer/routes/test_sequencer_panel/components/panels/TestGeneratorPanel/TestGeneratorPanel.tsx
Show resolved
Hide resolved
...erer/routes/test_sequencer_panel/components/panels/TestGeneratorPanel/TestGeneratorPanel.tsx
Show resolved
Hide resolved
221bca5
to
da8afe0
Compare
da8afe0
to
95af224
Compare
8cdfe19
to
c6346e6
Compare
GENERATOR PANEL: