-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add SBGN-ML importer #439
Draft
cthoyt
wants to merge
10
commits into
master
Choose a base branch
from
add-sbgnml-importer
base: master
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Add SBGN-ML importer #439
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## master #439 +/- ##
==========================================
- Coverage 79.10% 76.59% -2.52%
==========================================
Files 171 176 +5
Lines 9039 8962 -77
Branches 1307 1333 +26
==========================================
- Hits 7150 6864 -286
- Misses 1608 1832 +224
+ Partials 281 266 -15
Continue to review full report at Codecov.
|
Will time out due to pyobo though. need mocks that will be implemented with biopragmatics/pyobo#62
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #423
This PR introduces the
pybel.io.sbgnml
I/O module, which first consumes SBGN-ML and converts it into a facile JSON-like data structure (which is generally reusable) then reasons over it and converts to BEL.The code also has some grounding mechanisms built in to deal with the often missing identifiers from the COVID-Pathways content, but I might remove this to make the tool more generally reusable. I'm not really sure who else generates SBGN-ML if not from CellDesigner (and more specifically, coming out of MINERVA)
DEMO NOTEBOOK
Example SBGN-ML Content
For an example, see the content posted by @cannin https://cannin.github.io/covid19-sbgn. The SBGN-ML file (disregard the SIF and SIF SVG, since those are derived resources).
He notes on his site that these SBGN-ML files were converted from CellDesigner using https://github.com/sbgn/cd2sbgnml. It's clear that there's some lost information due to
cd2sbgnml
(or there are strange pockets of incredibly low quality curation), so it will be a future step to write a CellDesigner importer later (see #440).For example, this script can be run (after installing this branch in development mode) with the following code in the python REPL: