Skip to content
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

Problem with installing requirements #7

Closed
Thomaswbt opened this issue Oct 19, 2022 · 3 comments
Closed

Problem with installing requirements #7

Thomaswbt opened this issue Oct 19, 2022 · 3 comments

Comments

@Thomaswbt
Copy link

Hi!
I have an issue installing the required version of gpytorch and botorch. I cannot fetch the specific commit hash. Could you please check if this is a common problem or this is a problem of my network? If I skip this step and install gpytorch and botorch directly form pip, I found that I would encounter problems similar to #3 and #4 and I cannot trivially fix them. So I wonder the versions of gpytorch and botorch may matter. Thanks in advance!

屏幕快照 2022-10-19 下午10 45 37

屏幕快照 2022-10-19 下午10 46 04

@samuelstanton
Copy link
Owner

samuelstanton commented Oct 21, 2022

Hi, thanks for creating the issue and sorry to hear you're having trouble.

Unfortunately this package is quite sensitive to which version of gpytorch and botorch is used, which is why I pinned the requirements to those specific commit hashes. I'm not sure why you're having trouble fetching those commit hashes... would you mind seeing if you can clone the repo and check out those commits manually? In the short term this is the fastest way to get the code running.

The long-term fix is to update the requirements to stable releases of GPyTorch/BoTorch, however you are correct that it leads to non-trivial bugs that are likely caused by GPyTorch itself and which will require opening another issue there.

@samuelstanton
Copy link
Owner

samuelstanton commented Oct 21, 2022

This may help

https://bswen.com/2021/11/how-to-solve-github-https-clone-error.html

If switching from HTTPS to HTTP solves your problem, please let me know and I'll update the requirements

@Thomaswbt
Copy link
Author

Thomaswbt commented Oct 21, 2022

Thank you! I tried to manually install the specific commit hashes and most errors vanished consequently! However the numerical stability issue mentioned in #3 still exists and it causes error in validating the correlation matrix. I think I shall move to #3 if I cannot solve it trivially. I will close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants