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

Adiciona spiders para Abaré, Adustina, Almadina, Anagé e Andorinha #1137

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

cassiasamp
Copy link
Contributor

@cassiasamp cassiasamp commented Apr 26, 2024

Layout do site publicador de diários oficiais

Marque apenas um dos itens a seguir:

  • O layout não se parece com nenhum caso da lista de layouts padrão
  • É um layout padrão e esta PR adiciona a spider base do padrão ao projeto junto com alguns municípios que fazem parte do padrão.
  • É um layout padrão e todos os municípios adicionados usam a classe de spider base adequada para o padrão.

Código da(s) spider(s)

  • O(s) raspador(es) adicionado(s) tem os atributos de classe exigidos.
  • O(s) raspador(es) adicionado(s) cria(m) objetos do tipo Gazette coletando todos os metadados necessários.
  • O atributo de classe start_date foi preenchido com a data da edição de diário oficial mais antiga disponível no site.
  • Explicitar o atributo de classe end_date não se fez necessário.
  • Não utilizo custom_settings em meu raspador.

Testes

  • Uma coleta-teste da última edição foi feita. O arquivo de .log deste teste está anexado na PR.
  • Uma coleta-teste por intervalo arbitrário foi feita. Os arquivos de .loge .csv deste teste estão anexados na PR.
  • Uma coleta-teste completa foi feita. Os arquivos de .log e .csv deste teste estão anexados na PR.

Verificações

  • Eu experimentei abrir alguns arquivos de diários oficiais coletados pelo meu raspador e verifiquei eles conforme a documentação não encontrando problemas.
  • Eu verifiquei os arquivos .csv gerados pela minha coleta conforme a documentação não encontrando problemas.
  • Eu verifiquei os arquivos de .log gerados pela minha coleta conforme a documentação não encontrando problemas.

Os logs de Andorinha, Almadina, Andustina e Abaré tem a contagem de 1 erro.

Descrição

Terceiro PR com os primeiros 5 municípios da issue #1115. O primeiro PR foi #1134, então #1135 e agora os arquivos estão finalmente corretos.

Arquivos de csv e log gerados:

@cassiasamp
Copy link
Contributor Author

Oi @trevineju, não sei se ficou confuso, mas neste PR deve estar tudo certinho :)

@trevineju
Copy link
Member

oi, @cassiasamp.

Não sei te dizer quando volto aqui. Nós temos mais de 60 PRs abertas, então vamos adicionando aos poucos (de referência, fazemos revisões abertas toda quarta feira a tarde no nosso discord da comunidade)

Além disso, com o que está acontecendo no Rio Grande do Sul, fizemos um esforço de priorizar a adição de municípios de lá.

Mas muito obrigada pela contribuição e pela paciência

@trevineju
Copy link
Member

trevineju commented Jun 12, 2024

Obrigada pela PR e pela paciência, @cassiasamp! Segue a análise dos logs que você anexou na PR:

Abaré

Coleta por período está ok
Coleta completa tem duas edições abandonadas 'item_dropped_count': 2 que verifiquei serem erros no site mesmo (404 e empty-content)

Adustina

Coleta por período está ok
Coleta completa tem duas edições abandonadas 'item_dropped_count': 2 que verifiquei serem erros no site mesmo (ambos empty-content)

Almadina

Coleta por período está ok
Coleta completa tem duas edições abandonadas 'item_dropped_count': 4 que verifiquei serem erros no site mesmo (3 ocorrências de empty-content e 1 ocorrência de 404)

Anagé

Coleta por período está ok
Coleta completa teve 92 erros de sqlite3.IntegrityError que é quando tenta adicionar no banco querido-diario.db uma edição que já existe lá. Como só ocorreu para datas de 2007 e 2008, não ficou claro pra mim se era um erro do site (poderia ser edições repetidas) ou se por ventura a coleta foi executada duas vezes. Testei novamente esse período e o erro não ocorreu.

anage-2007-2008.csv | anage-2007-2008.log

Andorinha

Coleta por período está ok
Coleta completa tem duas edições abandonadas 'item_dropped_count': 4 que verifiquei serem erros no site mesmo (4 ocorrências de empty-content)


Como nenhum desses erros é culpa do raspador, visto que 404 (URL quebrada) e empty-content (arquivo vazio) são problemas do site publicador, tá aprovada a PR!

Obrigada!!

Copy link
Member

@trevineju trevineju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parabéns pela sua primeira contribuição ao Querido Diário!

adicionando importlib a build ao requirements.in

Tive este erro ao instalar os requirements de dev:

ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    importlib-metadata>=4.6 from https://files.pythonhosted.org/packages/2d/0a/679461c511447ffaf176567d5c496d1de27cbe34a87df6677d7171b2fbd4/importlib_metadata-7.1.0-py3-none-any.whl (from build==1.0.3->-r data_collection/requirements-dev.txt (line 48))

e precisei instalar importlib e build.

Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_adustina.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_almadina.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_anage.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_andorinha.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

arquivos corrigidos de adustina, anage e andorinha

mudando o requirements para fazer um pr depois

mudando o requirements para fazer um pr depois

mudando o requirements para fazer um pr depois

mudando o requirements para fazer um pr depois

arrumando arquivos de adustina, anage e andorinha

adicionando raspadores para abare, adustina, almadina, anage e andorinha

adicionando importlib a build ao requirements.in

Tive este erro ao instalar os requirements de dev:

ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    importlib-metadata>=4.6 from https://files.pythonhosted.org/packages/2d/0a/679461c511447ffaf176567d5c496d1de27cbe34a87df6677d7171b2fbd4/importlib_metadata-7.1.0-py3-none-any.whl (from build==1.0.3->-r data_collection/requirements-dev.txt (line 48))

e precisei instalar importlib e build.

Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_adustina.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_almadina.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_anage.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

Update data_collection/gazette/spiders/ba/ba_andorinha.py

Co-authored-by: Juliana Trevine <44185775+trevineju@users.noreply.github.com>
Signed-off-by: Cássia Sampaio <4005687+cassiasamp@users.noreply.github.com>

arquivos corrigidos de adustina, anage e andorinha

mudando o requirements para fazer um pr depois

mudando o requirements para fazer um pr depois

mudando o requirements para fazer um pr depois

mudando o requirements para fazer um pr depois

arrumando arquivos de adustina, anage e andorinha
@trevineju
Copy link
Member

Atualizei a história dessa branch com a main e também resumi os commits para não poluir o histórico.

Resumir os commits sendo: transformar todo o histórico de commits que tinha aqui antes e em apenas um.

Antes:
Screenshot from 2024-06-12 11-12-23

Depois:
6720b18, se você clicar no commit é possível ver o histórico lá, centralizado.

Terceiro PR com os primeiros 5 municípios da issue #1115. O primeiro PR foi #1134, então #1135 e agora os arquivos estão finalmente corretos.

Sei que nessa ocasião não consegui, por conta do RS, mas costumo também ajudar as pessoas com dificuldade com Git/GitHub. Não precisava ter aberto e fechado 2 PRs até chegar nessa 3ª, podíamos ter corrigido lá na primeira PR mesmo. Da próxima vez melhoramos isso! 😃

@trevineju trevineju merged commit d761bb0 into okfn-brasil:main Jun 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants