[DOC] Fix typos across the docs#904
Merged
Merged
Conversation
Ahmed-Zahran02
requested review from
felipeangelimvieira and
fkiraly
as code owners
There was a problem hiding this comment.
Pull request overview
This PR addresses issue #903 by correcting typos in docstrings/comments across the core skpro package, reducing potential confusion for users and contributors reading inline documentation.
Changes:
- Fixes spelling/typos in test utility documentation and multiple estimator/distribution docstrings.
- Corrects repeated misspellings like “probabilites”, “distirbutions”, and similar documentation-only errors.
- Minor wording cleanup in distribution/regression module comments.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
skpro/tests/test_switch.py |
Fixes typo in Condition 5 description for test-switch logic. |
skpro/regression/multiquantile.py |
Corrects comment typo (“empirical”). |
skpro/regression/linear/_glm.py |
Fixes docstring typos; one remaining grammar issue noted in review comment. |
skpro/regression/binned/_sklearn_bin_regressor.py |
Fixes multiple “probabilit*” typos in class docstring. |
skpro/regression/bayesian/_linear_conjugate.py |
Fixes typo in initializer docstring (“coefficient”). |
skpro/distributions/zeroinflated.py |
Fixes comment typo (“continuous”). |
skpro/distributions/trafo/_transformed.py |
Fixes typo in comment (“discrete”). |
skpro/distributions/meanscale.py |
Fixes typo in class docstring (“distributions”). |
skpro/datatypes/_table/_examples.py |
Fixes typo in module docstring (“the”). |
skpro/datatypes/_proba/_examples.py |
Fixes typo in module docstring (“the”). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Comment on lines
+211
to
212
| If the link function is also explicitly mentioned then include then | ||
| that must be passed to the family/distribution object. |
fkiraly
changed the title
fkiraly
changed the title
fkiraly
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reference Issues/PRs
fixes #903
What does this implement/fix? Explain your changes.
Fix typos in
skpro/skpro.