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 tests #513

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Fix tests #513

wants to merge 2 commits into from

Conversation

henninggaertner
Copy link
Collaborator

@henninggaertner henninggaertner commented Jun 21, 2024

Description

Fixes a bunch of tests and improves some form descriptions in the frontend

Testing

Run tests

PR checklist

Development

  • If necessary, I have updated the documentation (README, docstrings, etc.)
  • If necessary, I have created / updated tests.

Mergeability

  • main-branch has been merged into local branch to resolve conflicts
  • The tests and linter have passed AFTER local merge
  • The code has been formatted with black

Code review

  • I have self-reviewed my code.
  • At least one other developer reviewed and approved the changes

@henninggaertner henninggaertner changed the base branch from main to dev June 21, 2024 12:09
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  protzilla
  run.py
  runner.py 127-128
  steps.py 83, 101
  protzilla/data_analysis
  ptm_analysis.py
  protzilla/data_preprocessing
  filter_proteins.py
  outlier_detection.py
  peptide_filter.py
  transformation.py
  protzilla/importing
  metadata_import.py 150
  peptide_import.py
  protzilla/methods
  data_analysis.py 631-641
  data_preprocessing.py
  importing.py 142
  ui/runs/forms
  data_analysis.py 926-934, 962
Project Total  

This report was generated by python-coverage-comment-action

@henninggaertner henninggaertner removed the request for review from lucatreide November 19, 2024 14:14
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.

1 participant