Nieuwe opties beschikbaar gemaakt

[raptile]

Als je mijn bovenstaande wijzigingen wilt testen dan kan je mijn clone downloaden:

hg clone https://code.google.com/r/donnyvanosch-testing/

Thanks Donny!
Ik zie dat je er MinMatchScore RSS van hebt gemaakt. (Cool!)
Zou je mijn verzoekje aub door kunnen voeren voor de liefhebbers zodat ze per mail gelijk weten aan de hand van het onderwerp welke sub er gedownload is? (msg['Subject'] = subject + " " + message)


Als je de versie van Donny wilt testen:

Vanuit de command prompt:
hg clone https://code.google.com/r/donnyvanosch-testing/


C:Program Filesauto-sub>cd donnyvanosch-testing

C:Program Filesauto-subdonnyvanosch-testing>C:Python27python.exe AutoSub.py

[Donny]

Als je mijn bovenstaande wijzigingen wilt testen dan kan je mijn clone downloaden:

hg clone https://code.google.com/r/donnyvanosch-testing/

Thanks Donny!
Ik zie dat je er MinMatchScore RSS van hebt gemaakt. (Cool!)
Zou je mijn verzoekje aub door kunnen voeren voor de liefhebbers zodat ze per mail gelijk weten aan de hand van het onderwerp welke sub er gedownload is? (msg['Subject'] = subject + " " + message)


Als je de versie van Donny wilt testen:

Vanuit de command prompt:
hg clone https://code.google.com/r/donnyvanosch-testing/


C:Program Filesauto-sub>cd donnyvanosch-testing

C:Program Filesauto-subdonnyvanosch-testing>C:Python27python.exe AutoSub.py

De titel van je ondertiteling word al toegevoegd in de laatste commit van auto-sub:
http://code.google.com/p/auto-sub/sourc ... 736860627f

Klik maar eens op de Info knop naast Subject  

Even voor de duidelijkheid, als ik een change beschikbaar maak voor zyronix wil dat niet zeggen dat dit automatisch in de officiële auto-sub komt. zyronix zal eerst mijn versie ophalen en beslissen of dit in officiële auto-sub komt.

[raptile]

De titel van je ondertiteling word al toegevoegd in de laatste commit van auto-sub:
http://code.google.com/p/auto-sub/sourc ... 736860627f

Klik maar eens op de Info knop naast Subject  

Goed bezig! :-)

Even voor de duidelijkheid, als ik een change beschikbaar maak voor zyronix wil dat niet zeggen dat dit automatisch in de officiële auto-sub komt. zyronix zal eerst mijn versie ophalen en beslissen of dit in officiële auto-sub komt.

Dat was inderdaad al duidelijk.
Als ik zo vluchtig kijk zie ik dat het een behoorlijke verbetering is als ik het zo mag zeggen.
Meer rust/overzicht imo.

Nu nog subs voor films en kun je het verder met rust laten hahaha :-))



Edit:
Works like a charm!

Misschien nog even wat enters weghalen in de mail functionaliteit?
Hi,
 

 
AutoSub downloaded the following subtitle (language: nl):
 
Blue.Bloods.S02E21.720p.WEB-DL.DD5.1.H264-KiNGS.nl.srt

For the videofile:

Blue.Bloods.S02E21.720p.WEB-DL.DD5.1.H264-KiNGS.mkv

------ voorbeeld -------
Hi,
 
AutoSub downloaded the following subtitle (language: nl):
Blue.Bloods.S02E21.720p.WEB-DL.DD5.1.H264-KiNGS.nl.srt

For the videofile:
Blue.Bloods.S02E21.720p.WEB-DL.DD5.1.H264-KiNGS.mkv

[Donny]

ik zal zo na Formule 1 eens kijken

[raptile]

ik zal zo na Formule 1 eens kijken

HF :-)

