Skip to content

fix: correct typo prescision to precision#60

Open
Jah-yee wants to merge 1 commit into
benhamner:masterfrom
Jah-yee:master
Open

fix: correct typo prescision to precision#60
Jah-yee wants to merge 1 commit into
benhamner:masterfrom
Jah-yee:master

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Mar 2, 2026

Copy link
Copy Markdown

Summary

Fixes Issue #53: Fix simple typo prescision -> precision

Changes

  • Python/ml_metrics/average_precision.py: prescision -> precision
  • Python/ml_metrics/custom/kdd_average_precision.py: prescision -> precision

Testing

This PR has been tested locally.

Good day! Thank you for maintaining this project. I hope this fix helps. If there are any issues, please let me know.

Warmly,
Jah-yee

Issue benhamner#53: Fix simple typo in documentation
- average_precision.py: prescision -> precision
- kdd_average_precision.py: prescision -> precision

This PR has been tested locally.
@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏

@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👍 Looks good to merge — clean fix, thanks for contributing!

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Thank you for maintaining this project! Please let me know if any changes are needed — happy to iterate. Looking forward to your review!

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

LGTM! typo fix: prescision→precision looks correct. 👍

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Good precision typo fix! Just a reminder for maintainer review. Thanks! 🙏

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Looks good to merge! [merge promotion]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

This PR looks good and is ready to merge! [+1]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

LGTM! 👍 This PR looks good to merge. Keep up the great work! 🚀

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for this PR! Just a friendly nudge — I hope this can be reviewed and merged soon. Happy to make any changes if needed 🙂

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Hi @ALL, gentle reminder that this PR is waiting for review. Happy to make any changes — just let me know! 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for this contribution! Bumping for visibility.

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Would it be possible to merge this PR? Happy to make any adjustments if needed. Thank you for your time!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Bump — still waiting for review 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Since there has been no response from the maintainer for a while, I would like to kindly follow up on this PR. I believe the fix is valuable and would love to contribute. Please let me know if any changes are needed, or if it can be merged. Thank you!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Bump — ready to merge 🎯

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Bumping for merge consideration 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

+1 from a human — looks good to merge!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

+1

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for the fix! Gentle ping to see if this is ready for review. 👍

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

1 similar comment
@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

:merge:

1 similar comment
@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

:merge:

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

+:shipit: Please consider merging this PR. Thank you!

1 similar comment
@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

+:shipit: Please consider merging this PR. Thank you!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

➿ bump — keeping PR active in queue (R337)

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

🙌 bump

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Bump: mergeable ✅

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

👍 promote

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Thank you for this contribution! 🎉 This PR looks great and I'm happy to see this fix merged. Keep up the excellent work!

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

Successfully merging this pull request may close these issues.

1 participant