-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added Config UI #262
base: master
Are you sure you want to change the base?
Added Config UI #262
Conversation
@Ethan-Otto this is great! Can you please refine and include on the following params: Report format, total words, search engine, llm provider and max iterations? The idea is to expose only the report related params. Thanks! |
@assafelovic so hide everything else? It's a partial screen shot, it includes all variables in the config |
@Ethan-Otto Not to hide but remove, I think some of these config params should be hidden and not modified since it may confuse or break the UX/UI. Lmk if you can, if not I can do it later in the week |
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.
Didn't check it running atm, but as I can see in the code seems pretty nice.
@assafelovic Added a filter to control what's displayed/updated |
@assafelovic anything else needed? |
Added UI for controlling the config. I made the config.json in the root folder the default config, and the UI can update it directly. Allows much faster testing of different parameters especially when using docker.
Also added descriptions of each variable