-
Notifications
You must be signed in to change notification settings - Fork 10
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
Update TOPP parameter section #86
base: main
Are you sure you want to change the base?
Conversation
- sections derived from full parameter name (e.g. section:subsection) - remove option to display full parameter name (now done with sections) - add option to display TOPP tool name
PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here. PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here. PR Code Suggestions ✨Explore these optional code suggestions:
|
User description
This update addresses issues #85 and #84.
TOPP tool input sections are structured based on the full parameter name (e.g. all parameter belonging to section:subsection are grouped together. If a section description is available it is displayed as help text. Removed the option to display full parameter names on the widget itself (redundant with this solution).
Option to display TOPP tool name.
Added a streamlit error box if a TOPP tool was not found instead.
PR Type
enhancement, bug fix
Description
input_TOPP
method to display tool names and organize parameters into subsections, improving the user interface.Changes walkthrough 📝
StreamlitUI.py
Enhance TOPP tool parameter handling and error messaging
src/workflow/StreamlitUI.py
messages.
displaying tool names.
and adding comments.