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

POSSIBLE REQUIREMENTS FIX + GUI UPDATES #31

Closed
wants to merge 3 commits into from
Closed

POSSIBLE REQUIREMENTS FIX + GUI UPDATES #31

wants to merge 3 commits into from

Conversation

alxspiker
Copy link
Contributor

  • Should fix requirement conflict with dependency protobuf.
  • Disables ability to query the qa_system once already running.
  • Marks qa_system for reinitialization if changing any parameters in the GUI. This way your next question is answered with your new settings without having to reload.
  • Changing parameters in the GUI while already asking a question will reinitialize the qa_system with the same question and load the answer with the new parameters.

+ Should fix requirement conflict with dependency `protobuf`.
+ Disables ability to query the qa_system once already running.
+ Marks qa_system for reinitialization if changing any parameters in the GUI. This way your next question is answered with your new settings without having to reload.
+ Changing parameters in the GUI while already asking a question will reinitialize the qa_system with the same question and load the answer with the new parameters.
@alxspiker
Copy link
Contributor Author

GUI needs more work, this is just a quick PR to fix the requirements issue.

@alxspiker alxspiker marked this pull request as draft May 14, 2023 15:09
@alxspiker
Copy link
Contributor Author

Not ready to commit. Issues with streamlit, might think about abandoning it. I like it but the workflow is too weird IMO

@alxspiker alxspiker closed this May 14, 2023
@su77ungr
Copy link
Owner

streamlit working for you again?

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

2 participants