crewAI icon indicating copy to clipboard operation
crewAI copied to clipboard

Case insensitive tool and agent names

Open jkellerer opened this issue 1 year ago • 1 comments

Thanks for the great toolset.

Had tried it with a locally running Mistral 7b and found that this LLM, while being very usable on commodity HW, doesn't always get case and leading/trailing spaces right. With this change the trip planner example started to work.

This PR adds strip().casefold() for agent names and casefold() for tool names (as strip() was existing already)

jkellerer avatar Jan 13 '24 17:01 jkellerer

Do you mind fixing conflicts, so we can merge it?

gvieira avatar Mar 28 '24 18:03 gvieira

I tried resolving the conflicts but decided to pull these change into the new version myself as it felt simpler, it will be live int he next version so clsogin this one

joaomdmoura avatar Mar 31 '24 18:03 joaomdmoura