vsergeyev / loudml-grafana-app

Visualization panel and datasource for Grafana to connect with Loud ML AI solution for ICT and IoT automation
https://vsergeyev.github.io/loudml-grafana-app/
MIT License
48 stars 20 forks source link

[Feature Request] use "|" or any other "rare" character as concatenation character in the model model name instead of "_" #21

Open toni-moreno opened 4 years ago

toni-moreno commented 4 years ago

I've trying to design a common dashboard for all possible models where to see evolution for real/predicted data.

Something like that.

image

In order to build the queries by only selecting the model name could be interesting if could build variables like in the picture, from the incoming model name.

right now the name is something like "$db$measurement$oper$field$tagkey_$tagvaluetime$groupby", when trying to build variables in grafana with regexp is difficult because the "_" character is commonly used in db names , field names , tag names.

image

Would be nice if we can select the concatenation character , if the model name was any of these:

instead of

swarm_cpu_mean_usage_active_host_worker1_time_5m

build the regex would be easier than before , and could cover most of the normal cases.

If done I will be able to share these generic dashboards

Thank you very much for your work @vsergeyev

toni-moreno commented 4 years ago

Hi @vsergeyev I'm trying to automate with ansible with the before proposed format and is not working because of model name restrictions as in https://github.com/regel/loudml/issues/62

It seems that model name should match this regex ^[a-zA-Z0-9-_@]+$ the only non usual character perhaps could be '@'

vsergeyev commented 4 years ago

Hi @toni-moreno

Agree this should be a way to specify model name in panel itself. This may give a possibility to specify variables in model name field, so it will be converted to required when "Create baseline" button is clicked.

V.