search

cog-imperial / OMLT

Represent trained machine learning models as Pyomo optimization formulations
Other
257 stars 56 forks source link

Dependencies - ONNX and TensorFlow #52

Closed rmisener closed 2 years ago

rmisener commented 2 years ago

The ONNX reader requires ONNX and the Keras reader needs tensorflow.keras. We need better and more descriptive error handling to manage this.

carldlaird commented 2 years ago

Pyomo has a reasonable system for conditional importing. @jalving and I could look at this. I prefer not to have a growing list of dependencies.

rmisener commented 2 years ago

Agreed and thanks.

rmisener commented 2 years ago

Thanks to @chrismarquez for addressing this issue. His PR #83 fixes this issue.