View Issue Details

IDProjectCategoryView StatusLast Update
0006262The Dark ModSound Systempublic16.06.2023 13:38
Reporterstgatilov Assigned Tostgatilov  
PrioritynormalSeverityfeatureReproducibilityN/A
Status resolvedResolutionfixed 
Product VersionTDM 2.11 
Target VersionTDM 2.12Fixed in VersionTDM 2.12 
Summary0006262: Allow subtitles to display beyond sound duration
DescriptionSubtitles are implemented in such a way that they are displayed for all the sound samples playing inside the sound system.
As soon as sound ends, it gets deleted form the sound system, and there is no way to display subtitle for it.
Which is quite limiting, especially for very short and quick sounds like "Hey!".

So the idea is to not delete sound from sound system while its subtitles might still be playing.
Additional InformationSee also:
  https://forums.thedarkmod.com/index.php?/topic/21741-subtitles-possibilities-beyond-211/&do=findComment&comment=483182
TagsNo tags attached.

Relationships

related to 0006264 resolvedstgatilov Subtitle improvements 

Activities

stgatilov

stgatilov

26.02.2023 19:13

administrator   ~0015957

Some preparational commits:
  r10279 Added idSoundSample::DurationIn44kHzSamples in addition to LengthIn44kHzSamples, fixed some usages.
  r10280 Small tweak (use DurationIn44kHzSamples).
  r10281 Some refactoring in idSoundEmitterLocal::CheckForCompletion (avoid copy/paste).
  r10282 Fixed bug that stereo samples take 2x time to "complete" and stop playing.
  r10283 Reworked idSoundChannel::GatherSubtitles to support extension of sound duration for subtitles.
stgatilov

stgatilov

26.02.2023 19:39

administrator   ~0015958

Committed the change in svn rev 10284.
It adds three cvars for now:
  tdm_subtitles_inlineDurationExtension = 0.2 seconds: increase inline subtitles duration
  tdm_subtitles_inlineDurationMinimum = 1 second: inline subtitles duration must be at least this
  tdm_subtitles_durationExtensionLimit = 5 seconds: extension of sound duration due to subtitles is capped by this

The default value of 200 milliseconds is the same as the one hardcoded in conversation system.
So now in a conversation with "wait until finished" talk sounds, subtitles exactly follow each other.
stgatilov

stgatilov

13.04.2023 17:14

administrator   ~0015998

Supported "-dx 1.23" or "-durationExtend 1.5" option on inline subtitles in svn rev 10358.
Geep requested a way to tweak this extension on per-subtitle basis, so here it is.
Geep

Geep

20.04.2023 18:29

reporter   ~0015999

Is this available yet as a dev release, e.g., in the 2.12dev 16792-10357? Along with minor data fixups requested in [6284] would be most helpful.
stgatilov

stgatilov

21.04.2023 16:21

administrator   ~0016000

Not yet.
But I'll release a new dev build this weekend for sure =)
Geep

Geep

01.05.2023 20:49

reporter   ~0016004

Nice to see -dx in action now. For anyone reading this, -dx overrides (instead of adds to) the default 0.20 seconds.

Issue History

Date Modified Username Field Change
26.02.2023 19:11 stgatilov New Issue
26.02.2023 19:11 stgatilov Status new => assigned
26.02.2023 19:11 stgatilov Assigned To => stgatilov
26.02.2023 19:13 stgatilov Note Added: 0015957
26.02.2023 19:39 stgatilov Note Added: 0015958
26.02.2023 22:45 stgatilov Status assigned => feedback
26.02.2023 22:53 stgatilov Relationship added related to 0006264
13.04.2023 17:14 stgatilov Note Added: 0015998
20.04.2023 18:29 Geep Note Added: 0015999
21.04.2023 16:21 stgatilov Note Added: 0016000
01.05.2023 20:49 Geep Note Added: 0016004
16.06.2023 13:38 stgatilov Status feedback => resolved
16.06.2023 13:38 stgatilov Resolution open => fixed
16.06.2023 13:38 stgatilov Fixed in Version => TDM 2.12