Bug: can't differentiate between allosteric activators or inhibitors via allostery ID
Closed this issue · 4 comments
NicholasCowie commented
Need to include another field called regulation type in the input files
teddygroves commented
I think this isn't necessary as I think we would never have two allosteries with the same enzyme and mic but different modification types, and adding the extra field to the id would require adding lots of blanks to every priors csv file.
teddygroves commented
I actually implemented the suggested change before coming to this conclusion - branch here. @NicholasCowie let me know if you still think it would be better!
NicholasCowie commented
I like that implementation. Unfortunately, there are instances I’ve observed where at different concentrations metabolites can be both inhibitors and activators.
teddygroves commented
OK, looks like I didn't waste that work then!