Open toni-moreno opened 4 years ago
Hello again @regel
This regex would be nice ^[a-zA-Z0-9-_@\/\[\]{}:;,.|=%"\$]+$
https://regex101.com/r/w633je/2
it will match with
traffic.rxt_et-1/0/0
traffic.rxt_et-[1/0/0]
traffic.rxt_et-{1/0/0}
traffic.rxt_et-00:11:11
traffic.rxt_et-00,1,1
traffic.rxt_et=adfa
traffic.rxt_et=%20%a
traffic.rxt_et="foo"
traffic.rxt_et="${value}
Hi Toni. Yes. Good idea! Yes you can open a PR on this. The unit tests can be updated too based on the examples given.
Ps: I’m on vacation in the beautiful Loire valley. Responses will be delayed.
Thank you very much for your fast answer
Ok , I will be out for 1 week also just from today, I will build a PR when I come back.
Hello @regel , sorry for this new issue.
I'm trying to create a model with this definition.
it seems like doesn't like
traffic.rxt_et-1/0/0
as measurement maybe character validation?https://github.com/regel/loudml/blob/2d05a3e292954346fa9fd35c5fec7ced90428f4a/loudml/influx.py#L315
https://github.com/regel/loudml/blob/2d05a3e292954346fa9fd35c5fec7ced90428f4a/loudml/schemas.py#L28-L32
@regel IMHO is really important add as many characters in the measurement name as posibles in the influxdb database
As explained in the following doc
"',;:-/\.[]{}
are valid characters and should be only scaped "\" if conflicting with regex to use them, also key length limit should be 64Kb lengthWould you accept a PR to fix this error?
Thank you in advance