Mi media center - Foro

Hola,

Os copio lo que me han comunicado como requisitos para publicar la actualización de pelisalacarta en el repositorio oficial, para que lo discutamos aquí.

¿Qué opináis?

Esta es la solicitud:

sorry this took so long, been busy with my dissertation and the
necessary relaxation after it.

a few comments;

1) managing favorites.xml is NOT something an add-on should do - in
particular not using regexp for writing xml (parsing is dirty, writing
is outright a nono. you can do whatever you want to your own files
ofc). this is a deal breaker and there's absolutely no way i'd ever
accept it.

2) don't write to the global userdata folder - use
getAddonInfo("profile") to get to your add-on's writable directory.
for instance there's no guarantee you're running as the master user.

3) why are you using the old runplugin(plugins://video/pelisalacarta
..) ? should be runplugin(plugin.video.pelisalacarta)

4) don't use os.getcwd - use getAddonInfo("path"). os.getcwd will
break soon when we ditch the internal python

maintaining this code must be a major pita - i have spent over 1h just
reviewing the changes...

Y esta mi respuesta

Thanks Arne.

You must understand that this plugin is developed by several people, and each one works with his channels. I have developed the core features and write some of the channels, but each member work with his channels on his way.

The maintenance of the plugin is a distributed work, and it doesn't take a lot of individual work. I mantain the core files, and each of the channels is mantained by its authors. There is code of better or worse quality, depending of the experience of the individual team members, but the whole works fine and we are not very strict about this. The objective is to make online content available, and to be fun in the process.

Now i'll try to reply your comments:

1) This feature can be removed, and if it is a problem we can for sure remove the entire channels using it (casttv, animeforos). Please confirm us if this is ok for you, i ask the channel author for the change.

2) Sorry but i am not sure what do you mean with this. I have tested the plugin and we don't write to the global userdata folder, with the exception of the favourites file mentioned before. We find the addon folder using "xbmc.translatePath("special://profile/addon_data/%s" % PLUGIN_ID)" in config.py, but we will change as you request.

3) I never used runplugin command myself, it must be another team member code, but if it is a problem can be changed.

4) Ok, we'll make the change.

Regards,
Jesús

En general creo que es lógico que a medida que el plugin va teniendo más funcionalidad colisione con cosas que hace xbmc, como los favoritos, los subtítulos, las actualizaciones automáticas...

Creo que es mejor dejar en el repositorio una versión del plugin aunque sea incompleta, para conseguir una mayor "audiencia", y distribuir como hasta ahora la versión completa.

Pero lo discutimos entre todos, y decidimos si seguimos adelante o les decimos que no nos interesa

Actualización:

On Wed, Dec 8, 2010 at 7:56 PM, tvalacarta <tvalacarta@gmail.com> wrote:
> Thanks Arne.
>
> You must understand that this plugin is developed by several people, and
> each one works with his channels. I have developed the core features and
> write some of the channels, but each member work with his channels on his
> way.
>
> The maintenance of the plugin is a distributed work, and it doesn't take a
> lot of individual work. I mantain the core files, and each of the channels
> is mantained by its authors. There is code of better or worse quality,
> depending of the experience of the individual team members, but the whole
> works fine and we are not very strict about this. The objective is to make
> online content available, and to be fun in the process.

sure, it was a comment made in sympathy, not in spite

> Now i'll try to reply your comments:
>
> 1) This feature can be removed, and if it is a problem we can for sure
> remove the entire channels using it (casttv, animeforos). Please confirm us
> if this is ok for you, i ask the channel author for the change.

it's just the favorites.xml writing i don't like. problem is that we
cannot have random bits of code writing our files (we have in-memory
copies, changes may get lost if we flush these to disc etc). i'm more
than willing to discuss an interface to expose the requested
functionality, but it has to be just that - an interface. removing the
channels is not necessary, just the code touching our naught bits

> 2) Sorry but i am not sure what do you mean with this. I have tested the
> plugin and we don't write to the global userdata folder, with the exception
> of the favourites file mentioned before. We find the addon folder using
> "xbmc.translatePath("special://profile/addon_data/%s" % PLUGIN_ID)" in
> config.py, but we will change as you request.

this was related to 1) - just a general never-do-this. btw, you can
use getAddonInfo('profile') for a shorter version of what you do.

> 3) I never used runplugin command myself, it must be another team member
> code, but if it is a problem can be changed.

actually, this was also in the favorite management code.

> 4) Ok, we'll make the change.

thanks.

arnem

