Skip to content

Reference CalibrationDisplay from calibration_curve's docstring in a "See also section" #31311

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

Closed
metlouf opened this issue May 5, 2025 · 3 comments · Fixed by #31312
Closed

Comments

@metlouf
Copy link
Contributor

metlouf commented May 5, 2025

Describe the issue linked to the documentation

Enrich documentation like proposed in #31302 for calibration_curve's

Suggest a potential alternative/fix

No response

@Muwinuddin
Copy link

hi @metlouf

I'd like to work on this documentation improvement. Adding a reference to CalibrationDisplay in the "See also" section of the calibration_curve docstring makes perfect sense, as it helps users connect the computation function with the visualization utility.

I plan to:

Add CalibrationDisplay to the "See also" section of calibration_curve.

Ensure the formatting is consistent with other references.

Run the doc build to verify everything renders correctly.

Let me know if there's anything else you'd like me to include. I'd be happy to open a PR for this.

@metlouf
Copy link
Contributor Author

metlouf commented May 6, 2025

I already opened a PR and solve the problem

(In the context of the probabl hackathon 6 may)

@ogrisel ogrisel removed the Needs Triage Issue requires triage label May 6, 2025
@Muwinuddin
Copy link

@metlouf
Thanks for the update! Glad to see the issue resolved. Appreciate your quick work.

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