search

cog-imperial / entmoot

Multiobjective black-box optimization using gradient-boosted trees
https://entmoot.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
56 stars 12 forks source link

Make gurobipy import optional #3

Closed marcosfelt closed 3 years ago

marcosfelt commented 3 years ago

Currently, ENTMOOT requires GUROBIpy to be installed, though it should work without it. It would be great if gurobipy could be imported flexibly so that the package can be used without it.

ThebTron commented 3 years ago

Thanks @marcosfelt for the suggestion! In the current implementation ENTMOOT will only try to import GUROBIpy if "global" is selected for "acq_optimizer". If you pick "sampling" as the optimization strategy instead there shouldn't be any GUROBIpy imports. Can you give more details on when exactly you're experiencing this issue? I might have left an unnecessary import of GUROBIpy in there.

marcosfelt commented 3 years ago

My bad, that was an issue with Summit.