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

Agregar parametro date_end a las spiders. #30

Closed
matiskay opened this issue Aug 16, 2015 · 8 comments
Closed

Agregar parametro date_end a las spiders. #30

matiskay opened this issue Aug 16, 2015 · 8 comments

Comments

@matiskay
Copy link
Contributor

Con el fin de comenzar a testear las spider #24 y para verificar que los datos corresponden a una fecha especifica sugiero agregar una variable extra llamada date_end.

scrapy crawl minedu -a date_start="2015-08-14" -a date_end="2015-08-16"

Esta variable va permitir crawlear paginas espeficas y ademas va permitir testar las spider puesto que los datos de una determinada fecha son invariantes.

@aniversarioperu
Copy link
Member

buena idea

@matiskay
Copy link
Contributor Author

El problema de tener parametros requeridos es que al momento de hacer testing de las spidera estos parametros van a ser requeridos. Sugiero que las spiders tengan una fecha de inicio y fin definida como la fecha donde se corre la spider y los parametros date_start y date_end sean opcionales.

@matiskay
Copy link
Contributor Author

@aniversarioperu, para mi es mas natural usar dia-mes-año que usar año-mes-dia.

@matiskay
Copy link
Contributor Author

Ademas de agregar el parametro end_date hay que agregar una condicional para no permitir que start_date > end_date.

@aniversarioperu
Copy link
Member

creo que no sería bueno hacer hardcoding de las fechas. Tener los parámetros date_start y date_end nos dará bastante flexibilidad.

me parece haber visto una manera de usar los parametros en los tests de las spiders, pero no encuentro donde.

pero la fecha YYYY-MM-DD es del mismo formato del Python date object y es el estilo ISO.

@matiskay
Copy link
Contributor Author

@aniversarioperu, bacan, hay que normalizar todo a YYYY-MM-DD.

@aniversarioperu
Copy link
Member

el end_date puede ser opcional

@matiskay
Copy link
Contributor Author

All the spiders now have the date_end parameter. It's time to close this issue and tackle #49.

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

2 participants