maincenter / Logotipos-TV

Logotipos TV de España, la resolución es de 384x384 para su colocación en programas tipo TvHeadend de manera automática.
19 stars 29 forks source link

Ruta a los logos se añade con espacios en lugar de guiones #7

Open Sockolet opened 6 years ago

Sockolet commented 6 years ago

Hola, he añadido tu genial repositorio a mi servidor tvheadend, pero he notado que sólo muestra algunos logos (muy pocos). Revisando la configuración, veo que al hacer el mapeo de las rutas de los logos, estas añaden espacios (%20) en lugar de guiones y por ello, todos los logos de canales con nombres de más de una palabra, no se muestran.

¿Existe una manera de añadir las rutas correctas automáticamente o debo hacerlo a mano?

Adjunto una fotografía para que lo veas.

PS: utilizo Tvheadend 4.2.4-15~g1510fa0 en Ubuntu server 16.04 64bit. captura de pantalla de 2017-11-17 10-31-28

KellerNET commented 6 years ago

Hola, si, existe una forma automáticamente aun que faltan logos que tengo preparados para ir agregando pero que por falta de tiempo no lo he podido hacer. En el tutorial de como añadir este repositorio indica que opciones seleccionar para que te aparezca en lugar de los símbolos % un "_". Busca en la parte inferior de este link donde dice INSTALACIÓN https://github.com/maincenter/Logotipos-TV Si aún así sigues teniendo problemas, no dudes en contactar para intentar buscar una solución. Un saludo y gracias por el apoyo, suerte.

Sockolet commented 6 years ago

Ostras pues lo he hecho tal cual indica el tutorial y los vuelve a generar con espacios o.O

Gracias por la ayuda de todas formas, me tocará editarlos a mano.

KellerNET commented 6 years ago

De todos modos, también en su día creé un vídeo por que suele ser mas cómodo ver que leer. Es muy importante que tras configurar regeneres las listas, lo que viene siendo, que elimines las URL de los logos de los canales y que las vuelva a generar, en el vídeo a partir del minuto 6:30 puedes ver como funciona. https://youtu.be/bbZBz3ltues

Sockolet commented 6 years ago

He seguido todos los pasos, he visto el vídeo, lo he hecho tres veces, y las rutas siguen mostrándose con espacios. Las he modificado a mano y ahora ya se muestran correctamente. Pero repito, no he logrado que se añadieran las rutas de forma adecuada automáticamente.

Gracias por la ayuda y por el estupendo trabajo.

KellerNET commented 6 years ago

Buenas, pues no entiendo que te puede estar pasando, si quieres que le echemos un vistazo, dime un día y una hora y quedamos bien vía teamspeak y TeamViewer para poder ver que te ocurre he intentar solventar el problema. Un saludo y gracias por el apoyo y por el interés. Un saludo.

jesuslop commented 6 years ago

Buenas, a mí me pasa lo mismo que a Meiden, lo que pasa es que hay un bug en tvheadend que por mucho que le indiques %C en la ruta del picon, se la refanfinfla y los sigue buscando url-encodeados con sus %20. La solución sería si tuvieras la gentileza de hacer otra carpeta y tener otro juego entero de los mismos iconos pero con nombres de fichero url-encodeados, básicamente cambiando los subrayados por %20 se arreglaría todo. Yo ando con tvheadend 4.2.4-9~g0771e34. Por ejemplo me busca master/main/transparentes/CANAL%207%20TV%20%20%20%2021.png master/main/transparentes/Cuatro%20HD.png etc.

KellerNET commented 6 years ago

Buenas, no es una cuestión de cambiarlo por %20 ya que a la hora de publicar, habría que subirlos dos veces y no solo eso, el %20 se sustituye por un espacio, a lo que muchos canales cogen los nombres de los canales de forma automática y por consiguiente cuando pusiese una letra mayúsculas o un "_" tendrías el mismo problema, te lo comento por que empecé haciendolo en su día así y había muchas incompatibilidades, muchos errores y muchos logos que no cargaban bien, por lo que la opción mas razonable fue esta y ya que estáis utilizando versiones beta o alfa, os aconsejo iros a una mas estable y que sea compatible, eso que os pasa es un bug y será reparado, trabajar extra para dar soporte a un bug no tiene ningún sentido, Intentar no actualizar tan a menudo a no ser que os proporcione una gran mejora, no lo veo razonable. Yo estoy utilizando actualmente la versión: HTS Tvheadend 4.1-2409 para Debian 7.x x64 y esta versión está disponible para Debian 8 y similares. Lamento dar esta solución pero no es razonable esta petición. Saludos.

jesuslop commented 6 years ago

Entiendo, no es nada guapo como solución, veré lo del downgrade. Mientras gracias por el trabajo igual y un saludo.

KellerNET commented 6 years ago

Siento darte esa solución pero lo otro no resolvería el problema tampoco ya que muchos canales la primera letra aparece con Mayúsculas y por tanto si ponemos %c en vez de la actual %C no cargarían los logos, como te comentaba, ya lo cambié en su día por que yo tenía ese mismo problema y aun que había unos que si los pillaba, otros muchos no. Un saludo y gracias a ti por tomarte las molestias de notificar el problema con esas versiones de TVH :-)

alejandro-amo commented 5 years ago

Hola @KellerNET KellerNET

comentarte que el "bug" que comentas en la gestión de los iconos de TVH, no es tal, sino una decisión de desarrollo para ser fieles al nombre del programa. Si tiene espacios, tiene espacios.

Para los que quieran mantener cierta compatibilidad en linux, les sugiero en primer lugar clonar este repositorio a un directorio local de su máquina, y una vez clonado, pasarle un script sencillito usando las herramientas GNU find, sed y mvpara cambiar los underscores por espacios sistemáticamente (solo fallaría en canales que usan el guion bajo, pero eso es poco probable que suceda)

saludos

alejandro-amo commented 5 years ago

os dejo el comando que he usado yo para compatibilizar este repositorio con la estructura actual de TVH:

for archivo in `find . -name *.png`; do mv "$archivo" "`echo $archivo | sed 's/\_/ /g'`"; done

los canales sin guines bajos daran un mensaje de error inofensivo:

mv: './development/SAT/MoviStar+/Movistar+.png' y './development/SAT/MoviStar+/Movistar+.png' son el mismo fichero
mv: './development/SAT/MoviStar+/ETB1.png' y './development/SAT/MoviStar+/ETB1.png' son el mismo fichero
mv: './development/SAT/MoviStar+/Ten.png' y './development/SAT/MoviStar+/Ten.png' son el mismo fichero
(...)

pero en unos segundos la estructura de ficheros será justo la que TVH espera:

hts@retropie:~/tvicons$ find . -name "* *" # mostrar ficheros que tienen al menos un espacio en el nombre
./main/transparentes/TVE 1 HD.png
./main/transparentes/MTV Espana.png
./main/transparentes/7 CYLTV HD.png
./main/transparentes/FOX life HD.png
./main/transparentes/8 La 8 Burgos.png
./main/transparentes/HIT FM.png
./main/transparentes/AXN HD.png
./main/transparentes/DEPORTES HD.png
./main/transparentes/Movistar eSports.png
./main/transparentes/Non Stop People HD.png
./main/transparentes/TVE 2.png
./main/transparentes/AMC HD.png
./main/transparentes/Radio Nacional Pais Vasco.png
./main/transparentes/Movistar+ HD.png
./main/transparentes/Futbol Replay.png
./main/transparentes/COPE La Rioja.png
./main/transparentes/EuroSport 2 HD.png
./main/transparentes/Castilla y Leon 7.png
./main/transparentes/Hollywood HD .png
./main/transparentes/Disney Channel.png
./main/transparentes/Disney Channel HD.png
./main/transparentes/La Sexta.png
./main/transparentes/Odisea HD.png
./main/transparentes/MELODIA FM.png
./main/transparentes/ETB 2.png
./main/transparentes/Non Stop People.png
./main/transparentes/Calle 13.png
(...)
KellerNET commented 5 years ago

Hola @KellerNET KellerNET

comentarte que el "bug" que comentas en la gestión de los iconos de TVH, no es tal, sino una decisión de desarrollo para ser fieles al nombre del programa. Si tiene espacios, tiene espacios.

Para los que quieran mantener cierta compatibilidad en linux, les sugiero en primer lugar clonar este repositorio a un directorio local de su máquina, y una vez clonado, pasarle un script sencillito usando las herramientas GNU find, sed y mvpara cambiar los underscores por espacios sistemáticamente (solo fallaría en canales que usan el guion bajo, pero eso es poco probable que suceda)

saludos

Hola, Desconozco a quién va dirigido este mensaje, si es a mi contestación, no entiendo a que te refieres.

El repositorio está bien creado y tal cual la sintonizadora encuentra los canales los renombra tal cual, por lo que el repositorio se basa en los nombres asignados oficialmente y por consiguiente no fallan salvo usuarios que tienen una codificación extraña o cambiada. De todos modos no hay mucha gente que participe por lo que las cosas sin participación poco se pueden mejorar, así que me lo hago para mi y si a alguien no le funciona bien de una determinada manera, que lo detalle bien he intentaremos buscar solución. También estoy pendiente de terminar otras cuestiones que tras su finalización intentaré duplicar este repo, otro problema que tiene TVH es que si tienes mas de una sintonizadora, DVB-T, DVB-S o IPTV, solo te permite meter un link de los picons, por lo que todo a de estar de una determinada manera y es complicado satisfacer a todos los usuarios, como ya digo sin conocer exactamente como lo tienen. Saludos.

alejandro-amo commented 5 years ago

No te preocupes, si te funciona para ti, estupendo, y con los comentarios y explicaciones tecnicas que he dado, resuelvo el problema de otras perosnas que han comentado (y de las que llegaran aqui mas tarde, como es mi caso :) ), aunque tu manera de expresar la condición natural del uso de "%C" en TVH no sea correcta y mi intento por explicar una decisión de desarrollo que afecta nada menos que a todos los sistemas *NIX quizas no ha sido eficaz.

En todo caso, como digo, con ese simple comando se puede compatibilizar este repo con un tvheadend actualizado que se ejecuta, por ejemplo, en mi raspberry Pi con la configuración de fábrica y sin "codificaciones raras", como tú lo llamas :)

alejandro-amo commented 5 years ago

Para el que tenga dudas de dónde procede la incoherencia, si del comportamiento del software o de la documentación, la respuesta es "de la documentación".

Clarkstone (manager de lproyecto y encargado de la documentacion) lo explicó hace un año aqui: https://tvheadend.org/issues/2967#note-10 y aunque dijo que la documentación se tenia que actualzar para quitar la expresión "no spaces", y que se habia decidido tomar literalmente los espacios (sin convertir), la verdad es que la documentación sigue obsoleta un año despues, quizas porque en ese proyecto como en todos, hace falta que se les eche una mano :D http://docs.tvheadend.org/webui/config_misc/

Esto explica, resuelve, y desmitifica el problema de @Meiden-fc :D