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.

Sign up for GitHub

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

Jump to bottom

Add option to omit speaker if it hasn't changed #20

Closed
stevencrader opened this issue Apr 23, 2023 · 0 comments · Fixed by #21
Closed

Add option to omit speaker if it hasn't changed #20

stevencrader opened this issue Apr 23, 2023 · 0 comments · Fixed by #21
Assignees
@stevencrader

Comments

@stevencrader
Copy link
Owner

If the speaker has not changed, don't include the speaker field.

This shall be an option. The default behavior will be to include speaker always.

From https://podcastindex.social/@mitch/110250281748259259
@stevencrader stevencrader self-assigned this Apr 24, 2023
stevencrader added a commit that referenced this issue May 16, 2023
@stevencrader
Add support for controlling how segments are combined
f316ebc 
- `combineEqualTimes`: Combine segments if the `startTime`, `endTime`, and `speaker` match between the current and prior segments. Resolves #19
- `speakerChange`: Only include `speaker` when speaker changes. Resolves #20
- `combineSegments`: Replaces `combineSingleWordSegments` function. Combine segments where speaker is the same and concatenated `body` fits in the `combineSegmentsLength`
@stevencrader stevencrader mentioned this issue May 16, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stevencrader stevencrader

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for specifying how segments should be combined stevencrader/transcriptator
1 participant
@stevencrader