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

Added Config UI #262

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added Config UI #262

wants to merge 4 commits into from

Conversation

Ethan-Otto
Copy link

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

image

@assafelovic
Copy link
Owner

@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!

@Ethan-Otto
Copy link
Author

Ethan-Otto commented Nov 24, 2023

@assafelovic so hide everything else? It's a partial screen shot, it includes all variables in the config

@assafelovic
Copy link
Owner

assafelovic commented Nov 25, 2023

@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

Copy link

@notvicent3 notvicent3 left a 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.

@Ethan-Otto
Copy link
Author

@assafelovic Added a filter to control what's displayed/updated
image

@Ethan-Otto
Copy link
Author

@assafelovic anything else needed?

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.

None yet

3 participants