Auteur Topic: mogelijk bug in ProcessName()  (gelezen 5403 keer)

Offline stappel

  • Bedankjes
  • -Gegeven: 0
  • -Ontvangen: 0
  • Berichten: 7
mogelijk bug in ProcessName()
« Gepost op: 12 mei 2018, 10:51:04 »
Ik denk dat er een bug in de functie ProcessName() zit.
ik vond zelf correcte subs op addic7ed maar AutoSub niet. Daarom maar eens gaan debuggen.
Ik denk dat het ergens fout gaat als ProcessName() voor het eerst wordt aangeroepen. Maar dat is slechts een vermoeden.

Debug info:
Het gaat bijvoorbeeld fout bij de serie The Expanse, AutoSub gebruikt de volgende URL voor Addic7ed.
http://www.addic7ed.com/ajax_loadShow.php?show=5513&season=3&langs=|1|17|&hd=0&hi=-1
Daar zijn bij episode 5, 2 regels met subs. de eerste heeft SVA als "version" staan. dat is de release groep.
Mijn bestand is: The.Expanse.S03E05.720p.HDTV.x264-SVA.mkv
als ik extra debug informatie in de .py files stop zie ik dat:
ProcessName(SubInfo[4]) wordt aangeroepen met: SVA in de eerste run (en daarna de volgende regel met WEB-DEFLATE). Dat klopt dus.
Het resultaat in Release is echter de eerste keer:
    {'info': u'..', 'releasegrp': None, 'source': None, 'codec': None, 'rlsgrplst': [], 'quality': None, 'distro': None}
geen releasegrp gevonden dus.
in de 2e run/regel/check met WEB-DEFLATE komt er terug:
    {'info': u'.-deflate.', 'releasegrp': u'deflate', 'source': u'web', 'codec': None, 'rlsgrplst': [u'deflate'], 'quality': None, 'distro': None}
die is dus wel gevult.

dit zie bij meerder series.
http://www.addic7ed.com/ajax_loadShow.php?show=5810&season=2&langs=|1|17|&hd=0&hi=-1
eps 21, ook de eerste keer blijft de release grp op None staan.

  • Mijn Synology: DS414
  • HDD's: 2

Offline stappel

  • Bedankjes
  • -Gegeven: 0
  • -Ontvangen: 0
  • Berichten: 7
Re: mogelijk bug in ProcessName()
« Reactie #1 Gepost op: 12 mei 2018, 11:26:14 »
Volgens mij komt het hierdoor:
    show_dict['info'] = show_dict['info'][Pos:]
in de file ProcessFilename.py

Als er GEEN codec en  GEEN quality en GEEN distro, etc gevonden wordt in de infon string, staat Pos nog op de lengte van de gehele string en wordt dus geheel verwijderd en wordt show_dict['info'] een lege string.

Er moet dus (vermoedelijk) een check bij komen of Pos nog de orginele Pos is en dan skippen.
Als ik even snel deze regel uit comment, dan vindt hij 5 extra subs in mijn library. allemaal correcte downloads.
  • Mijn Synology: DS414
  • HDD's: 2

Ben(V)

  • Gast
Re: mogelijk bug in ProcessName()
« Reactie #2 Gepost op: 12 mei 2018, 13:28:16 »
Klopt die bug was al gevonden en gecorrigeerd.
Oplossing komt met de volgende release mee.

Prima analyse overigens.

Offline stappel

  • Bedankjes
  • -Gegeven: 0
  • -Ontvangen: 0
  • Berichten: 7
Re: mogelijk bug in ProcessName()
« Reactie #3 Gepost op: 12 mei 2018, 20:54:38 »
Ok.
er is geen development branch in github?
  • Mijn Synology: DS414
  • HDD's: 2

Ben(V)

  • Gast
Re: mogelijk bug in ProcessName()
« Reactie #4 Gepost op: 13 mei 2018, 00:59:18 »
Nee development die ik alleen lokaal.
Meestal release ik de bug fixies vrij snel,maar door omstsndigheden loop ik nu wat achter.


 

Is het maken van een Root map mogelijk?

Gestart door VuurvreterBoard Synology DSM 6.1

Reacties: 9
Gelezen: 4631
Laatste bericht 28 november 2017, 21:08:51
door Vuurvreter
Download station - Re-seed een torrent niet mogelijk?

Gestart door wiske80Board Download Station

Reacties: 0
Gelezen: 590
Laatste bericht 08 september 2020, 08:48:54
door wiske80
DS411 slim firmware update mislukt. Recovery mogelijk?

Gestart door cdronBoard Synology DSM algemeen

Reacties: 6
Gelezen: 653
Laatste bericht 17 september 2023, 22:26:02
door cdron
is het mogelijk een ircbot te installeren?

Gestart door HaVBoard Overige 3rd party packages

Reacties: 10
Gelezen: 8098
Laatste bericht 30 augustus 2011, 20:45:18
door cyrus1977
Live weergave werkt niet meer, mogelijk na laatste update DSM

Gestart door Herman GiebelsBoard Surveillance Station

Reacties: 3
Gelezen: 2496
Laatste bericht 04 februari 2014, 21:05:08
door Herman Giebels