Skip to content
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

add kwargs argument as a workaround to ignore boolean_number arguments #365

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adrian-forest
Copy link

@adrian-forest adrian-forest commented Mar 16, 2024

add kwargs argument as a workaround to ignore boolean_number arguments passed to the function

I encountered and reported #364 . This change seems to avoid this error on my local.

I know that this is BAD CODE! I'm a .NET developer by trade, but I could navigate python enough to identify this workaround, but I also know enough to assume it's VERY BAD and DOES NOT ACTUALLY SOLVE THE PROBLEM. However, this does avoid the error on my local, so far as I can tell, and I'm hoping maybe a PR might prompt somebody to make a proper fix?

Honestly, this is my first time making a PR to a public repo, so please be nice?

…ore boolean_number arguments passed to the function
@WAS-PlaiLabs
Copy link
Collaborator

This may not be needed now as per 7fb0766

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.

2 participants