Bug #4816
No more information in "Subtitle" available
100%
Description
Hi,
see screenshot. Im using the EIT DVB grabber only without any change for a long time and this column used to be filed for the most channels.
br
Alfred
Files
Subtasks
History
Updated by Jaroslav Kysela almost 7 years ago
- Status changed from New to Fixed
- % Done changed from 0 to 100
Applied in changeset commit:tvheadend|47660042aa6db77b34551a1053991bb8e4831258.
Updated by La Marqe almost 7 years ago
I would like to reopen this bug.
The above change fixes only the WebUI, but not the file name pattern %s for recordings.
Updated by La Marqe almost 7 years ago
- File recordings_grid.jpg recordings_grid.jpg added
Not just the recording file names are incomplete, also the WebUI is fixed only partially.
The subtitle columns in the recording grids are empty.
Updated by Jaroslav Kysela almost 7 years ago
- Subject changed from No more information in "Subtitle" availble to No more information in "Subtitle" available
- Status changed from Fixed to Accepted
There is no subtitle in EIT data. Previous version of tvh just make a copy of the summary text to the subtitle, but it's not correct.
Updated by La Marqe almost 7 years ago
I see your point.
Adding a new column and new % pattern for the summary text would be an equally good solution to me and would avoid further extension of the previous hack.
Updated by Robin Mitra almost 7 years ago
Jaroslav Kysela wrote:
There is no subtitle in EIT data. Previous version of tvh just make a copy of the summary text to the subtitle, but it's not correct.
The epg shows subtitle, so the eit has it (and I confirmed that with other tools), and that was also used in the dvr settings, but not anymore.
I can understand that some providers don't set it, but most do. Shouldn't that take precedence over any other info gleaned from guessing at description fields?
Updated by Jaroslav Kysela almost 7 years ago
EPG grid shows summary when subtitle is empty (see the new header description). Something similar should be added for DVR, but I'm working on other things at the moment.
Updated by Robin Mitra almost 7 years ago
I get what you're trying to do.
Only I fear that an advanced (as in nice to have) feature is currently breaking a working basic feature.
Shouldn't a newly commited feature attempt be rejected if it breaks an existing and working basic feature?
Updated by Michael Schönborn almost 7 years ago
I don't know if this issue was already fixed, but after I recreated all autorecs out of the EPG all subtitles were also shown in the upcoming recordings tab. Even the duplicates from the finished recordings where filtered.
I will wait, if the next recordings will be successful and the subtitle will be added to the filename. And if the future recordings after a epg update will also show the subtitle.
Updated by Daniel Mayer almost 7 years ago
+1 to buggy-in-usage.
The working not-correct feature should not removed before "correct" feature runs.
It's fact that now there's no recording of series possible as the files are only counted "Stargate SG1 - 1.ts" -2, -3, etc. Until December the subtitles where in the filename.
1) You could provide an option field for old/new habit in Recording-profile
2) I understand the changeset above as: "summary" should not fill in for "subtitle". So: Add the action to save "summary" to disk instead als .eit-file with same filename to your changeset.
Updated by Daniel Mayer almost 7 years ago
Edit: possibly faulty "summary" should not fill in for "subtitle" is what I understood out of your changeset.
Updated by Jaroslav Kysela almost 7 years ago
It's definitely bug to "propagate" directly and blindly a copy of summary to subtitle field from the EPG grabber to the upper layers. The commit which removed this (by default) is https://github.com/tvheadend/tvheadend/commit/9b40e5a26915161e6904001154afdb3e68de49d0 , so as you can see, it's possible to revert to the old behaviour in the EIT config by checking one checkbox.
I'm working on a proper solution (will be released probably tomorrow, because other code cleanups).
Updated by Jaroslav Kysela almost 7 years ago
- Status changed from Accepted to Fixed
Applied in changeset commit:tvheadend|10e3db3726124515711b1e36f26b7c2feb45fced.
Updated by Robin Mitra almost 7 years ago
Jaroslav Kysela wrote:
...
revert to the old behaviour in the EIT config by checking one checkbox.
I need to find that config, didn't know that existed (or forgot about it)
Updated by Robin Mitra almost 7 years ago
This is still broken.
The fact that it got rejected tells me that the subtitle feature is gone in favor of something that noone understands and can use except maybe the author.
This renders the dvr pretty much useless for series recordings for normal mortals.
Updated by Mark Clarkstone almost 7 years ago
From my small testing there does seem to be some issues with the subtitle processing, but imo, it's nothing major.
What I observed.
If you have more than one grabber enabled & scraping the subtitle from the description is enabled for all, it will sometimes be correctly processed and added to the DVR subtitle, but not always. This is usually due to the winning grabber not having that info, so, no problem there..
Before I started to tinker, I'd not set the "Set subtitle to summary" option for any of the grabbers, I did have the scrap subtitle option enabled (for all).
My DVR list of upcoming recordings had the correct subtitle for "Doctors" on BBC One. Once I had enabled "Set subtitle to summary" (for all) and re-enabled the autorec of "Doctors" the subtitle disappeared for all but one entry.
This is with 4.3-985, I see there's 4.3.991 out so I'll update and see if that makes a difference.