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

Add a custom meta field per camera to be appended to mqtt events topic #14275

Open
phillmac opened this issue Oct 11, 2024 · 7 comments
Open
Labels
enhancement New feature or request

Comments

@phillmac
Copy link

I have cameras where the name is changed often but I have a DB indexed by camera serial and I would like to have a static reference as well as the camera name in the mqtt events feed

I'd like to be able to add an key & value e.g. cameraSerial in the config per camera that is appended verbatim to the mqtt events feed

I had a quick look at the mqtt communication module however I could not make heads or tails of how to go about it.

@phillmac phillmac added the enhancement New feature or request label Oct 11, 2024
@phillmac
Copy link
Author

@NickM-27
Copy link
Collaborator

I think more information is needed, why do the names of cameras change? Why do you need to configure it Frigate this extra data, do you not already know it on the other side of MQTT?

I don't think it makes sense for Frigate to send arbitrary data along with all camera MQTT messages

@phillmac
Copy link
Author

phillmac commented Oct 26, 2024

@NickM-27
Why does the name of the cameras change?

  • Because they are portable and the display name of the cameras change depending on location.

Do you not know it already on the other side of MQTT?

  • No, there is a very real chance that the names will no longer match. The canonical identifier for the camera is the serial.
  • Alternately having a 'display nick name' option that shows in the web UI and elsewhere might solve the problem.
  • The serial number doesn't mean anything to a human so its not practical to use that as the displayed name, it would make frigate next to unusable with anything more than a couple of cameras

This should be a relatively trivial feature to add however I'm lost as to where to start on it.
If you are so inimical to adding this to the main code base would you kindly give me a few hints where to look?
Then I can at least try to muddle this thru myself..

@NickM-27
Copy link
Collaborator

It just doesn't make sense, maybe I'm misunderstanding. If you know the name of the camera to put into a friendly name field in the config why not just put that name in the existing name field in the config.

@phillmac
Copy link
Author

phillmac commented Oct 27, 2024

I tried that, everything went to hell in a handbasket. None of the live feeds would load anymore.
The every part of the api started throwing http 500 errors
I could load the bare UI itself but nothing more.

I couldn't find how to use it in the full config reference so im guessing it no longer works?

Edit: @NickM-27 Any comments? I can privately share the config & logs if needed

@NickM-27
Copy link
Collaborator

NickM-27 commented Nov 1, 2024

I can see why this might occur, I am not sure if this is a usecase that we are looking to support

@pnogemane
Copy link

Is the issue still available to pick up?

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

4 participants
@phillmac @NickM-27 @pnogemane and others