cog-imperial / OMLT

Represent trained machine learning models as Pyomo optimization formulations
Other
284 stars 59 forks source link

Fix missing dependencies in notebooks #88

Closed adi4656 closed 2 years ago

adi4656 commented 2 years ago

@tsaycal @jalving Here are the remaining changes I think need to be made so the notebooks run correctly; I've put them in a pull request because it was getting a bit confusing on the Slack. With these changes added (and the solvers installed as per the "NOTE"s in the notebooks), everything runs correctly. NB These are the changes needed with OMLT installed from source instead of PyPI - there are still a couple of missing dependencies when building from source. @tsaycal I saw you had made a commit removing onnx from neural_network_formulations. I tried running the notebook in an environment with OMLT built from source but without onnx directly installed. However, there was still a ModuleNotFoundError. I have added it back as a dependency but can remove it from this pull request if that is incorrect.

Legal Acknowledgement\ By contributing to this software project, I agree my contributions are submitted under the BSD license. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.