Auteur Topic: mogelijk bug in ProcessName()  (gelezen 4726 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.


 

Engelstalige forum - i.v.m. massa's SPAM - geen reacties mogelijk.

Gestart door BabyloniaBoard Synology Router

Reacties: 19
Gelezen: 1064
Laatste bericht 21 november 2023, 12:46:20
door Briolet
Extra functionaliteit in sickbeard TPB mogelijk?

Gestart door Satch_23Board SickBeard

Reacties: 0
Gelezen: 959
Laatste bericht 10 februari 2014, 10:36:49
door Satch_23
Connectie niet mogelijk met Foscam FI9821P, test werkt wel

Gestart door DikkesnoekBoard Surveillance Station

Reacties: 0
Gelezen: 1359
Laatste bericht 26 maart 2015, 10:52:13
door Dikkesnoek
is het mogelijk om debug logs aan te zetten?

Gestart door raptileBoard Headphones

Reacties: 0
Gelezen: 3044
Laatste bericht 22 oktober 2012, 01:05:26
door raptile
rs3411xs cpu upgrade van i3 naar i5 mogelijk ?

Gestart door frikandelBoard NAS hardware vragen

Reacties: 5
Gelezen: 1425
Laatste bericht 02 januari 2018, 14:59:56
door florisvn