------ laatste gedeelte uit mail.py ------
def send_notify(lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi,n
AutoSub downloaded the following subtitle (language: %s):
%s

For the videofile:
%s
    """ %(lang, subtitlefile, videofile))
    message['To'] = email.utils.formataddr(('Recipient', autosub.MAILTOADDR))
    message['Subject'] = '%s %s' %(autosub.MAILSUBJECT, subtitlefile)
    message = message.as_string()
    return _send_notify(message)

-------


Ik zie in mn log files overigens alleen maar info logs.
Debugging logs enzo zie ik niet.
Ik had e.e.a. verprutst in het mail.py bestand, heb auto-sub herstart, nogmaals een srt gedownload maar kreeg geen mail. ik zag niets terug in de logs.
Alleen de info log is gevuld. de rest is leeg. (Klopt dit?)

[Donny]

Heb je de aanpassing in mail.py opgeslagen? je kan python files niet zomaar in notepad ofzo aanpassen, dat werkt niet.

start je AutoSub wel met het juiste commando, zodat je de config file gebruikt die je opgeslagen hebt?

[raptile]

Heb je de aanpassing in mail.py opgeslagen? je kan python files niet zomaar in notepad ofzo aanpassen, dat werkt niet.

start je AutoSub wel met het juiste commando, zodat je de config file gebruikt die je opgeslagen hebt?

Ja, de laatste aanpassing zoals hierboven werkt overigens vlekkeloos.

Ik heb geen programmeer vaardigheden en probeerde het script ietsje aan te passen.

Ik had dit angepast:
def send_notify(lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi,

naar:
def send_notify(to, lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi s%,


Om te kijken of ik "Hi <email adres>," ervan kon maken.
Uiteraard werkt dit niet
Maar ik had het bestand aangepast, opgeslagen en vervolgens auto-sub gestopt en gestart zonder problemen.
Ook werd de sub gewoon gedownload.
De mailfunctionaliteit werkte echter niet meer.
Ik wist wel dat dit kwam door mijn aanpassing maar had verwacht dat hier iets over terug te vinden zou zijn in één van de logs.
Dit was echter niet het geval.

Alleen de info log wordt gevuld. de overige blijven leeg.
(Wat vreemd is aangezien debug bijvoorbeeld alles zou moeten tonen)

[Donny]

Heb je de aanpassing in mail.py opgeslagen? je kan python files niet zomaar in notepad ofzo aanpassen, dat werkt niet.

start je AutoSub wel met het juiste commando, zodat je de config file gebruikt die je opgeslagen hebt?

Ja, de laatste aanpassing zoals hierboven werkt overigens vlekkeloos.

Ik heb geen programmeer vaardigheden en probeerde het script ietsje aan te passen.

Ik had dit angepast:
def send_notify(lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi,

naar:
def send_notify(to, lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi s%,


Om te kijken of ik "Hi <email adres>," ervan kon maken.
Uiteraard werkt dit niet
Maar ik had het bestand aangepast, opgeslagen en vervolgens auto-sub gestopt en gestart zonder problemen.
Ook werd de sub gewoon gedownload.
De mailfunctionaliteit werkte echter niet meer.
Ik wist wel dat dit kwam door mijn aanpassing maar had verwacht dat hier iets over terug te vinden zou zijn in één van de logs.
Dit was echter niet het geval.

Alleen de info log wordt gevuld. de overige blijven leeg.
(Wat vreemd is aangezien debug bijvoorbeeld alles zou moeten tonen)

Heb je debug wel aangezet in je settings?

[raptile]

Heb je debug wel aangezet in je settings?

Doh! (Nu gedaan hahaha)


------- Dit werkt -------
def send_notify(lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi %s,n
AutoSub downloaded the following subtitle (language: %s):
%s

For the videofile:
%s
    """ %(email.utils.formataddr(('', autosub.MAILTOADDR)), lang, subtitlefile, videofile))
    message['To'] = email.utils.formataddr(('Recipient', autosub.MAILTOADDR))
    message['Subject'] = '%s %s' %(autosub.MAILSUBJECT, subtitlefile)
    message = message.as_string()
    return _send_notify(message)

