-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New release of ax-platform is raising ModuleNotFoundError
#387
Comments
Hi, @richardliaw, thanks so much for letting us know! I believe this is due to our forgetting to bump our BoTorch version requirement. If you upgrade |
@lena-kashtelyan thanks for the fast response. I think it doesn't work actually:
|
Summary: Addresses facebook#387 Reviewed By: sdsingh Differential Revision: D24025000 fbshipit-source-id: 54ceaf70414d92be72e7b9e811d58d563245b261
That's perplexing, let me investigate. |
Summary: Pull Request resolved: facebook#389 Addresses facebook#387 Reviewed By: sdsingh Differential Revision: D24025000 fbshipit-source-id: bfca9d156002d5e58e34fb8db821c412c3da2502
Update: Found the source of the problem, currently working on getting a fix committed (should be out in a bit) after which we will release a new version. |
Summary: In #387, users are having issue importing from the botorch_modular module. Maybe it's because of the missing __init__.py. But I have no clue how this works **without** it . . . Reviewed By: lena-kashtelyan Differential Revision: D24031280 fbshipit-source-id: 3cbfebf3f756927a78979941676d8c2aa34ac2f7
Fix is out 16cb868, also released a new version (v0.1.16) which should be available from PyPl shortly |
Thanks for the fast turnaround!
…On Wed, Sep 30, 2020 at 5:22 PM Steve Mandala ***@***.***> wrote:
Closed #387 <#387>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#387 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCRZZJUU5A6AIZWHZEHQZDSIPDVFANCNFSM4R7OC4SA>
.
|
Is there a workaround for this?
The text was updated successfully, but these errors were encountered: