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

Replace GPy package with less messier alternative #131

Open
mcw92 opened this issue May 16, 2024 · 0 comments
Open

Replace GPy package with less messier alternative #131

mcw92 opened this issue May 16, 2024 · 0 comments
Assignees
Labels
maintenance Improve code readability and robustness.

Comments

@mcw92
Copy link
Member

mcw92 commented May 16, 2024

Currently, the Gaussian processes used in Propulate's surrogate models rely on the GPy package which repeatedly caused problems and errors, in particular in the context with matplotlib.
According to SheffieldML/GPy#1034, matplotlib is not listed in GPy's install_requires but necessary for some imports, causing errors.
That is why we manually install matplotlib in Propulate (and this is also the only reason why matplotlib appears in Propulate's dependencies). I find this very ugly.
Also, GPy is apparently incompatible with matplotlib v3.9.
We should try to get rid of GPy and find a better maintained alternative.
Also see SheffieldML/GPy#1029 😁.

@mcw92 mcw92 added the maintenance Improve code readability and robustness. label May 16, 2024
@mcw92 mcw92 self-assigned this May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Improve code readability and robustness.
Projects
None yet
Development

No branches or pull requests

1 participant