Eduard Kerkhoven

Other instances of generic reactions: ### Generic lipid rxnID | reaction name | equation :--- |:--- |:--- r_4225 | Broad-range acid phosphatase DET1 (EC 3.1.3.-) (Decreased ergosterol transport protein 1)...

Good point @hongzhonglu, but I'd argue that protein-modifying enzymes are still outside the scope of a purely metabolic model, which is what yeast-GEM represents. So should we then remove the...

@wtscott31 That's your paper, is this included in #300 ?

But by definition, there should really be only one EC number per reaction, although perhaps with some wildcards if the exact substrate or cofactor has not been given a specific...

Indeed, there are many that are already fine. I'm not 100% certain how automated curation would work, because many gene/protein based approaches (e.g. BridgeDB, Uniprot) are not ideal as it...

@ae-tafur, could you please make sure that [this box](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork) is ticked? Thanks!

Line 118-132 should be removed, no longer required as invalid IDs would already have thrown an error. https://github.com/SysBioChalmers/RAVEN/blob/a459d6c10714b673878ad62790d6e19c68fefb7c/io/exportModel.m#L118-L132

> > no longer required as invalid IDs would already have thrown an error. > > @edkerk this got me thinking - are invalid reported throwing errors for _all_ import...

Should not be merged before #364 is merged and pushed from `devel` to `main`. Release 2.5.4 has then minor changes, while release 2.6.0 (this PR) will have the significant change...

I'm still hesitant with this, as it can break backwards compatibility with people's scripts. It changes the identifier representation (e.g. `M_glc6p` instead of `glc6p`) when changing RAVEN version. Might it...