Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
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

Video/object track aggregation event #437

Open
wants to merge 14 commits into
base: development
Choose a base branch
from

Conversation

bsriramprasad
Copy link
Contributor

@bsriramprasad bsriramprasad commented Jun 24, 2024

Here is the summary of changes included in PR

  • Created a new section "Object Track Aggregation" event description in Analytics specification
  • Created schema structures referred in the event description
  • No additional changes in Metadata configuration, follows existing Event transport.
  • Not defined as a property event
    • Primary use case to standardize only the 'final aggregation'
      • Initial/intermediate aggregations are made optional
    • use case NOT at all focused to get the state of every object in the scene at any time.
  • If we are sending only one (final aggregation) event per object, it is an overkill to fit into the property event paradigm.

23/08/2024 Update

  • Removed individual structures created for snapshot/bounding box
    • Reused the tt:Object with the capture time attribute
  • Included Rule options for below
    • ReportTimeInterval
    • Data filter for including complement data like snapshot/bounding box
    • Add Rule options response sample

10/09/2024 Update
Addressed below changes based on the Bangkok F2F discussions

  • Rename ObjectComplement as ObjectTrack
  • Clarify Object track data as additional data generated by the device based on the ObjectTrackDataFilter configured by the client
  • Fixed rule type as tt:ObjectTrack, incorrectly referring tt:ObjectClassification
  • Remove AggregationEnd and use event time instead
  • Fixed the extension structure as suggested by @HansBusch

@bsriramprasad bsriramprasad marked this pull request as ready for review June 26, 2024 09:15
doc/Analytics.xml Outdated Show resolved Hide resolved
doc/Analytics.xml Outdated Show resolved Hide resolved
doc/Analytics.xml Outdated Show resolved Hide resolved
doc/Analytics.xml Outdated Show resolved Hide resolved
Address review feedback
Copy link
Member

@HansBusch HansBusch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There seems to be no way for the client to control the amount of produced track information.
Do we need a parameter to limit the number of snapshots?

doc/Analytics.xml Outdated Show resolved Hide resolved
wsdl/ver10/schema/metadatastream.xsd Outdated Show resolved Hide resolved
doc/Analytics.xml Show resolved Hide resolved
doc/Analytics.xml Outdated Show resolved Hide resolved
@bsriramprasad
Copy link
Contributor Author

bsriramprasad commented Aug 15, 2024

There seems to be no way for the client to control the amount of produced track information. Do we need a parameter to limit the number of snapshots?

If we modify the Object track complement structure to change to include tt:Object to make it more generic, Are we not opening up for more data to be sent that could be hard to filter?

  • That's the reason we have created a focused structure with just snapshot and bounding box.
    • Our previous F2F discussion focused a lot on trying to filter data - but it turned out to quite complex and hence opted for more focused proposal.

wsdl/ver10/schema/metadatastream.xsd Outdated Show resolved Hide resolved
wsdl/ver10/schema/metadatastream.xsd Outdated Show resolved Hide resolved
@bsriramprasad bsriramprasad added the enhancement New feature or request label Aug 26, 2024
@HansBusch
Copy link
Member

Editorial remark on extending types:

	<xs:complexType name="ObjectComplementData">
		<xs:complexContent>
			<xs:extension base="tt:Object">
				<xs:attribute name="CaptureTime" type="xs:dateTime"/>
				<xs:anyAttribute processContents="lax"/>
			</xs:extension>
		</xs:complexContent>
	</xs:complexType>

@bsriramprasad
Copy link
Contributor Author

Editorial remark on extending types:

	<xs:complexType name="ObjectComplementData">
		<xs:complexContent>
			<xs:extension base="tt:Object">
				<xs:attribute name="CaptureTime" type="xs:dateTime"/>
				<xs:anyAttribute processContents="lax"/>
			</xs:extension>
		</xs:complexContent>
	</xs:complexType>

Thanks for the suggestion, updated the schema as suggested.

listed scenarios </para>
<itemizedlist>
<listitem>
<para> Optionally, Initial aggregation after an object appears OR an Intermediate aggregation in the field of view
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the redundant text about intermediate aggregation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants