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.

Sign up for GitHub

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

Jump to bottom

update TOPP framework #49

Merged
merged 1 commit into from
May 6, 2024
Merged

update TOPP framework #49

merged 1 commit into from
May 6, 2024

Conversation

axelwalter
Copy link
Collaborator

StreamlitUI:

  • show parameter summary (non-defaults) in expander in run section
  • check if workflow run completes, show error if not
  • date and time of latest complete run
  • input_topp:
    • include_params list for custom param selection
    • option to display full parameter name (e.g. algorithm : common : param_name)
    • custom_defaults: overwrite default parameters

ParameterManager:

  • TOPP boolean params as strings

Logger:

  • write three log levels to select any time (minimal, commands and execution times, full)

@axelwalter
update TOPP framework
5c75afb 
StreamlitUI:
- show parameter summary (non-defaults) in expander in run section
- check if workflow run completes, show error if not
- date and time of latest complete run
Logger:
- write three log levels to select any time (minimal, commands and execution times, full)
CommandExecutor:
- run_topp:
  - include_params list for custom param selection
  - option to display full parameter name (e.g. algorithm : common : param_name)
  - custom_defaults: overwrite default parameters
ParameterManager:
- TOPP boolean params as strings
@axelwalter axelwalter merged commit 79d7aaa into OpenMS:main May 6, 2024
5 of 6 checks passed
JohannesvKL pushed a commit to JohannesvKL/PTMScanner that referenced this pull request Dec 19, 2024
@axelwalter
Merge pull request OpenMS#49 from axelwalter/update-topp-framework
fb447be 
update TOPP framework
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@axelwalter