464 unify uniprot protein graph data across runs - Githubissues

cschlaffner / PROTzilla2

12 stars 0 forks source link

464 unify uniprot protein graph data across runs #467

Closed antonneubauer closed 2 months ago

antonneubauer commented 2 months ago

Description

refere to #464

Changes

protein_graphs.py: changes to functionality -> these are the relevant changes. please specifically pay attention to directory creation and the control flow i.e. what is ensured/checked where

Testing

create a protein graph in run A for e.g. Protein P10636 (example data set BA46 works fine)
create the same protein graph for P10636 in a new, separate run B
check the logs, file explorer, whether the files for P10636 have been downloaded again and whether the unmodified graph generated in run A has been used in run B to match peptides

PR checklist

Development

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

Mergeability

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

Code review

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

antonneubauer commented 2 months ago

some problems with a path ^^ trying to work that out rn

github-actions[bot] commented 2 months ago

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
protzilla/constants
paths.py
protzilla/data_analysis
protein_graphs.py					99, 222, 233, 335, 423, 438-459
Project Total

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