-
Notifications
You must be signed in to change notification settings - Fork 66
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
MAINT: Cleaner traceback on error #951
Conversation
Error is unrelated but I'll investigate to see if I can figure out where it's coming from. Probably a |
@@ -1,9 +1,10 @@ | |||
{# Modified from https://github.com/mkdocstrings/python/blob/master/src/mkdocstrings_handlers/python/templates/material/_base/attribute.html #} | |||
{# Updated 2024/05/20. See "START NEW CODE" for block that is new. #} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the real necessary change here was to git mv
the file. But while I was at it I also took the latest version from the URL above. We'll see if it renders okay. If not we can revert
Comparing:
I think things look okay, so ready for review/merge from my end @hoechenberger ! |
title=f"{kind} cleaning", | ||
caption=caption, | ||
section=title, | ||
tags=tags, | ||
replace=True, | ||
) | ||
plt.close(fig) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need this one for green CIs and this PR is pretty trivial I think, so I'll go ahead and merge. Happy to make changes if you see issues though @hoechenberger
No need for changelog update I think, just shortens the traceback and fails faster when
cross_val_score
fails for CSP just like it does for other decoding types.