-
Notifications
You must be signed in to change notification settings - Fork 18
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
Call for feedback: New lipid naming convention #354
Comments
Sounds like a good plan to me. |
It seems reasonable. A question remains: Will old and new IDs coexist in the future? Would BiGG list the old IDs then as legacy identifiers and store them in a separate table? |
@draeger In this case, we will not replace any old IDs. Just add new ones. |
I don't like that this are not valid SBML identifiers, also it should be made clear that these are all triglycerols, i.e. using a glycerol backbone with possible three connections. I would highly recommend to prefix these to make the ids more clear, valid (SBML) and usable:
|
I think added the triglycerol here causes confusion with triacylglycerols. The name already contains g3pc, which is the ID for glycero-3-phosphocholine, the 1ac and 2ac state that it has an additional acyl group at positions 1 and 2. |
It will make it much easier to work with all triacylglycerols, if these have a common prefix because you can just filter the subset based on the prefix and listing the remaining 3 chains, i.e., than I even understand the rules for creating all possibilities:
|
By the way also super easy to parse and no dependency if the |
I see your points, but not all of them are tri-acyl-glycerols. Based on lipid biochemistry the glycerol-backbone is fixed to be sn-glycero-3-phosphate (coming from the synthesis). |
Only listing the chains without a prefix will create problems if there is only one modification, which for instance would than be |
Then I would go for version with |
Made up my mind. We should go for the prefix. Works also well with some IDs that are already in BiGG. |
Here is a table with examples from the WormJam model.
With multiple phosphorylated PI headgroups (PIP, PIP2 and PIP3) the back part of the ID gets a bit complicated, but I guess it is still fine. Alternative is to separate this with a |
This looks great. Some comments below, not sure we have to solve all of this.
The following often occurs: |
This is now for the moment for only the generic versions. I already though about ways how to encode specific acyl, alkyl or alkenyl chains. The position and stereochemistry of the double bond should be encoded. |
I will think about the cardiolipins and sphingolipids, might a bit tricky. |
@michaelwitting Yes, please send the preprint. I will give you feedback on it (konigmatt[AT]googlemail.com). |
Hi all. I would like to revive the discussion here. I was thinking about how to encode the side chains and the sphingoid bases etc. |
There is a nice proposal from @michaelwitting at WormJam for a systematic naming convention for lipids. It's described here:
JakeHattwell/wormjam#11
This convention would generate nice-looking BiGG IDs. A couple examples:
If we adopt this, we will probably do it through the standard BiGG process: we will not remove old IDs (so
pchol
will stay), and new IDs will come in as we add models to BiGG. However, we can provide an extra level of checking to help users adopt these new IDs.Tagging some BiGG watchers who might have feedback on this: @nel3 @neemajamshidi @matthiaskoenig @draeger @rmtfleming @phantomas1234 @willigott @cdanielmachado @smoretti @djinnome @cnorsig @jmcconn @jtyurkovich
The text was updated successfully, but these errors were encountered: