Autosub blijft niet draaien

[Donny]

Ik had gisteren al even gekeken en ik snap niet waarom dat ie crasht. In Addic7ed.py zit een stukje code die DIMENSION<>LOL (etc.) releases vergelijkt en om de een of andere reden wordt een stuk code uitgevoerd terwijl de release groep KINGS niet in de lijst staat. De code denkt dat het een hdtv release is en crasht omdat er geen 1080p in de quality list staat voor hdtv.

[Martijn85]

Aha jij bent al de derde die meld dat autosub stopt bij "Person of Interest" seizoen 4 aflevering 13.

Inmiddels heeft AutoSub deze wel al gedownload, eens kijken of die nu blijft draaien...

edit: lijkt nu wel goed door te draaien.

[Satch_23]

Geen idee hoe de code eruit ziet, maar op basis van wat ik lees ben ik geneigd te concluderen dat verschillende uitzonderingssituaties niet of onvoldoende worden afgevangen/afgehandeld. Wellicht zijn in deze uitzonderingssituaties ook meer expliciete meldingen handig zodat de situatie beter te achterhalen is.
Deels off topic: opensubtitles kan bepaalde subtitels ook uitzetten. terwijl er wel een verwijzing naar de subtitle is. Deze situaties ontdek je pas weer wanneer je zelf naar desbetreffende sub gaat zoeken op hun site. Overigens geen idee of je dergelijke situaties in de code kunt afvangen....

[DanielV_]

Ik heb hetzelfde probleem...
Van de week had ik toevallig al een tijd geen nieuwe subs meer gezien, waarop ik zag dat AutoSub was gestopt. Ik had blijkbaar nog een oude repo van M. de Vries erop staan dus was niet op de hoogte van de update. Vandaag heb ik dit aangepast (fijn dat opensubtitles is toegevoegd!  ), maar helaas loopt AutoSub nog steeds vast.

In de logs is niet te zien waarom de service stop, maar elke keer is de laatste check "Person of Interest S04 E13". Hierbij ook nog een kopie van mijn log.

[Ben(V)]

Ik had gisteren al even gekeken en ik snap niet waarom dat ie crasht. In Addic7ed.py zit een stukje code die DIMENSION<>LOL (etc.) releases vergelijkt en om de een of andere reden wordt een stuk code uitgevoerd terwijl de release groep KINGS niet in de lijst staat. De code denkt dat het een hdtv release is en crasht omdat er geen 1080p in de quality list staat voor hdtv.

@Donny Je zat dicht in de buurt van het probleem.
Autosub probeert met de informatie die hij heeft mogelijke combinaties van quality en releasegroup te matchen.
Bij deze aflevering stond er in op de website van Addic7ed als releasegroep 'dimension' en kwaliteit 1080p. Dit is een combinatie die autosub niet voorziet en als hij probeert een twinrelease daar voor te zoeken gaat het mis. Dat zou 'lol' met kwaliteit 'sd' moeten zijn.
Maar omdat er in de tabel geen 1080p is opgenomen en hij dat wel als index gebruikt crashed hij.
De oplossing is om in de functie _MakeTwinRelease (module addic7ed.py) dit stukje code:

Code: (python) [Selecteer]

    qualSwitchDict_hdtv = {u'sd' : u'720p',
                           u'720p' : u'sd'}

te vervangen door

Code: (python) [Selecteer]

    qualSwitchDict_hdtv = {u'sd' : u'720p',
                           u'720p' : u'sd',
                           u'1080p' : u'sd'}


[Donny]

Code: (python) [Selecteer]

    qualSwitchDict_hdtv = {u'sd' : u'720p',
                           u'720p' : u'sd',
                           u'sd' : u'1080p,
                           u'1080p' : u'sd'}


Ik zat niet alleen dicht bij het probleem, ik wist wat het probleem was.
Ik had alleen nog niet gezien dat op Addic7ed daar 1080p.dimension stond.

Dit is misschien wel de oplossing voor dit specifieke probleem, maar het is natuurlijk niet de bedoeling om elke fout die op addic7ed kan komen te staan af te vangen door 1080p als SD te gaan zetten. Of zijn er tegenwoordig 1080p HDTV downloads?

Overigens heb je in jouw code een fout staan, je sluit 1080p niet goed af:
u'sd' : u'1080p',

Ook is die regel niet nodig lijkt me, aangezien sd daarboven al omgezet wordt naar 720p.

edit 2: Ook zou ik het op prijs stellen als we eerst samen wat dingen testen voordat je zomaar een patch uitbrengt waar dus blijkbaar ook fouten in zitten, het is niet dat ik je 3 weken laat wachten als je mij een bericht stuurt.

[Ben(V)]

Sorry voor het kopieer foutje. Is alleen op het forum zo.

Ja dimension heeft tegenwoordig HDTV releases van 1080p, maar buiten dat zou autosub toch niet mogen crashen als er iets verkeerd ingevuld zou zijn.
Als 1080p als quality voorkomt in autosub moet die bij het zoeken naar een twinrelease natuurlijk niet crashen maar afvangen als dat geen geldige combinatie met een van de releasegroepen zou zijn.

EDIT Point taken, zal het niet meer doen.

[Donny]

Als 1080p als quality voorkomt in autosub moet die bij het zoeken naar een twinrelease natuurlijk niet crashen maar afvangen als dat geen geldige combinatie met een van de releasegroepen zou zijn.

Dat heb je nu nog steeds niet opgelost, je hebt alleen dit specifieke probleem opgelost.

Ja dimension heeft tegenwoordig HDTV releases van 1080p, maar buiten dat zou autosub toch niet mogen crashen als er iets verkeerd ingevuld zou zijn.

Zijn de 1080p versies sync met de sd versies?

[Ben(V)]

Ja ook voor HDTV geldt dat 720 en 1080 met elkaar in sync.

Verder zou daar de enig overgebleven kwaliteit zijnde 480 ook afgevangen moeten worden.
Dus de combo u'480':u'720' zou er ook in moeten om crashes te voorkomen.

PS Of de 1080 met sd syncen gaat het toch niet over. Die module twin release is toch om mogelijke combinaties te maken.
De scorematch later bepaalt of er iets geselecteerd wordt.

Of begrijp ik het niet goed?

[Donny]

Ja ook voor HDTV geldt dat 720 en 1080 met elkaar in sync.

Waarom staat de gewone DIMENSION en de 1080p DIMENSION dan apart aangegeven?
http://www.addic7ed.com/serie/Person_of_Interest/4/13/M.I.A.

[Ben(V)]

Geen flauw idee. Heb even gekeken maar ze zijn exact gelijk

[Donny]

Geen flauw idee. Heb even gekeken maar ze zijn exact gelijk

Ik heb het nu zo aangepast en hij crasht niet meer

Code: [Selecteer]

    qualSwitchDict_hdtv = {u'sd' : u'720p',
                           u'720p' : u'sd',
                           u'1080p' : u'720p',
                           u'480p' : u'720p'}

Ik denk dat ik het zo ga pushen, akkoord?

[Ben(V)]

Lijkt me een goed plan.

Kan Martijn of Matr1x er een nieuwe release voor Synology van maken.

[Donny]

De aanpaste patch staat nu online, je kan Martijn of Matr1x lastigvallen 

[Matr1x]

De aanpaste patch staat nu online, je kan Martijn of Matr1x lastigvallen 

Pakket is bijgewerkt. Zie ook: http://www.synology-forum.nl/auto-sub/autosub-bootstrap-bill-v0-6-2-1/msg148870/#msg148870