Repository navigation
Sync with template (4aa6051) - #278
Merged
Merged
Conversation
Between a tool's exit and wait() the process is a zombie, and the Run page's status check (pid_files.reap) removes the pid file of a zombie. run_command() then raised FileNotFoundError on its own unlink and the workflow failed although the tool had succeeded. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGekq5UCb6U4f2bXigdC
On macOS and Linux the desktop app reads Thermo .raw files with OpenMS's built-in reader (share/OpenMS/openms_thermo_bridge) on a bundled .NET runtime instead of ThermoRawFileParser. The About dialog now shows the RawFileReader notice for that reader too, lists it under Programs, and lists the .NET runtime (DOTNET_ROOT) with its license and notices under Libraries. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGenTCt1fet8pkp5v8T2
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGenTCt1fet8pkp5v8T2
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGenTCt1fet8pkp5v8T2
…mo-reader About dialog: list OpenMS's built-in Thermo reader and the .NET runtime
Local runs: a tool's pid file removed by the status check no longer fails the workflow
The full images include RawFileReader in /thirdparty/ThermoRawFileParser. Put Thermo's notice and terms next to its THERMO_LICENSE as NOTICE.txt, so the About dialog lists it, and print it when the container starts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGenTCt1fet8pkp5v8T2
…er-thermo Ship Thermo's RawFileReader notice in the full Docker images
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGenTCt1fet8pkp5v8T2
Contributor
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Tom · project thread
Merges streamlit-template main (4aa6051) into this app. Among the changes:
NOTICE.txtnext toTHERMO_LICENSE, listed in the About dialog and printed at container start (Ship Thermo's RawFileReader notice in the full Docker images streamlit-template#570).Conflicts were resolved by keeping files this app removed or owns. Merge with a merge commit, not squash.
Generated by Claude Code