trafficstars

Reference Issues/PRs

Fixes #210

What does this implement/fix? Explain your changes

support for loading both local and downloading pdb files, and save at self.raw_dir

What testing did you do to verify the changes in this PR?

Currently, none of this, just test on dataset loader doc notebook.

More changes and complements will be done at tomorrow morning.........

Pull Request Checklist

[ ] Added a note about the modification or contribution to the ./CHANGELOG.md file (if applicable)
[ ] Added appropriate unit test functions in the ./graphein/tests/* directories (if applicable)
[x] Modify documentation in the corresponding Jupyter Notebook under ./notebooks/ (if applicable)
[ ] Ran python -m py.test tests/ and make sure that all unit tests pass (for small modifications, it might be sufficient to only run the specific test file, e.g., python -m py.test tests/protein/test_graphs.py)
[ ] Checked for style issues by running black . and isort .

Sep 19 '22 14:09 1511878618

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

Sep 19 '22 14:09 review-notebook-app[bot]

Awesome work, thanks @1511878618 ! If you merge into #213 the CI pipeline should be much faster & I can push a release to PyPI tonight :)

Sep 19 '22 14:09 a-r-j

Awesome work, thanks @1511878618 ! If you merge into #213 the CI pipeline should be much faster & I can push a release to PyPI tonight :)

I'm not sure about how to change into #213, and i see master branch is associated with #213. Or, maybe you can help me?

Also, I'm not sure whether are there some potential bugs in it~

Sep 19 '22 15:09 1511878618

Codecov Report

Base: 40.27% // Head: 48.00% // Increases project coverage by +7.73% :tada:

Coverage data is based on head (07cd92a) compared to base (8123f42). Patch coverage: 51.68% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #214      +/-   ##
==========================================
+ Coverage   40.27%   48.00%   +7.73%     
==========================================
  Files          48       86      +38     
  Lines        2811     5458    +2647     
==========================================
+ Hits         1132     2620    +1488     
- Misses       1679     2838    +1159

Impacted Files	Coverage Δ
graphein/grn/parse_trrust.py	`37.77% <ø> (ø)`
graphein/ml/diffusion.py	`0.00% <0.00%> (ø)`
graphein/ml/transform.py	`0.00% <0.00%> (ø)`
graphein/ppi/edges.py	`100.00% <ø> (ø)`
graphein/ppi/graph_metadata.py	`0.00% <ø> (ø)`
graphein/ppi/graphs.py	`54.34% <ø> (ø)`
graphein/ppi/parse_biogrid.py	`75.00% <ø> (ø)`
graphein/ppi/visualisation.py	`0.00% <0.00%> (ø)`
graphein/protein/analysis.py	`0.00% <0.00%> (ø)`
graphein/protein/edges/intramolecular.py	`22.68% <0.00%> (ø)`
... and 79 more