Auteur Topic: mogelijk bug in ProcessName()  (gelezen 5286 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 mogelijk een ircbot te installeren?

Gestart door HaVBoard Overige 3rd party packages

Reacties: 10
Gelezen: 8042
Laatste bericht 30 augustus 2011, 20:45:18
door cyrus1977
Hot swap ...mogelijk

Gestart door ALKMAARBoard Synology DSM algemeen

Reacties: 7
Gelezen: 2527
Laatste bericht 16 augustus 2016, 22:19:03
door Birdy
Download station - Re-seed een torrent niet mogelijk?

Gestart door wiske80Board Download Station

Reacties: 0
Gelezen: 582
Laatste bericht 08 september 2020, 08:48:54
door wiske80
raid verkleinen, is dat mogelijk?

Gestart door B3rtBoard NAS hardware vragen

Reacties: 4
Gelezen: 2221
Laatste bericht 17 maart 2015, 22:28:54
door B3rt
DS411 slim firmware update mislukt. Recovery mogelijk?

Gestart door cdronBoard Synology DSM algemeen

Reacties: 6
Gelezen: 636
Laatste bericht 17 september 2023, 22:26:02
door cdron