Entity names should not be allowed to include () bracket openings and closings

[stschiff]

In Xerxes, we have a YAML and a simple text interface to read in statistics. The simple text interface looks like

F3(Pop1, Pop2, Pop3)

because of this structure, the names of the groups cannot contain bracket openings and closings, and that is something that has to part of the parser. Right now, poseidon-hs does not include that constraints, which unfortunately leads to Xerxes having a redundant parser (https://github.com/poseidon-framework/poseidon-analysis-hs/blob/fedf433d10b5fb9f7bd9847928193c8fbc7f4318/src/Poseidon/Analysis/FStatsConfig.hs#L162)

It would be nice if we could simply add the two symbols ( and ) to the forbidden symbols in entity names here (

poseidon-hs/src/Poseidon/EntitiesList.hs

Line 126 in e7c1ce6

parseName = P.many1 (P.satisfy (\c -> not (isSpace c || c `elem` ":,<>*")))

) so we can avoid this redundancy.

[nevrome]

Should this be added to the Poseidon schema? 🤔

[stschiff]

Hmm... good point. I don't really know, to be honest. Technically, it is unnecessary to forbid brackets altogether, since I could in principle parse things like "MyFavoriteGroup(take 1)" by matching opening and closing brackets. The only thing that wouldn't work is "LetsTryAndHackThis)".

Finally, I could also change the interface to accept group names in quotation marks, optionally. Perhaps that would even be better...

I would not act on the schema for now, I will think more about it.