-
Notifications
You must be signed in to change notification settings - Fork 30
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
Possible error in stats #552
Comments
It seems like you're right. Did you discover this by some weird or
unexpected behavior? If so can you provide an example where this object
doesn't behave as expected?
…On Sun, Jun 9, 2024, 5:15 AM Nicholas Landry ***@***.***> wrote:
In this line in the constructor of IDStat, there is this line:
self.kwargs = {} if args is None else kwargs
Shouldn't it be kwargs?
—
Reply to this email directly, view it on GitHub
<#552>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAILYACRI5YOUOBKFWCZDMDZGPCDRAVCNFSM6AAAAABJAQEEOSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2DCOJZGQ3DSMI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Merged
Hi @leotrs --- I tried my best to create an issue, but I can't create an example which will raise undesired behavior. Nonetheless, I think my PR makes the code more explainable. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In this line in the constructor of IDStat, there is this line:
Shouldn't it be
kwargs
?The text was updated successfully, but these errors were encountered: