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

Fix/filename not in msms #137

Merged
merged 4 commits into from
Oct 19, 2023
Merged

Fix/filename not in msms #137

merged 4 commits into from
Oct 19, 2023

Conversation

picciama
Copy link
Contributor

@picciama picciama commented Oct 6, 2023

PR Checklist

  • This comment contains a description of changes (with reason)
  • Referenced issue is linked
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated

Description of changes

  • handle keyerror: Warn the user in case raw file names have no corresponding peptides in search results instead
  • log file created for each run based on job type and date and time

Technical details

Additional context

@github-actions github-actions bot added the bug Something isn't working label Oct 6, 2023
@picciama picciama merged commit 31c458d into development Oct 19, 2023
34 of 37 checks passed
@tobiasko
Copy link

👍

@picciama picciama deleted the fix/filename_not_in_msms branch November 1, 2023 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants