Skip to content

Conversation

@kamurani
Copy link
Contributor

@kamurani kamurani commented Jul 2, 2022

Fixes #15

Add capability to use rsa and dim_[1-7] of the meiler embeddings as features by which the node size is scaled in protein.visualisation.plotly_protein_structure_graph.

Testing by plotting graph:
rsa-node-size

Copy link
Owner

@a-r-j a-r-j left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but for a minor nit. Thanks for the contribution!

@a-r-j
Copy link
Owner

a-r-j commented Jul 6, 2022

LGTM, happy to merge. Could you quickly pull this up to head to incorporate the latest changes so tests can run?

@a-r-j a-r-j marked this pull request as draft July 6, 2022 15:55
@a-r-j a-r-j marked this pull request as ready for review July 6, 2022 15:55
@kamurani
Copy link
Contributor Author

kamurani commented Jul 7, 2022

@a-r-j sure thing; although I'm not exactly sure how I would go about doing that. This is the first time i've submitted a PR sorry!

Do you want me to re-commit to the already existing pull request? Or is there a simpler way?

Cheers

@codecov-commenter
Copy link

codecov-commenter commented Jul 8, 2022

Codecov Report

Merging #186 (024e7a0) into master (8123f42) will increase coverage by 7.98%.
The diff coverage is 52.10%.

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
+ Coverage   40.27%   48.25%   +7.98%     
==========================================
  Files          48       92      +44     
  Lines        2811     5326    +2515     
==========================================
+ Hits         1132     2570    +1438     
- Misses       1679     2756    +1077     
Impacted Files Coverage Δ
graphein/grn/parse_trrust.py 37.77% <ø> (ø)
graphein/ml/diffusion.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/features/sequence/utils.py 28.00% <0.00%> (+3.00%) ⬆️
graphein/protein/features/utils.py 27.77% <0.00%> (ø)
... and 78 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 011d880...024e7a0. Read the comment docs.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2022

Please retry analysis of this Pull-Request directly on SonarCloud.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@a-r-j a-r-j merged commit b7721de into a-r-j:master Jul 8, 2022
a-r-j added a commit that referenced this pull request Jul 11, 2022
* update bioservices to account for new UniProt API (#187)

* update bioservices to account for new UniProt API

* update changelog

* Update PULL_REQUEST_TEMPLATE.md

* Graph plots (#186)

* Fix param name typo in function docstring

* add scaling node size by "rsa" feature as well as degree

* add option for scaling node size by meiler embedding dimensions.  Takes negative values to be zero.

* remove walrus operator := for compatability

* Add type hints

* Update changelog

Co-authored-by: Arian Jamasb <[email protected]>

Co-authored-by: Cam <[email protected]>
a-r-j added a commit that referenced this pull request Jul 11, 2022
* refactor PDB download to support parallelisation

* add tests for PDB download

* refactor PDB download function call in getcontacts

* add salt bridges and vdw interactions

* use multiprocess downloading in torch datasets

* add additional edge funcs

* add residue_id to dfs and additional constants

* update tests to account for residue id column

* update dl func to check for obsolete PDBs

* fix type error on paths

* fix obsolete test

* add bad pdb attr

* add return to PDB download util

* Merge master -> Patch (#190)

* update bioservices to account for new UniProt API (#187)

* update bioservices to account for new UniProt API

* update changelog

* Update PULL_REQUEST_TEMPLATE.md

* Graph plots (#186)

* Fix param name typo in function docstring

* add scaling node size by "rsa" feature as well as degree

* add option for scaling node size by meiler embedding dimensions.  Takes negative values to be zero.

* remove walrus operator := for compatability

* Add type hints

* Update changelog

Co-authored-by: Arian Jamasb <[email protected]>

Co-authored-by: Cam <[email protected]>

* patch torch geometric dataset and update changelog

* black, isort

* isort

Co-authored-by: Cam <[email protected]>
a-r-j added a commit that referenced this pull request Jul 12, 2022
* refactor PDB download to support parallelisation

* add tests for PDB download

* refactor PDB download function call in getcontacts

* add salt bridges and vdw interactions

* use multiprocess downloading in torch datasets

* add additional edge funcs

* add residue_id to dfs and additional constants

* update tests to account for residue id column

* update dl func to check for obsolete PDBs

* fix type error on paths

* fix obsolete test

* add bad pdb attr

* add return to PDB download util

* Merge master -> Patch (#190)

* update bioservices to account for new UniProt API (#187)

* update bioservices to account for new UniProt API

* update changelog

* Update PULL_REQUEST_TEMPLATE.md

* Graph plots (#186)

* Fix param name typo in function docstring

* add scaling node size by "rsa" feature as well as degree

* add option for scaling node size by meiler embedding dimensions.  Takes negative values to be zero.

* remove walrus operator := for compatability

* Add type hints

* Update changelog

Co-authored-by: Arian Jamasb <[email protected]>

Co-authored-by: Cam <[email protected]>

* patch torch geometric dataset and update changelog

* black, isort

* isort

* add edge tests

* remove unused workflow action

* bump for 1.5.0

* add vdw clashes to constants

Co-authored-by: Cam <[email protected]>
a-r-j added a commit that referenced this pull request Jul 14, 2022
* update bioservices to account for new UniProt API (#187)

* update bioservices to account for new UniProt API

* update changelog

* Update PULL_REQUEST_TEMPLATE.md

* Graph plots (#186)

* Fix param name typo in function docstring

* add scaling node size by "rsa" feature as well as degree

* add option for scaling node size by meiler embedding dimensions.  Takes negative values to be zero.

* remove walrus operator := for compatability

* Add type hints

* Update changelog

Co-authored-by: Arian Jamasb <[email protected]>

* 1.5.0 patch3 (#189)

* refactor PDB download to support parallelisation

* add tests for PDB download

* refactor PDB download function call in getcontacts

* add salt bridges and vdw interactions

* use multiprocess downloading in torch datasets

* add additional edge funcs

* add residue_id to dfs and additional constants

* update tests to account for residue id column

* update dl func to check for obsolete PDBs

* fix type error on paths

* fix obsolete test

* add bad pdb attr

* add return to PDB download util

* Merge master -> Patch (#190)

* update bioservices to account for new UniProt API (#187)

* update bioservices to account for new UniProt API

* update changelog

* Update PULL_REQUEST_TEMPLATE.md

* Graph plots (#186)

* Fix param name typo in function docstring

* add scaling node size by "rsa" feature as well as degree

* add option for scaling node size by meiler embedding dimensions.  Takes negative values to be zero.

* remove walrus operator := for compatability

* Add type hints

* Update changelog

Co-authored-by: Arian Jamasb <[email protected]>

Co-authored-by: Cam <[email protected]>

* patch torch geometric dataset and update changelog

* black, isort

* isort

Co-authored-by: Cam <[email protected]>

* 1.5.0 patch3 (#191)

* refactor PDB download to support parallelisation

* add tests for PDB download

* refactor PDB download function call in getcontacts

* add salt bridges and vdw interactions

* use multiprocess downloading in torch datasets

* add additional edge funcs

* add residue_id to dfs and additional constants

* update tests to account for residue id column

* update dl func to check for obsolete PDBs

* fix type error on paths

* fix obsolete test

* add bad pdb attr

* add return to PDB download util

* Merge master -> Patch (#190)

* update bioservices to account for new UniProt API (#187)

* update bioservices to account for new UniProt API

* update changelog

* Update PULL_REQUEST_TEMPLATE.md

* Graph plots (#186)

* Fix param name typo in function docstring

* add scaling node size by "rsa" feature as well as degree

* add option for scaling node size by meiler embedding dimensions.  Takes negative values to be zero.

* remove walrus operator := for compatability

* Add type hints

* Update changelog

Co-authored-by: Arian Jamasb <[email protected]>

Co-authored-by: Cam <[email protected]>

* patch torch geometric dataset and update changelog

* black, isort

* isort

* add edge tests

* remove unused workflow action

* bump for 1.5.0

* add vdw clashes to constants

Co-authored-by: Cam <[email protected]>

Co-authored-by: Cam <[email protected]>
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.

Visualisation of Protein Graphs

3 participants