sasaki77 / archiverappliance-datasource

EPICS Archiver Appliance plugin for Grafana dashboard
https://sasaki77.github.io/archiverappliance-datasource
MIT License
14 stars 7 forks source link

Resolves multi-variables bug #41

Closed n-wbrown closed 3 years ago

n-wbrown commented 3 years ago

Allows PVs of the form prefix:(var:1|var:2|...|):intermediate:(a|b|...):suffix to work properly with the backend and alerting.

Closes #40

klauer commented 3 years ago

What would happen with nested (invalid, I suppose) syntax such as prefix:(var:1|(var:2|var:3)|var:4):suffix?

n-wbrown commented 3 years ago

@klauer Currently that's not supported. I would be interested in adding that though. What are your thoughts @sasaki77 ?

n-wbrown commented 3 years ago

@sasaki77, if you feel ready to merge this, could you also create a new release after merging?

sasaki77 commented 3 years ago

@n-wbrown Should I create a new release before merging #42? If so, I'll release 1.3.1.

n-wbrown commented 3 years ago

That would be great. #42 needs some work before it will be ready.

sasaki77 commented 3 years ago

@klauer, @n-wbrown, That sounds interesting. Actually, that would be helpful if AA server supported full regular expression. In any case, it's worth to add this feature.