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

Error al crear fichero csv #122

Closed
adriancm92 opened this issue Apr 10, 2024 · 13 comments
Closed

Error al crear fichero csv #122

adriancm92 opened this issue Apr 10, 2024 · 13 comments

Comments

@adriancm92
Copy link

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

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

@mx-psi
Copy link
Owner

mx-psi commented Apr 10, 2024

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

@adriancm92
Copy link
Author

adriancm92 commented Apr 10, 2024

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
Copy link
Author

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

@mx-psi
Copy link
Owner

mx-psi commented Apr 12, 2024

@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
Copy link
Author

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

@adriancm92
Copy link
Author

Alguna novedad @mx-psi ?

@ArnewGonz
Copy link
Contributor

ArnewGonz commented Apr 28, 2024

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: #123

@adriancm92
Copy link
Author

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

@mx-psi
Copy link
Owner

mx-psi commented May 2, 2024

@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
Copy link
Author

Si, ahora funciona perfecto! @mx-psi

@mx-psi
Copy link
Owner

mx-psi commented May 3, 2024

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

@mx-psi mx-psi closed this as completed May 3, 2024
@adriancm92
Copy link
Author

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
Copy link
Owner

mx-psi commented May 3, 2024

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

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

No branches or pull requests

3 participants