------ Zo is het net iets beter omdat je de naam voor de @ ziet staan en niet het hele email adres -----

def send_notify(lang, subtitlefile, videofile):
    log.debug("Mail: Trying to send a mail")
    message = MIMEText("""Hi %s,n
AutoSub downloaded the following subtitle (language: %s):
%s

For the videofile:
%s
    """ %(email.utils.formataddr(('', autosub.MAILTOADDR)).split('@')[0], lang, subtitlefile, videofile))
    message['To'] = email.utils.formataddr(('Recipient', autosub.MAILTOADDR))
    message['Subject'] = '%s %s' %(autosub.MAILSUBJECT, subtitlefile)
    message = message.as_string()
    return _send_notify(message)

[Matr1x]

In het postprocess script is de exacte naam van de het srt bestand dat gedownload is niet bekend. Alleen de naam die het krijgt en die is dus hetzelfde als het filmbestand (muv de taal toevoeging). Ik vind het persoonlijk dus niet zinvol om beide namen te vermelden in de body van de e-mail, tenzij ze verschillende kunnen zijn. En dat zijn ze nu dus nooit.

[raptile]

In het postprocess script is de exacte naam van de het srt bestand dat gedownload is niet bekend. Alleen de naam die het krijgt en die is dus hetzelfde als het filmbestand (muv de taal toevoeging). Ik vind het persoonlijk dus niet zinvol om beide namen te vermelden in de body van de e-mail, tenzij ze verschillende kunnen zijn. En dat zijn ze nu dus nooit.

Dat heb ik niet aangepast/toegevoegd.

Ik heb de Enters tussen de regels weg gehaald en in mn laatste stukje heb ik het mail to (voor de @) toegevoegd in de body.
Stel, je mail ades is matrix@gmail.com, dan staat er in de body:
Hi Matrix,

AutoSub downloaded the following subtitle (language: nl):
Blue.Bloods.S02E18.720p.WEB-DL.DD5.1.H264-KiNGS.nl.srt

For the videofile:
Blue.Bloods.S02E18.720p.WEB-DL.DD5.1.H264-KiNGS.mkv

Dat laatste zou ik uiteraard weg kunnen halen, maar heb dit niet gedaan omdat ik het niet toegevoegd heb.

[Matr1x]

Het was ook meer een opmerking over hoe jammer het is dat we niet de werkelijk naam van het srt betand hebben dat gedownload wordt. Ik merk toch wel dat er voor de NL subs vaak net even een andere sub gedownload wordt. Meestal om een exacte match gewoon niet gevonden wordt. En omdat in de mail niet te zien is welk srt bestand er nu eigenlijk gedownload wordt, moet ik altijd even in de log kijken welke het nu was. Beetje jammer dat deze oorspronkelijke naam niet gewoon beschikbaar is voor de email. Misschien dat Donny nog een oplossig weet.

[raptile]

Ahh, ok, duidelijk verhaal :-) dat zou inderdaad wel cool zijn.

[Matr1x]

Staat er in de database misschien de originele naam naam het filmbestand (Wanted) en/of de originele naam van de srt die gedownload is (latest downloaded subtitles)? Want dan zou je in ieder geval op de AutoSub webpagine de info kunnen zetten. En dan in de notificatie email een hyperlink naar de AutoWeb webpagina, zodat je vanuit de e-mail met 1 click alsnog de info ziet op een webpagina.

En oh ja, de datum en tijd van de latested downloaded subtitle zou ook wel fijn zijn op dit op de webpagina te zetten (als dit beschikbaar is in de database).

[Donny]

Grote kans dat dit ook in Auto-Sub komt.
[ Gespecificeerde bijlage is niet beschikbaar ]autosub-changelog.png[/attachment:382b6kd4]
De Changelog zal online bijgehouden worden door zyronix op de Source pagina, zodat je direct in je huidige installatie kan zien welke features/bugfixes beschikbaar zijn in de nieuwe versie.