The traits are at their best when they are flavorful and descriptive, and at their least interesting when they are almost entirely redundant with a stat. The biggest offender is "Charismatic", and I'm not even talking about the bonus it gives. There might as well be a "Beautiful" trait. If a girl's charismatic, give her a high charisma stat, you know? To a lesser degree, other traits such as "Charming", "Tough", "Fragile", "Fleet of Foot" and "Strong Magic" seem to fall in the same category. I mean, is there really a difference between being "Fragile" and having extremely low constitution? And if there is, why? If a girl with 100 Constitution acquires "Fragile", does it really mean she's become fragile, or is she just not as tough as she used to be?
It seems to me that these traits would be better used as indicators than modifiers. If a girl has over 70 Charisma (or any arbitrary number that seems appropriate), she would get the Charismatic trait, only the trait itself wouldn't do anything, it'd just be there to tell you the girl has high charisma without straight up shoving a number in your face.
On the other hand, traits like "Elegant", "Good kisser", "Long legs", "Small scars", "Clumsy", "Cool person" and "Nerd" are wonderful, because they're important and flavorful details, or stuff that doesn't need to (or can't) be defined in terms of numbers. And the fact that they modify stats is sensible. I mean, if a girl has Horrific Scars, and I remove these scars during the game, it's entirely logical that she gets those 5 points of Beauty back, and on top of that there's a strong role-play element involved. So these traits are basically all advantage and, in my opinion, fine as they are. (Though their specific effects could probably do with some slight rebalancing in certain cases.)