Opened 8 years ago
Closed 4 years ago
#12901 closed Bug Report - General (Unverified)
longeraudiograph patch
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | MythTV - General | Version: | Unspecified |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Ever since lvr's patch from #10848 was committed, the audiograph has seemed to me to be impressive and pretty but not really useful. I use the editor many times a day, almost always with a smallest step of 1-keyframe in TV or 1-second in audio, and the claimed 1-frame display width isn't a good fit. I think it actually varies, too.
I believe that the attached patch widens the scope to 32 frames - 1.28 seconds for the UK. I'm less happy about the accuracy with which the audio buffer is refreshed and tracked during editing, so I'm posting it in the hope that someone who understands this (and C++) will have a look. It applies cleanly to current master.
Attachments (3)
Change History (6)
Changed 8 years ago by
| Attachment: | longeraudiograph.patch added |
|---|
Changed 8 years ago by
| Attachment: | audiograph.png added |
|---|
Changed 8 years ago by
| Attachment: | themediff_selected.txt added |
|---|
comment:1 Changed 8 years ago by
I initially forgot to add a patch for the MythCenter?-Wide theme. It's a diff between current master and my local modified copy in ~/.mythtv. I don't think the second part should have been included; it just shifts the results box in the Search Recordings screens.
comment:2 Changed 7 years ago by
A few new comments. I find that the patch above is moderately useful when using larger step sizes as quoted. But the display does not update well during 1-frame steps, and the scale on the time axis is not as intended, by a factor of (samples per ms), ie 48. Work-in-progress, slowly.
comment:3 Changed 4 years ago by
| Resolution: | → Unverified |
|---|---|
| Status: | new → closed |
Closing all old tickets in trac.
If your issue still persists, please open an issue in Github https://github.com/MythTV/mythtv/issues
and reference the existing trac ticket.
patches to MythCenter?-Wide for this Ticket