Auteur Topic: mogelijk bug in ProcessName()  (gelezen 4828 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: 1151
Laatste bericht 21 november 2023, 12:46:20
door Briolet
Extra functionaliteit in sickbeard TPB mogelijk?

Gestart door Satch_23Board SickBeard

Reacties: 0
Gelezen: 968
Laatste bericht 10 februari 2014, 10:36:49
door Satch_23
Meerdere Drive's mogelijk ?

Gestart door dirklammersBoard Cloud Station & Drive

Reacties: 1
Gelezen: 365
Laatste bericht 16 januari 2021, 16:58:13
door Briolet
Is deezer op nas synology mogelijk

Gestart door guppie1Board Media Streaming mods

Reacties: 1
Gelezen: 3653
Laatste bericht 23 september 2016, 18:39:13
door Stephan296
Connectie niet mogelijk met Foscam FI9821P, test werkt wel

Gestart door DikkesnoekBoard Surveillance Station

Reacties: 0
Gelezen: 1367
Laatste bericht 26 maart 2015, 10:52:13
door Dikkesnoek