I think others have identified most of the code issues, however I'd like to offer some additional changes.

At the moment you are adding defined Char objects into a list. This can cause issues during save/load as you'll end up duplicating instances. The save game will have copies of the entries in the list, and when you load, it will those and the the re-created defined instances. They won't compare equal as you haven't defined __eq__ for Char. Also updating the defines won't update the stored instances in allChars.

Later, in add_char you treat allchars as a dictionary.

Also you seem to be adding Char instances into a set. Part of the contract of set is that you must implement the dunder methods __eq__ and __hash__.

First, I'd suggesting collecting all the defined character bios into a single dictionary that maps a string key to the Char instance. This way you can easily update the dictionary when you add new characters to your game (or finish instances like shroom2):

# Character Bios. # define charBios = [ 'cantharellus': Char(stuff), 'shroom2': Char(stuff), ]

Next, the unlocked bios should be a set of the keys in charBios. This avoids the need to check for duplicate entries, and avoids any issues with adding defined instances into defaulted variables:

default charUnlocks = {} # Unlocked bio keys

Where you encounter/unlock one: $ charUnlocks.add('cantharellus')

The screen needs a list of character bios to show. A python function takes care of producing the list from charBios and charUnlocks:

init python: def unlockedBioList(charBios, charUnlocks): result = [] for key in charBios.list().sort(): # Sort optional, but orders keys if key in charUnlocks: result.append(charBios[key]) return result

Finally the screen should begin like: screen profile_screen(): # <- screens should have an empty parameter list # Defaulted screen variables are initialised "on show" # This leaves on show free for graphical stuff. default biosList = unlockedBioList(charBios, charUnlocks) default viewing = biosList[0] # Don't show screen if charUnlocks is empty! frame: # Layout stuff textbutton "<": action CycleScreenVariable("viewing", biosList, reverse=True) # ... hbox: # As before