Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

actualización a v1.5.2 #28

Merged
merged 13 commits into from Dec 12, 2012
Merged

actualización a v1.5.2 #28

merged 13 commits into from Dec 12, 2012

Conversation

pancheto
Copy link
Collaborator

si bien la búsqueda en FA (o a través de google) no se puede mejorar, ya que usa las capacidades de FA (o google) y esas son las que son, la búsqueda de contenido externo a FA basado en el IMDBid sí puede trabajarse un poco a raíz de los recientes cambios en el sistema de búsquedas de IMDB. he actualizado pues la búsqueda del IMDBid para ser lo más óptima posible, a la par que rápida y flexible, como se explica en el commit de filmaffinity.xml

desconozco la manera de eliminar ficheros a través de github, así que transmito aquí lo que el usuario davidmuma comentó en XBMCSPAIN: salvo los ficheros init.py y log.py dentro de lib/utils, todos los demás de ese directorio son prescindibles y por tanto eliminables.

recientes cambios de IMDB obligaron a actualizar la búsqueda del IMDBid en IMDB. aprovecho para usar la información AKA para enriquecer dicha búsqueda.
traer v1.5.1 a patch-2 para crear v1.5.2
actualizada la búsqueda del IMDBid para conseguir información de fuentes externas a FilmAffinity. ahora se hace a través de http://www.imdb.com/xml/find?xml=1&nr=1&tt=on&q= sin incluir el año, para luego rebuscar en los resultados usando el año y/o el nombre del director/es sin iniciales, lo cual permite una mayor flexibilidad a la hora de dar con el resultado óptimo.
corregida la expresión "aún más a la vista"
actualizados los cambios v1.5.1 al español y añadidos los cambios v1.5.2
actualizado número de versión a 1.5.2 y cambiado "Lechu" por "pancheto" para atender a peticiones en foros
actualizada la búsqueda del IMDBid para conseguir información de fuentes externas a FilmAffinity. ahora se hace a través de http://www.imdb.com/xml/find?xml=1&nr=1&tt=on&q= sin incluir el año, para luego rebuscar en los resultados usando el año y/o el nombre del director/es sin iniciales, lo cual permite una mayor flexibilidad a la hora de dar con el resultado óptimo.

también he añadido movido bloques de código para facilitar la comprensión del scraper, así como unos cambios cosméticos de código.
actualizada la búsqueda del IMDBid para conseguir información de fuentes externas a FilmAffinity. ahora se hace a través de http://www.imdb.com/xml/find?xml=1&nr=1&tt=on&q= sin incluir el año, para luego rebuscar en los resultados usando el año y/o el nombre del director/es sin iniciales, lo cual permite una mayor flexibilidad a la hora de dar con el resultado óptimo.

también he añadido movido bloques de código para facilitar la comprensión del scraper, así como unos cambios cosméticos sin funcionalidad a lo largo de todo el scraper para mejorar su robustez.
@MaDDoGo
Copy link
Owner

MaDDoGo commented Dec 11, 2012

Perfecto, cuando llegue de trabajar me lo miro y acepto el commit. Vaya
currazos que te pegas tío, que pasada :)

2012/12/11 Lechu notifications@github.com

si bien la búsqueda en FA (o a través de google) no se puede mejorar, ya
que usa las capacidades de FA (o google) y esas son las que son, la
búsqueda de contenido externo a FA basado en el IMDBid sí puede trabajarse
un poco a raíz de los recientes cambios en el sistema de búsquedas de IMDB.
he actualizado pues la búsqueda del IMDBid para ser lo más óptima posible,
a la par que rápida y flexible, como se explica en el commit de
filmaffinity.xml

desconozco la manera de eliminar ficheros a través de github, así que
transmito aquí lo que el usuario davidmuma comentó en XBMCSPAIN: salvo los
ficheros init.py y log.py dentro de lib/utils, todos los demás de ese

directorio son prescindibles y por tanto eliminables.

