Skip to content

Conversation

@aseembits93
Copy link
Contributor

No description provided.

Comment on lines +29 to +30
def _get_wrapped_exception(exc: BaseException) -> Optional[BaseException]: # noqa: FA100
"""Get the wrapped exception if this is a simple wrapper.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe I should just rename it to Claude.md since we've all started using it, will do that

@codeflash-ai
Copy link
Contributor

codeflash-ai bot commented Jan 20, 2026

⚡️ Codeflash found optimizations for this PR

📄 92% (0.92x) speedup for _extract_exception_from_message in codeflash/verification/comparator.py

⏱️ Runtime : 1.99 milliseconds 1.04 milliseconds (best of 208 runs)

A dependent PR with the suggested changes has been created. Please review:

If you approve, it will be merged into this PR (branch comparator-wrapped-exceptions).

Static Badge

@codeflash-ai
Copy link
Contributor

codeflash-ai bot commented Jan 20, 2026

⚡️ Codeflash found optimizations for this PR

📄 98% (0.98x) speedup for _get_wrapped_exception in codeflash/verification/comparator.py

⏱️ Runtime : 991 microseconds 501 microseconds (best of 250 runs)

A dependent PR with the suggested changes has been created. Please review:

If you approve, it will be merged into this PR (branch comparator-wrapped-exceptions).

Static Badge

@KRRT7 KRRT7 merged commit 3418b61 into main Jan 22, 2026
23 of 28 checks passed
@KRRT7 KRRT7 deleted the comparator-wrapped-exceptions branch January 22, 2026 11:00
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.

3 participants