search

mx-psi / fa-scraper

A FilmAffinity web scraper compatible with Letterboxd
GNU General Public License v3.0
18 stars 9 forks source link

Error al crear fichero csv #122

Closed adriancm92 closed 3 months ago

adriancm92 commented 4 months ago

Parece que el error es parecido al aquí ya solucionado #66

Lanzando esta vez el comando: fa-scraper --csv films.csv 834871

mx-psi commented 4 months ago

Hey, gracias por reportar el error! :) También usas Windows? el error es exactamente el mismo que en #66?

adriancm92 commented 4 months ago

Si, es también en Windows, el resultado es este concretamente: fa-scraper --csv films.csv 834871

Traceback (most recent call last): File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main return _run_code(code, main_globals, None, File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code exec(code, run_globals) File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python310\Scripts\fa-scraper.exe__main__.py", line 7, in File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python310\lib\site-packages\fa_scraper\cli.py", line 78, in main save_to_csv(data, fieldnames, export_file) File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python310\lib\site-packages\fa_scraper\fa_scraper.py", line 217, in save_to_csv for d in dicts: File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python310\lib\site-packages\fa_scraper\fa_scraper.py", line 149, in get_profile_data "Year": title.next_sibling.strip()[1:-1], TypeError: 'NoneType' object is not callable

adriancm92 commented 4 months ago

Has podido reproducir el error o te falta algún dato mas?

mx-psi commented 4 months ago

@adriancm92 no tendré tiempo hasta el finde al menos para echarle un vistazo a esto. Sí puedo decirte que el error parece diferente, probablemente alguna de las películas que tienes no tiene el formato que espera la herramienta (bien porque nunca había encontrado ese caso, o bien porque FilmAffinity ha cambiado algo en su interfaz)

adriancm92 commented 4 months ago

Genial @mx-psi, si puedo ayudarte en algo para seguir la traza estaria encantado

adriancm92 commented 3 months ago

Alguna novedad @mx-psi ?

ArnewGonz commented 3 months ago

He encontrado que parece que, al menos en PC, parece que el nuevo diseño ha movido como obtener el año. He creado PR con la solución que he encontrado: https://github.com/mx-psi/fa-scraper/pull/123

adriancm92 commented 3 months ago

Gracias @ArnewGonz veamos si valida los cambios @mx-psi y pruebo a ver si realiza la exportación correctamente

mx-psi commented 3 months ago

@adriancm92 Si quieres puedes ayudarme a validarlo probándolo en tu caso. Si tienes instalado gh es cuestión de hacer lo siguiente (con bash, creo que debería funcionar en PowerShell también igual quitando los comentarios):

gh repo clone mx-psi/fa-scraper  # Clona el repositorio
cd fa-scraper   # Cambia de directorio al repositorio que has clonado
gh pr checkout 123   # Copia el código de @ArnewGonz 
pip install .  # Instala la copia local

Con eso deberías tener fa-scraper instalado con los cambios de @ArnewGonz y me ayudas a testear que funciona en tu caso :)

adriancm92 commented 3 months ago

Si, ahora funciona perfecto! @mx-psi

mx-psi commented 3 months ago

Perfecto, muchas gracias por comprobarlo :) He hecho merge de #123 y lo incluiré en la próxima release

adriancm92 commented 3 months ago

Genial! Como apunte a tener en cuenta (no si lo has visto ya), los nombres de los encabezados para importarlo a Letterboxd han cambiado. https://letterboxd.com/about/importing-data/

mx-psi commented 3 months ago

Genial! Como apunte a tener en cuenta (no si lo has visto ya), los nombres de los encabezados para importarlo a Letterboxd han cambiado. letterboxd.com/about/importing-data

Uh, no lo había visto, gracias. He creado #124 para trackear esto