You can merge this Pull Request by running:

git pull https://github.com/pancheto/metadata.filmaffinity.com master

Or view, comment on, or merge it at:

#28
Commit Summary

File Changes

  • M GoogleAdvancedSearch.txt (8)
  • M addon.xml (4)
  • M changelog.txt (6)
  • M filmaffinity.xml (231)

Patch Links

Enric Soler Rastrollo

@pancheto
Copy link
Collaborator Author

lo cierto es que quería esperar a que la gente de XBMCSPAIN lo probara, pero ya que me funciona perfectamente en una librería bastante amplia y que no han respondido los más activos, sumado a que el resto de la semana estaré bastante liado, preferí dejarlo resuelto ya ahora. en caso de que se encuentre cualquier mejora (gazapos creo haberlos resuelto todos ya yo con mi propia librería, que revisé en numerosas ocasiones para desgracia de FA, que volvió a banearme la IP de nuevo en el proceso) ya sacaremos una nueva versión. desde luego esta v1.5.2 es mucho mejor que la v1.5.1, así que la actualización ya en sí misma merece no esperar mucho más.

en cuanto a todos esos cambios que constan en github, olvídate de mis propios "pull requests" y de alguna entrada más en ese listado de cambios. en mi camino de comprender github he intentado eliminar ramas que tenía abiertas inútilmente, y a lo mejor he liado las cosas más de lo necesario. quédate con que al final conseguí traerme tu máster a mi máster, editarlo todo a piñón con copy-paste, y hacer el pull request de mi máster a tu máster. todos los archivos están perfectamente actualizados, y lo único que hay que hacer a mayores es eliminar todo lo de lib/utils que no sean esos dos archivos .py mencionados.

nada más. y nada menos. un saludo.

-------- Original Message --------
Subject: Re: [metadata.filmaffinity.com] actualización a v1.5.2 (#28)
From: Enric Soler Rastrollo
To: MaDDoGo/metadata.filmaffinity.com
Cc: Lechu
Date: 11/12/2012 12:26

Perfecto, cuando llegue de trabajar me lo miro y acepto el commit. Vaya
currazos que te pegas tío, que pasada :)

2012/12/11 Lechu notifications@github.com

si bien la búsqueda en FA (o a través de google) no se puede mejorar, ya
que usa las capacidades de FA (o google) y esas son las que son, la
búsqueda de contenido externo a FA basado en el IMDBid sí puede trabajarse
un poco a raíz de los recientes cambios en el sistema de búsquedas de IMDB.
he actualizado pues la búsqueda del IMDBid para ser lo más óptima posible,
a la par que rápida y flexible, como se explica en el commit de
filmaffinity.xml

desconozco la manera de eliminar ficheros a través de github, así que
transmito aquí lo que el usuario davidmuma comentó en XBMCSPAIN: salvo los
ficheros init.py y log.py dentro de lib/utils, todos los demás de ese

directorio son prescindibles y por tanto eliminables.

You can merge this Pull Request by running:

git pull https://github.com/pancheto/metadata.filmaffinity.com master

Or view, comment on, or merge it at:

#28
Commit Summary

File Changes

  • M GoogleAdvancedSearch.txt (8)
  • M addon.xml (4)
  • M changelog.txt (6)
  • M filmaffinity.xml (231)

Patch Links


Reply to this email directly or view it on GitHubhttps://github.com//pull/28.

Enric Soler Rastrollo


Reply to this email directly or view it on GitHub #28 (comment).

Jorge Amigo Lechuga

web: http://www.xente.mundo-r.com/lechu/

con tanto copiar-pegar de distintas versiones de pruebas se me pasó un detalle: en la línea 178 la expresión a buscar es (\S+) y no (\w+)
MaDDoGo added a commit that referenced this pull request Dec 12, 2012
@MaDDoGo MaDDoGo merged commit 770888c into MaDDoGo:master Dec 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants