Conversation
|
Questa opzione direi che e' fondamentale! 😎 |
rfc1459
left a comment
rfc1459
left a comment
There was a problem hiding this comment.
Riassunto:
- cambio non necessario dell'interprete che - tra l'altro - rompe la compatibilità con sistemi dove bash non è in
/bin/bash - bashismi (ecco perché è stato cambiato l'interprete)
- merge conflict
Come minimo ci vuole un rebase, ma direi che l'intera PR non va bene. (nota a margine: chi ha privilegi un pelino più alti dei miei potrebbe attaccare il repo a Travis CI così da eseguire shellcheck su ogni PR)
| @@ -1,4 +1,4 @@ | |||
| #!/bin/sh | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
No, no, no.
E se non si fosse capito: no.
Non ha senso modificare l'interprete solo perché ci sono dei bashismi.
|
|
||
| espeak_best_voice() { | ||
| v=$(espeak --voices=it | tail -n +2 | awk '{ print $4 }' | grep -v mbrola | head -n1) | ||
| if [[ -n "$v" ]]; then |
| return | ||
| fi | ||
| v=$(espeak --voices=it | tail -n +2 | awk '{ print $4 }' | head -n1) | ||
| if [[ -n "$v" ]]; then |
|
ti va di chiarire il bashismo? trovo sempre difficile riconoscere i bashismi perché l'unico modo per farlo è leggere l'intero elenco di bashismi e capire cosa non va (o non imparare bash ma solo shell posix, certo). Se mi dai una dritta al volo poi provo a fixare. |
|
|
|
ok grazie, credo di aver risolto. In realtà shellcheck continua a lamentarsi, ma secondo me si sbaglia lui. Dice che echo "-v $v -k10 -g1 -p 30"non va bene perché |
|
E' decisamente un falso positivo, prima della riga incriminata aggiungi |
rfc1459
left a comment
rfc1459
left a comment
There was a problem hiding this comment.
Much much better :-)
Manca giusto un rebase sul master attuale.
|
Manca giusto un rebase sul master attuale.
dunque io un rebase l'ho pure fatto, ma c'è un problema antipatico. Nel
frattempo la sorgente dell'audio è cambiata da google ad ispeech. A
parte che ora l'opzione --google prenderebbe gli audio da ispeech.org,
il che è quantomento confondente :)
Ma poi ispeech.org non funziona, lamentandosi (in modo spassosissimo)
che "invalid license, please visit ispeech.org for a valid license".
Quindi dovrei fare un rebase che rompe un po' di cose e questo non mi va
tantissimo. Quasi quasi aggiungo l'opzione --google e --ispeech va. Tu
che ne pensi?
|
Che belli i servizi che nel giro di un anno (data del merge di #15) da free diventano a pagamento... Mi sa che l'unica è ranzare del tutto il supporto ai TTS esterni e usare solo espeak se disponibile sulla macchina. |
|
Mi sa che l'unica è ranzare del tutto il supporto ai TTS esterni e usare solo espeak se disponibile sulla macchina.
@henryx, @LegolasTheElf, @aleskandro, @dariomas: commenti?
per ora ho pushato un coso che in teoria supporta tutto, anche se in
pratica funziona solo espeak (che ora è il default). Ho migliorato il
supporto ad espeak in modo da passare tramite mplayer.
|
|
Probabilmente rimuovere il supporto ai TTS mi sembra l'unica soluzione, a meno che non si trovi un'alternativa valida (e.g. Zanata, Bing o Yandex) |
|
Il 06/02/2017 13:24, Matteo Panella ha scritto:
Ma poi ispeech.org non funziona, lamentandosi (in modo
spassosissimo) che "invalid license, please visit ispeech.org for a
valid license".
Che belli i servizi che nel giro di un anno (data del merge di #15
<#15>) da free diventano
a pagamento...
Mi sa che l'unica è ranzare del tutto il supporto ai TTS esterni e usare
solo espeak se disponibile sulla macchina.
Personalmente chiederei espeak come prerequisite e fine supporto a
google e ispeech.
in primo luogo tutte le volte si fanno i cavoli loro cambiando licenza e
cazziamazzi vari, in secondo luogo oggi ci sono domani non più.
già dipendiamo comunque da santiebeati per l'elenco dei santi, ridurrei
il più possibile le dipendenze da servizi esterni
… @henryx <https://github.com/henryx>, @LegolasTheElf
<https://github.com/LegolasTheElf>, @aleskandro
<https://github.com/aleskandro>, @dariomas
<https://github.com/dariomas>: commenti?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIhH_VvLjiMBLiyOelPwnAxqsCCDelCPks5rZxEDgaJpZM4FoNtn>.
--
salumi e caci a tutti
Pietro Suffritti - Legolas The Elf
http//www.suffritti.it - http//gioco.net/startrek
http//www.treemmecarpi.org - http//www.adigitali.it
|
|
Il 06/02/2017 14:12, Enrico Bianchi ha scritto:
Probabilmente rimuovere il supporto ai TTS mi sembra l'unica soluzione,
a meno che non si trovi un'alternativa valida (e.g. Zanata, Bing o Yandex)
ecco, questa è un'altra idea sensata.
o niente TTS, o TTS locale, per me .
che ne dite voi?
a me è bastata la prima scottata per dire che non mi piace dipendere da
altre robe
…
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIhH_ekL4YDx5p2NW7PON5zBTD4xTqD8ks5rZxw7gaJpZM4FoNtn>.
--
salumi e caci a tutti
Pietro Suffritti - Legolas The Elf
http//www.suffritti.it - http//gioco.net/startrek
http//www.treemmecarpi.org - http//www.adigitali.it
|
LegolasTheElf
left a comment
LegolasTheElf
left a comment
There was a problem hiding this comment.
i don't like anymore TTS services via internet because of the rapid change of their licensing and of their availability.
i propose to evaluate the idea of leaving TTS ONLY as a flag called on the command line, wiping it at all or use a local package adding it as a dependence
5 participants
I got banned by google, so I wrote this patch that seems to me of general interest.
With
--espeakyou can use espeak: the default is still google. You can, however, be more explicit with--google.They only make sense if you use
--audio.I also did some minor refactoring of the audio-related code.