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

Verbose for Gaussian Process regression #25740

Open
ithakis opened this issue Mar 1, 2023 · 3 comments · May be fixed by #26012
Open

Verbose for Gaussian Process regression #25740

ithakis opened this issue Mar 1, 2023 · 3 comments · May be fixed by #26012
Assignees

Comments

@ithakis
Copy link

ithakis commented Mar 1, 2023

Describe the workflow you want to enable

I would like to monitor the progress of my fit function for a Gaussian Process.

Describe your proposed solution

if verbose >= 1 print the parameters "theta" and the log marginal likelihood. Maybe offer the option to return the log likelihood with a dict for the tried thetas.

  • In general any verbose would be very welcome. To get a rough estimate of the progress.

Please and thank you.

Describe alternatives you've considered, if relevant

No response

Additional context

No response

@ithakis ithakis added Needs Triage Issue requires triage New Feature labels Mar 1, 2023
@ogrisel
Copy link
Member

ogrisel commented Mar 9, 2023

if verbose >= 1 print the parameters "theta" and the log marginal likelihood.

Feel free to open a PR for this. This should be pretty uncontroversial to review and merge.

Maybe offer the option to return the log likelihood with a dict for the tried thetas.

Assuming those are not numerous values, we could probably store 2 arrays as fitted attributes on the estimator for post-fit inspection.

self.explored_thetas_ = ...
self.explored_thetas_loglikelihoods_ = ...

@ogrisel ogrisel removed the Needs Triage Issue requires triage label Mar 9, 2023
@ROMEEZHOU
Copy link
Contributor

Hi! Is someone working on this issue? If not, I would be glad to help.

@ROMEEZHOU
Copy link
Contributor

/take

@ROMEEZHOU ROMEEZHOU linked a pull request Mar 29, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants