[SOLVED] isinstance(var,str) always returning False

[Tess]

I have a function I'm trying to use to make **kwargs in some of our functions not case sensitive. This is what I have so far.

Code: Select all

def dict_lower(dictionary):
        """Takes a dict with string keys and makes them all lower case"""
        for key,value in dictionary.items():
            if isinstance(key,str):
                if key.lower() not in dictionary:
                    dictionary[key.lower()] = value
                    dictionary.pop(key)
        return dictionary

Right now this does nothing, however if I comment out "if isinstance(key,str)" it works perfectly.

Code: Select all

#This works perfectly
def dict_lower(dictionary):
        """Takes a dict with string keys and makes them all lower case"""
        for key,value in dictionary.items():
            #if isinstance(key,str):
            if key.lower() not in dictionary:
                dictionary[key.lower()] = value
                dictionary.pop(key)
        return dictionary

I can't figure out why isinstance doesn't want to recognize key as a string, when it seems to be behaving as one everywhere else.

[Ocelot]

How do you call that function?
And try to print actual object class before checking it. Then look in console to see what do you have. If you ever see something strange, usually you want to add a ton of print statements displaying everything you might need about processes that happen inside.

[Tess]

I added "print key" and "print type(key)" to the beginning of the for loop. For each one it printed the string fine and then <type 'str'> after it. So key seems to be a string, isinstance just doesn't want to accept it

I also added print "Made it here" right inside isinstance and it never printed...

Code: Select all

def dict_lower(dictionary):
        """Takes a dict with string keys and makes them all lower case"""
        for key,value in dictionary.items():
            print key #This prints the string fine
            print type(key) #This prints <type 'str'>
            if isinstance(key,str):
            	print "Made it here" #This never prints
                if key.lower() not in dictionary:
                    dictionary[key.lower()] = value
                    dictionary.pop(key)
        return dictionary

Really baffling...

[Ocelot]

Are you on RenPy 7.5 by any chance?

[Tess]

Yup!

[Ocelot]

Try to check for (str, pystr, future.utils.text_type) - that's it, tuple of three types. IIRC RenPy replaces standard string as part of Python3 compatibility fixes.

If it works, figure out concrete type you want. My bet is on pystr.

[Tess]

You were exactly right, pystr did the trick. Thanks!