Global elevenlabs voice config overrides Wingman elevenlabs config | Breaks app

[JaydiCodes]

If a global voice name is supplied for the elevenlabs config object, voice: Adam, and a later elevenlabs config object has a voice object specified voice: id: <id>, it breaks the app trying to merge an item with a string.

Tower.py - line 32. Not exact area but starting point

def __instantiate_wingmen(self) -> list[Wingman]:
...
            merged_config = self.__merge_configs(global_config, wingman_config) <- HERE
            class_config = merged_config.get("class")

            wingman = None

[Shackless]

Also related to

wingman-ai/wingmen/open_ai_wingman.py

Lines 354 to 358 in 0d7e80c

	voice = elevenlabs_config.get("voice")
	if not isinstance(voice, str):
	voice = Voice(voice_id=voice.get("id"))
	else:
	voice = next((v for v in voices() if v.name == voice), None)

which is not correct as it always disabled the id if a name is set as well.

[Shackless]

I fixed it. Do you want to take a look at the PR, @JaydiCodes? Thank you!

[JaydiCodes]

@Shackless I took a look. It's what we both were thinking. Sure that would work fine.