specs icon indicating copy to clipboard operation
specs copied to clipboard

Add field of view support to PTZ metadata streams

Open Ekmansoft opened this issue 2 years ago • 12 comments

For testing and improving analytics apps and services (for example object detection) when a PTZ camera is moving, it is helpful to keep track of the camera's horizontal and vertical field of view.

This patch adds support for recording horizontal and vertical field of view in the ONVIF PTZ metadata together with the already present information about pan- and tilt-coordinates.

Use case: A crime has been committed. Suspects may have passed a street which is visible by one or more PTZ cameras when looking in certain directions. This PTZ meta-data can help an operator find the sections of the recordings from these cameras where the camera is looking in the direction where they may have been visible. 

In some cameras / applications translation from PTZ camera view (Pan-tilt-direction and FOV) to geolocation (GPS coordinate) may be attempted. However, since the camera need very precise and extensive information about the surrounding geography for these calculations to be accurate, pan-, tilt-position together with horizontal and vertical FOV is often a good alternative way to calculate the geolocation of visible objects on the server side, either when the video is recorded or when replayed instead of calculating it on the end-device when recorded.

To be able to do this we need to be able to record pan- and tilt-position along with FOV.

Ekmansoft avatar Mar 02 '23 09:03 Ekmansoft

Based on Zagreb F2F meeting, decided to close this PR since there is no response from @Ekmansoft.

sujithhanwha avatar Sep 13 '23 13:09 sujithhanwha

I've been on vacation, and didnt follow up on this since I'm not the one monitoring the Onvif standard.

My memory is not perfect sorry @sujithhanwha .

Ekmansoft avatar Sep 13 '23 13:09 Ekmansoft

Based on Zagreb F2F meeting, decided to close this PR since there is no response from @Ekmansoft.

@sujithhanwha sorry for missing this.

How can it be reopened?

Ekmansoft avatar Sep 13 '23 13:09 Ekmansoft

Reopening the PR based on request from @Ekmansoft.

sujithhanwha avatar Sep 13 '23 14:09 sujithhanwha

Close as there is no update on this item for quite some time.

HansBusch avatar Nov 09 '23 22:11 HansBusch

Close as there is no update on this item for quite some time.

What are you talking about? I updated it yesterday!

Ekmansoft avatar Nov 10 '23 12:11 Ekmansoft

Close as there is no update on this item for quite some time.

Please reopen @HansBusch

Ekmansoft avatar Nov 10 '23 14:11 Ekmansoft

@Ekmansoft Here is my suggestion to be in sync with the existing structure of "PTZStatusFilterOptions" in onvif.xsd


<xs:element name="FieldOfViewSupported" type="xs:boolean" minOccurs="0">       xs:annotation             xs:documentationTrue if the device is able to stream the field-of-view information.</xs:documentation>       </xs:annotation> </xs:element>


The structures "PTZStatusFilterOptions" and "PTZFilter" are seemingly similar and quite natural to see the ambiguity and just to clarify, hope you see the difference.

PTZStatusFilterOptions

  • This includes the device capability to include FOV in the stream

PTZFilter:

  • This is a client configuration to actually request/expect the FOV data in the metadata stream. This would mean even if the device is capable to include the data, with this configuration client may choose to request/reject that information from the device.

bsriramprasad avatar Nov 22 '23 11:11 bsriramprasad

Thanks for the comments. I have updated and rebased now so from my side I think it's worth reviewing again.

Ekmansoft avatar Nov 30 '23 10:11 Ekmansoft

@jflevesque-genetec I reviewed this PR and was happy to approve. I noted that you requested changes and blocked merging. Can you please re-review and approve if your happy.

kieran242 avatar Feb 06 '24 15:02 kieran242

@HansBusch @sujithhanwha This has been approved from Feb before the last F2F in March. Are we holding off on this PR from merging for a reason that I may have missed?

kieran242 avatar May 27 '24 13:05 kieran242

This PR is marked for 24.12 release, so it will be merged after 24.06 release.

venki5685 avatar May 27 '24 13:05 venki5685