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

Track how often TTS is played #26689

Merged
merged 1 commit into from Jan 23, 2019
Merged

Track how often TTS is played #26689

merged 1 commit into from Jan 23, 2019

Conversation

epeach
Copy link

@epeach epeach commented Jan 22, 2019

We have anecdotal evidence that TTS is used by students and teachers have requested more of it.
This change will help us build a set of data to support or refute these anecdotes, answer questions about TTS usage, and measure changes in TTS behavior as we extend TTS support.

This change tracks when a user selects play on a TTS audio file and records which file got played (which is associated with a single level), the puzzle number it was played on (in case a single level is used in multiple scripts), and the user id of who played it.

cc: @ryansloan

@epeach epeach requested a review from joshlory January 22, 2019 18:43
study: 'tts-play',
study_group: 'v1',
event: 'play',
data_string: this.props.src,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it straightforward to map audio file src back to the message being played?

Copy link
Author

Choose a reason for hiding this comment

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

If I understand your question correctly:
The src will be something like: https://tts.code.org/sharon22k/180/100/dcd9c0d72e54896a7d9a3399997885f1/courseC_maze_programming8_predict1_2018.mp3

Each level has a corresponding TTS file, so we can associate the above audio url with the level: courseC_maze_programming8_predict1_2018

Copy link
Contributor

@joshlory joshlory left a comment

Choose a reason for hiding this comment

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

LGTM!

@epeach epeach merged commit 3875464 into staging Jan 23, 2019
@epeach epeach deleted the TTS-logging branch March 26, 2019 17:55
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

Successfully merging this pull request may close these issues.

None yet

2 participants