CovertLab / wcEcoli

Whole Cell Model of E. coli
Other
18 stars 4 forks source link

Better LP/QP solver import failure messages #306

Open jmason42 opened 6 years ago

jmason42 commented 6 years ago

In addition to #305 - I noticed that we don't really get an informative message if we fail to find any solvers at all when importing wholecell.utils.modular_fba. It would be nice if the total failure came with a message about what was missing, so it was more obvious what needed to be installed. Also, if our truly preferred solver isn't available, I think that should at least raise a warning. Otherwise there will likely be significant system-dependent behaviors.

tahorst commented 6 years ago

Feel free to add more detailed messages. I think the failure points you in the right direction to figure out what needs to be fixed.