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

Al maragogi #1290

Merged
merged 2 commits into from
Oct 16, 2024
Merged

Al maragogi #1290

merged 2 commits into from
Oct 16, 2024

Conversation

victorfernandesraton
Copy link
Contributor

@victorfernandesraton victorfernandesraton commented Sep 29, 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.

Descrição

Reimplementação do raspador do município de Maragogi AL , anteriormente citado nesta issue #1176 e feito parcialmente nesta PR #1186, porém o mesmo ficou muito tempo esperando retorno do desenvolvedor, visto que houve mudanças drásticas no site dos diários oficiais em questão, decidi abrir esta nova PR para facilitar o fluxo de revisão

@victorfernandesraton
Copy link
Contributor Author

Criei esta PR com base no que foi feito em #1186
Porém esta está dezatualizada,e devido a mudança nos raspadores eu acredito que revisar do começo faça mais sentido

@victorfernandesraton victorfernandesraton marked this pull request as ready for review October 1, 2024 00:26
@trevineju trevineju linked an issue Oct 2, 2024 that may be closed by this pull request
@trevineju
Copy link
Member

Valeu, @victorfernandesraton! Coloquei na fila de revisão.

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

@victorfernandesraton o raspador tá bem eficiente. Tá ótimo requisitar todas as páginas disponíveis no intervalo de datas.

Fiz alguns comentários de melhoria aqui, qualquer coisa grita aí

data_collection/gazette/spiders/al/al_maragogi.py Outdated Show resolved Hide resolved
data_collection/gazette/spiders/al/al_maragogi.py Outdated Show resolved Hide resolved
data_collection/gazette/spiders/al/al_maragogi.py Outdated Show resolved Hide resolved
data_collection/gazette/spiders/al/al_maragogi.py Outdated Show resolved Hide resolved
data_collection/gazette/spiders/al/al_maragogi.py Outdated Show resolved Hide resolved
@victorfernandesraton victorfernandesraton marked this pull request as draft October 11, 2024 14:48
@victorfernandesraton victorfernandesraton marked this pull request as ready for review October 16, 2024 03:17
@victorfernandesraton
Copy link
Contributor Author

Segue em anexos extração completa com csv e logs
output-all.log
output-all.csv

@victorfernandesraton
Copy link
Contributor Author

Testes de busca entre 04-06-2024 e 02-07-2024
output-06-07.log
output-06-07.csv

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

Comment on lines +53 to +54
url=f"{self.BASE_URL}/busca",
formdata={"page": str(page), **self.__create_params()},
Copy link
Member

Choose a reason for hiding this comment

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

💯

(cherry picked from commit 08919b7)
@ogecece
Copy link
Member

ogecece commented Oct 16, 2024

@victorfernandesraton vou fazer squash pra ficar somente um commit do @gabrielqr e um seu, beleza? Como é apenas um raspador, 12 commits onde a maioria é de correções ficaria excessivo. Normalmente apenas 1 já seria suficiente

@ogecece ogecece merged commit 3eecd1b into okfn-brasil:main Oct 16, 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.

[Manutenção] Novo raspador para Maragogi-AL
4 participants