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

[WIP] Refatora recapturas de GPS do SPPO #168

Closed
wants to merge 62 commits into from
Closed

Conversation

Hellcassius
Copy link
Contributor

@Hellcassius Hellcassius commented Aug 17, 2022

(1) Refatoração do flow captura_sppo_v2: Ajustes para uso da captura_sppo_v2 na recaptura *

Changelog:

  • adição de 2 parâmetros opcionais de recaptura - bool indicando se é recaptura + erro que gerou a recaptura
  • modificação do parametro timestamp - quando há recaptura, utiliza parametro opcional datetime_filter, cc usa timestamp atual; fixa formatacao em isoformat
  • upload_logs_to_bq: mantém log do erro anterior que foi recapturado
  • run_config: limita uso de cpu e memoria

(2) Refatoração do flow recaptura: Ajustes para uso da captura_sppo_v2 na recaptura

Changelog:

  • create_flow_run: usa o flow de captura_sppo_v2 para criar múltiplas runs do mesmo, substituindo o uso do método .map em cada task
  • query_logs: aumenta limite de 40 para 60 recapturas por run; retorna dicionário com timestamp e erro + flag de recaptura, novos parametros que são passados no create_flow_run para a captura_sppo_v2
  • run_config: limita uso de cpu e memoria

(3) Refatoração do flow materialize: Corrije atraso na materialização quando há falhas

Changelog:

  • get_materialization_date_range: parametriza o delay da materialização de acordo com o flow + varia start_ts para preencher buracos de materialização (caso haja algum), fixando end_ts com base no delay_hour

  • Detalhamento do flow de captura:

image

@github-actions
Copy link

===> 1 warning(s) found:

  • warning [W0611] Unused log_critical imported from pipelines.rj_smtr.utils (pipelines/rj_smtr/tasks.py:24:0)

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @Hellcassius
- @fernandascovino

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/utils/dump_url/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/init.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @d116626
- @fernandascovino
- @Hellcassius
- @gabriel-milan

@github-actions
Copy link

===> 1 warning(s) found:

  • warning [W0611] Unused log_critical imported from pipelines.rj_smtr.utils (pipelines/rj_smtr/tasks.py:24:0)

1 similar comment
@github-actions
Copy link

===> 1 warning(s) found:

  • warning [W0611] Unused log_critical imported from pipelines.rj_smtr.utils (pipelines/rj_smtr/tasks.py:24:0)

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/utils/dump_url/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @d116626
- @Hellcassius
- @gabriel-milan

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/utils/dump_url/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius
- @gabriel-milan
- @d116626

@github-actions
Copy link

===> 1 warning(s) found:

  • warning [W0611] Unused log_critical imported from pipelines.rj_smtr.utils (pipelines/rj_smtr/tasks.py:24:0)

@github-actions
Copy link

===> 1 warning(s) found:

  • warning [W0611] Unused log_critical imported from pipelines.rj_smtr.utils (pipelines/rj_smtr/tasks.py:24:0)

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/utils/dump_url/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @Hellcassius
- @d116626
- @fernandascovino
- @gabriel-milan

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/utils/dump_url/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @d116626
- @fernandascovino
- @gabriel-milan
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/utils/dump_url/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @d116626
- @gabriel-milan
- @Hellcassius
- @fernandascovino

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @Hellcassius
- @fernandascovino

@@ -83,6 +85,9 @@ def pre_treatment_br_rj_riodejaneiro_onibus_gps(
timezone = constants.TIMEZONE.value

log(f"Data received to treat: \n{status['data'][:5]}")
if status["data"] == []:
Copy link
Collaborator

Choose a reason for hiding this comment

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

remover, ja esta na linha 80

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/tasks.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @Hellcassius
- @fernandascovino

Copy link
Collaborator

@fernandascovino fernandascovino left a comment

Choose a reason for hiding this comment

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

Tudo parece OK. Basta fechar os testes para subir.

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/init.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/constants.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @Hellcassius
- @fernandascovino

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/constants.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@github-actions
Copy link

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/registros_ocr_rir/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/tasks.py
- pipelines/rj_smtr/schedules.py
- pipelines/rj_smtr/utils.py
- pipelines/rj_smtr/constants.py
- pipelines/rj_smtr/tasks.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/tasks.py
- pipelines/rj_smtr/registros_ocr_rir/tasks.py

Os seguintes usuários devem ser avisados sobre a alteração:
- @fernandascovino
- @Hellcassius

@mergify
Copy link
Contributor

mergify bot commented Sep 23, 2022

@Hellcassius esse pull request tem conflitos 😩

@mergify mergify bot added the conflict label Sep 23, 2022
@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2022

@Hellcassius esse pull request tem conflitos 😩

@fernandascovino fernandascovino changed the title [WIP][smtr][sppo] Refatora recapturas [WIP] Refatora recapturas de GPS do SPPO Sep 30, 2022
@fernandascovino fernandascovino added the standby Waiting blockers to be solved label Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflict rj-smtr standby Waiting blockers to be solved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[smtr][gps] Mudar o flow de recapturas para criar novas flow runs
3 participants