Skip to content

Fix #1142: SyntaxWarning: Invalid escape sequences in calibration.py do - #1240

Open
danielalanbates wants to merge 1 commit into
sunlabuiuc:masterfrom
danielalanbates:fix/issue-1142
Open

danielalanbates wants to merge 1 commit into
sunlabuiuc:masterfrom
danielalanbates:fix/issue-1142

Conversation

@danielalanbates

Copy link
Copy Markdown

Fixes #1142

Summary

This PR addresses: SyntaxWarning: Invalid escape sequences in calibration.py docstrings

Changes

docs/api/metrics.rst                           | 2 +-
 examples/cardiology_detection_isAR_SparcNet.py | 2 ++
 pyhealth/metrics/calibration.py                | 4 ++--
 3 files changed, 5 insertions(+), 3 deletions(-)

Testing

Please review the changes carefully. The fix was verified against the existing test suite.


This PR was created with the assistance of Claude Agy:Gemini 3.8.flash by Anthropic | effort: high (extended thinking). Happy to make any adjustments!

By submitting this pull request, I confirm that my contribution is made under the terms of the project's license (contributor license agreement).

@MateehUllah MateehUllah left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The raw-string changes look aligned with #1142. One issue with the current PR contribution rules: both ece_confidence_multiclass and ece_confidence_binary are top-level public functions whose docstrings are modified here, and CONTRIBUTING.md requires every new or modified public function to include a >>> usage example. Neither modified docstring currently has one. Could you add a minimal executable example to each and run python tools/check_pr_rules.py --base 1308ac6 --head HEAD?

This branch has not been deployed

No deployments
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.

SyntaxWarning: Invalid escape sequences in calibration.py docstrings

2 participants