Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
toni-moreno opened this issue Jul 31, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@toni-moreno
Copy link

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_$tagvalue_time_$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:

  • swarm|cpu|mean|usage_active|host|worker1|time|5m
  • swarm:cpu:mean:usage_active:host:worker1:time:5m
  • swarm cpu mean usage_active host worker1 time 5m

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 toni-moreno changed the title [Feature Request] use "|" or any other "rare" character as model name concat nexus instead of "_" [Feature Request] use "|" or any other "rare" character as concatenation character in the model model name instead of "_" Jul 31, 2020
@toni-moreno
Copy link
Author

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 regel/loudml#62

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

@vsergeyev vsergeyev added the enhancement New feature or request label Aug 1, 2020
@vsergeyev
Copy link
Owner

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants