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 played_from, scene_name, and presentation_name for the attract loop director messages #468

Open
jandro-air opened this issue Sep 3, 2024 · 1 comment
Assignees

Comments

@jandro-air
Copy link
Contributor

Overview:
Director stats allow us to record statistical information about how the system is used via director messages triggered from the touchscreen and other interfaces. To support this, we added a few fields to the message that we send when playing presentations - mainly played_from, scene_name, presentation_name. An example of what fields are referenced in director stats is as follows:

> SELECT * FROM "director_stats";
name: director_stats
time                duration             host       played_from presentation_name product scene_name  support_name
----                --------             ----       ----------- ----------------- ------- ----------  ------------
1724914032029817276 4.882199287414551    lg-head-06 unknown     AMN00103          roscoe  AMN00103    ephq
1724914055993343204 0.1641829013824463   lg-head-06 unknown     AMN00103          roscoe  AMN00103    ephq
1724914061059054473 5.063735008239746    lg-head-06 unknown     NL000524          roscoe  NL000524    ephq
1724914064350011100 0.018364429473876953 lg-head-06 unknown     NL000524          roscoe  NL000524    ephq
1724914068660155101 4.3060526847839355   lg-head-06 unknown     open-cesium       roscoe  open-cesium ephq
1724914077074943033 8.412491083145142    lg-head-06 unknown     Force earth       roscoe  Force earth ephq

One place we did not update was in the messages that the Attract Loop sends when playing content. It would be good to add this so that there are no discrepancies or errors in the stats we pull. See below for current messages:

Requirements:
List of requirements that need to be satisfied that are needed to consider this feature implemented.

  • Attract loop sends additional fields in the message to match new touch interfaces changes.
@constantegonzalez
Copy link
Contributor

#469

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

No branches or pull requests

2 participants