Jesús, para terminar pronto y rápido, yo estoy completamente de acuerdo en que elimines de la versión que envias al repositorio CastTV y AnimeForos, y si tienes que eliminarlo del resto de versiones lo haces y punto.

EDITO: Para que nadie se lie y quede claro, los tres primeros puntos que te comenta, vienen únicamente por la nueva función para administrar algunos de los contenidos añadidos al Favoritos de XBMC, que agregué en CastTV y que también usa AnimeForos, sólo accesible desde el apartado "Todos mis Favoritos" cuando existe la ruta que yo he especificado y el archivo, cosa que seguramente no sucede en Dharma...

Comprendo la cuestión, aunque a mi juicio es un tema muy puntual que sólo afecta a los propios usuarios (porque favourites.xml es un archivo de usuario ¿o me equivoco?) que usen el apartado "Todos Mis Favoritos" de CastTV y AnimeForos (cuatro gatos que seguro agradeden la función) , y de hecho yo sólo tengo claro que funcione en XBMC4XBOX, si no existe la ruta o el archivo no se muestra el apartado, lo más probable es que en Dharma no se muestre nada y no se pueda usar.... En cuanto a lo de runplugin yo tampoco lo uso sólo aparece en CastTV dentro de un patrón para favourites.xml, según el formato que yo tengo en XBMC4XBOX.

Pues eso que elimines mis canales de la versión del repositorio, y de todos los sitios donde tengas que quitarlos, ya más adelante miraré si cambio algo o si subo una versión para repositorio y otra full si tu me dejas, o lo que sea.

Siento los inconvenientes que te haya podido causar

P.D.: Yo ni puedo usar Dharma ni no Dharma ni nada, no tengo ningún ordenador donde pueda instalar algún media center de los actuales, con lo que podrás comprender mi pasotismo total respecto al tema...

Lily, y en donde programaste el código para esos canáles, en linux en la xbox o en el móvil? ;-D

En cuanto a tus disculpas, esperando que Jesús me permita el exceso de confianza, pues aún no he hecho ni un commit ;-D, no creo que sean necesarias, estoy seguro, por sus comentarios (aún no os he tratado mucho), que está de acuerdo conmigo.

Como ha remarcado Jesus, cada uno ha puesto su trozo de código y sus horas, remarco de nuevo, yo aún no, para desarrollar uno de los addons más útiles, y estoy seguro que, más extendido en la comunidad xbmc, por lo menos en España. Cometeremos errores, y colaboraremos en la evolución del propio xbmc, por ejemplo utilizando ese maravilloso interface que comenta para utilizar los favoritos, aunque tengo que revisar mejor a que parte se refiere, y aún me queda mucho por entender en el addon.

Personalmente, no creo que interese mantener una doble versión oficial y no oficial. Veo más interesante el colaborar con el grupo XBMC, aunque les tendríamos que solicitar un poco de confianza y flexibilidad.

Si por ejemplo dicen que habrá un interface para atacar al tema de los favoritos, pues perfecto, y en cuanto esté nos moveremos para utilizarlo, e incluso lo podemos programar pensando en que existen dichos métodos para que vean la buena intención y en el momento que las tengan las sustituimos por las llamadas a las que marquen ellos, pero mientras que nos permitan que se vaya actualizando la versión, que hay muchos usuarios noveles que utilizan distribuciones como la xbmc-live o la de xbmcfreaks, y no quieren hacer, como dice un amigo mío: sudos, cosudos y tangentes para poder actualizar el plugin.

Jesús, como te comenté sería interesante analizar todos los canales e intentar crear algún tipo de clase abstracta para evitar como dice el controlador aéreo de xbmc ;-D, que tengan que dedicar 1 hora en revisar el código. No será fácil seguro, pero creo que vale la pena. Por cierto la última integración con foros ya es la leche, enhorabuena.

dreamlomax escribió:Lily, y en donde programaste el código para esos canáles, en linux en la xbox o en el móvil? ;-D

¿Con esto me quieres decir algo o es que eres menor de edad?, porque yo tengo 39 años y ¿tu crees que yo puedo aceptar ese tipo de indirectas o gracias, y después del rollo que te echas a continuación?, pues no.

Pero para tu entretenimiento y chanza, el código lo escribí en el bloc de notas de Windows XP, de un portatil de mi generación. Espero haber respondido adecuademente a tu pregunta.

Saluditos

Lily,
te pido disculpas por el mal entendido. Si soy mayor de edad, 32.

Cuando hice el comentario de desde donde programaste los canales, ni mucho menos era por la calidad, es más aún no he visto como están programados los canales. Lo dije por que como decías que no tenías ordenador me dejó impresionado que hubieras podido programar los canales.

