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

Problems with multiple metrics in popup content when using InfluxDB #90

Closed
amotl opened this issue Jan 29, 2021 · 6 comments
Closed

Problems with multiple metrics in popup content when using InfluxDB #90

amotl opened this issue Jan 29, 2021 · 6 comments
Labels
popup All things about popups

Comments

@amotl
Copy link

amotl commented Jan 29, 2021

Dear community,

we are happy to see this fork of the original Worldmap Panel is used by some people for their projects out there [1].

While we integrated #77 by @matschaffer into the most recent release 0.15.0 (thanks again!), @oliveres told us at #77 (comment) that they are not amused:

Hi, how to hide unwanted fields in popup? I have multiple tags in InfluxDB and I don't want to display this. Thx

Snímek obrazovky 2021-01-24 v 22 51 34

@jm66 and @DmitrySidorow also chimed in. While @matschaffer already provided another patch to revert this change on behalf of #89, I would like to see this going forward rather than backwards. Attaching to that, I would like to follow @jm66's suggestion at #77 (comment) to provide an option in the UI to disable the additional fields.

In the meanwhile, I would like to humbly recommend to just downgrade to version 0.14.0 [2,3], like @jm66 did.

With kind regards,
Andreas.

[1] In this spirit, we will also be happy to see some of the outcomes where this panel has been used if that would be possible and your project policies will permit you to share those. @matschaffer and @robouden are apparently using it for Safecast. While we haven't been able to create a documentation page including some sort of gallery/exhibition yet, that would definitively be a nice thing to have.
[2] https://github.com/panodata/grafana-map-panel/releases/tag/0.14.0
[3] https://github.com/panodata/grafana-map-panel/releases/download/0.14.0/grafana-map-panel-0.14.0.zip

@DmitrySidorow
Copy link

DmitrySidorow commented Jan 29, 2021

I use this to track the quality of data transmission from passenger carriages
Screenshot_1
Screenshot_2

In the future, carriages with errors and accidents will also be shown on the another map.

Customizable popups would be a killer feature 😃.
Screenshot_3

@jm66
Copy link

jm66 commented Jan 29, 2021

Hi @amotl, first of all, thanks for all that you do to maintain this plugin. The feature integrated in #77 by @matschaffer looks like a great addition and would be even better if we can decide from the UI wether to show "label", "all" or select which fields should be shown.

We are using this great plugin to show the number of active wireless users connected to the campus wireless network. https://wifimap.eis.utoronto.ca/

Again, thanks a lot.
JM

@matschaffer
Copy link
Collaborator

matschaffer commented Jan 30, 2021

My thinking on my first pass was to use the query metrics to show/hide entries in the popup.

This way we can use the query panel as UI to select which fields. If you don't want it, leave it out of the query, if you want it, include it.

LiZZ1wZZOm

I'm thinking what we can do is work out what other "special" fields (when, timestamp, etc) show up in other data sources and filter those as well in this code: https://github.com/panodata/grafana-map-panel/pull/77/files#diff-eac8713e7ec990341c87c6c0d1787ac626e2055b7e6f9d62c524771aa35c0a74R442

Alternatively, we could do a comma-separated list of fields to include in the worldmap panel settings off to the right, but I liked the simplicity of using the query panel to control field retrieval & visibility.

@oliveres
Copy link

Hi @matschaffer, this is not a universal solution because in InfluxDB v2 you cannot select some and unselect other fields. Flux query language working with filtering only, there is no SELECT xy FROM yz WHERE...(at least what I know about it). Comma-separated field names in UI settings will be OK.

@amotl
Copy link
Author

amotl commented Aug 6, 2021

Hi everyone,

I just created #118, where I tried to summarize the topic about customizing the popup contents as good as possible, taking requests from different users into account. Feel free to add anything what comes to your mind about this.

With kind regards,
Andreas.

@amotl amotl added the popup All things about popups label Aug 6, 2021
@matschaffer
Copy link
Collaborator

With #89 merged I think we can close this in favor of #118

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
popup All things about popups
Projects
None yet
Development

No branches or pull requests

5 participants