Skip to content

fix: correct cis-vaccenic acid cross-references to the cis compound - #1034

Merged
edkerk merged 1 commit into
developfrom
fix/cis-vaccenic-xref
Jul 11, 2026
Merged

fix: correct cis-vaccenic acid cross-references to the cis compound#1034
edkerk merged 1 commit into
developfrom
fix/cis-vaccenic-xref

Conversation

@edkerk

@edkerk edkerk commented Jul 11, 2026

Copy link
Copy Markdown
Member

Main improvements in this PR:

As reported in #766, the structure shown for cis-vaccenic acid (MAM01585) is the trans compound (vaccenic acid). The cause is in the model: three of MAM01585's cross-references point to the trans compound, and Metabolic Atlas renders the structure from a SMILES mapped through these IDs.

  • Correct MAM01585 cross-references (all four compartments) from vaccenic acid (trans) to cis-vaccenic acid:
    • metKEGGID: C08367 -> C21944
    • metMetaNetXID: MNXM92713 -> MNXM1372019
    • metHMDBID: HMDB0003231 -> HMDB0240219

The ChEBI (CHEBI:50464), PubChem (5282761) and LipidMaps (LMFA01030076) references were already correct. The second metabolite in the report, cis-vaccenoyl-CoA (MAM01586), has correct cross-references (a monounsaturated cis-vaccenoyl-CoA); its incorrect display is a Metabolic Atlas rendering issue and is not addressed here.

I hereby confirm that I have:

  • Tested my code on my own computer for running the model
  • Selected develop as a target branch
  • Any removed reactions and metabolites have been moved to the corresponding deprecated identifier lists

… compound (#766)

MAM01585 is cis-vaccenic acid ((11Z)-octadecenoic acid), but three of its
cross-references pointed to the trans compound, vaccenic acid ((11E)):
- metKEGGID: C08367 (vaccenic acid) -> C21944 (cis-vaccenic acid)
- metMetaNetXID: MNXM92713 (trans-vaccenic acid) -> MNXM1372019 (cis)
- metHMDBID: HMDB0003231 (Vaccenic acid) -> HMDB0240219 (cis-Vaccenic acid)

The ChEBI (CHEBI:50464), PubChem (5282761) and LipidMaps (LMFA01030076)
references were already correct. Metabolic Atlas renders the structure
from a SMILES mapped via these cross-references, so the trans MetaNetX ID
is why the trans structure was displayed for cis-vaccenic acid (#766).
Applied to all four compartments (c, e, l, r).
@edkerk

edkerk commented Jul 11, 2026

Copy link
Copy Markdown
Member Author

No structural model change

@edkerk edkerk added this to the 2.0.1 milestone Jul 11, 2026
@edkerk
edkerk merged commit c852d15 into develop Jul 11, 2026
1 of 6 checks passed
@edkerk
edkerk deleted the fix/cis-vaccenic-xref branch July 11, 2026 11:11
@edkerk edkerk mentioned this pull request Jul 15, 2026
3 tasks
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.

1 participant