Si te fijas en mis comentarios solo dejo caer alagos para todos, y por eso digo que no es necesario pedir disculpas de ningún tipo, al contrario, estaros agradecidos por vuestro trabajo.

De todos modos, si la broma de donde lo habías programado te ha afectado de algún modo, lo siento. Si quieres Edito el mensaje y lo elimino.

No @Lily, no te equivoques. No quiero quitar tus canales, se lo propuse como opción a ellos para saber exactamente qué querían.

Creo que son dos canales muy completos, yo uso sobre todo animeforos y en menor medida casttv aunque no uso la parte de favoritos.

Y no me supone ningún problema. Si estamos en el repo oficial de xbmc mejor, pero si no estamos tampoco vamos a llorar

Su punto de vista es que si quieres que tu addon forme parte de la distribución oficial tienes que seguir unas reglas, que aún no están escritas porque entre otras cosas también ellos son nuevos en esto. Eso me parece lógico por su parte, por eso trato de "negociar" con ellos la fórmula que nos guste a todos.

Creo que lo mejor es que saquemos esa opción de favoritos como un fichero .py aparte, y luego simplemente que ese fichero no esté en la versión oficial para Dharma. Es lo que se hace ahora por ejemplo con updater.py, donde está la actualización automática de canales y del propio plugin.

Si te parece puedo echarle un vistazo al código, pienso cómo hacerlo y lo vemos.

EDITO: Releyendo el post me he olvidado comentar lo del "runplugin". La verdad es que no se lo que quiere decir, pero creo que es fácil cambiarlo

@dreamlomax, estoy más o menos de acuerdo contigo con algunos matices

Ya hay una doble versión, que deberíamos llamar más bien "oficial" y "capada-para-xbmc-dharma", y no me apetece perder ninguna de las funcionalidades que al XBMC Team no les gusta.

En cuanto a lo de la estructura "abstracta", no quiero complicar en exceso el trabajo necesario para realizar un canal. La versión 1.X de los plugins corría sobre un framework llamado XOT-UZG, un script para XBMC que hacía algo parecido con una superclase y callbacks, y te aseguro que era mucho más complicado. Prefiero un modelo con menos arquitectura en el canal: aquí va la expresión regular, aquí va la navegación, y poco más. Pero que además te deje montarte cosas realmente complicadas si quieres, como los canales yonkis por ejemplo...

Ok,
ya habrás visto los 3 canales que he subido, aunque uno de ellos se podrá eliminar por que en realidad fue el primero que utilicé para hacer la primera prueba y me parece que en realidad no tendrá actualización de contenidos.

Saludos

Por poner un granito de arena.

Ya sabéis que yo siempre abogo por no trabajar en las áreas en las que hay solapamiento con funciones estándar de XBMC (aka Subtítulos, Favoritos, etc), pero como a vosotros os gustan pues no tengo problemas.

Pero lo que sería discutible (con el TeamXBMC) es la negativa a permitir modificar el favorites. Como bien dice jesús, lo que se puede o no hacer va a establecerse poco a poco según se ande el camino. Pero para mi es claro que favoritos es un área donde se podrían crear muchos plugins para gestionar... esa regla impediría de facto scripts de ese tipo y no creo que debiera ser necesario.

Quizá en un futuro establezcan funciones para interactuar de forma segura con los distintos apartados de xbmc. ej: favoritos, biblioteca, player, etc... Mientras tanto es cuestión de negociar.

Evidentemente la solución radical es quitar todo y evitar los problemas...

El hecho de tener 2 versiones distintas a mi me parece una pesadilla, pero como lo gestionas tu jesus, pues adelante

Yo, como habéis comprobado, me he mantenido muy distante en los últimos meses. Principalmente por trabajo, pero también porque las ideas que tengo de como trabajar en un plugin de streaming/descarga de series o películas no encaja en la filosofía actual de pelisalacarta.

Estoy creando un marco "distinto", con otra filosofía, para un script de series que llamo SeriesAlacarta y que, si jesus quiere, publicaremos por primera vez aquí.

En los comienzos el contenido se limitará a 1 o 2 canales pero estoy poniendo mucho esfuerzo en plantear un marco más estricto que permita integrar mas canales y servidores con seguiridad. Tambien integra la gestión de descargar/streaming de multirars, integración en la biblioteca xbmc e integración con sickbeard (un programa externo a xbmc que gestiona la busqueda y descarga de episodios nuevos de nuestras series favoritas).

Tengo libre mucho tiempo desde ahora hasta finales de enero... os iré contando.