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][smtr] fix: corrigir código de recaptura #128

Closed
wants to merge 10 commits into from

Conversation

fernandascovino
Copy link
Contributor

@fernandascovino fernandascovino commented Jul 28, 2022

Closes #126

Changelog:

  • registros_logs: Adicionar busca pelos minutos sem timestamp_captura (além de erros capturados nos logs)
  • upload_logs: Corrige status para registro dos logs

TODO:

@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/br_rj_riodejaneiro_stpl_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/init.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/br_rj_riodejaneiro_onibus_gps/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_stpl_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/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_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/rj_smtr/br_rj_riodejaneiro_stpl_gps/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
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/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/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py

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

@fernandascovino fernandascovino changed the title [WIP][smtr] fix: corrrigir código de recaptura [WIP][smtr] fix: corrigir código de recaptura Jul 28, 2022
@github-actions
Copy link

===> 3 convention(s) found:

  • convention [C0103] Variable name "df" doesn't conform to snake_case naming style (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:110:12)
  • convention [C0103] Variable name "df" doesn't conform to snake_case naming style (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:126:8)
  • convention [C0103] Variable name "df" doesn't conform to snake_case naming style (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:126:13)
    ===> 1 refactor(s) found:
  • refactor [R0914] Too many local variables (16/15) (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:49: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/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.py
- pipelines/rj_smtr/init.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_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.py

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

@github-actions
Copy link

===> 3 convention(s) found:

  • convention [C0103] Variable name "df" doesn't conform to snake_case naming style (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:110:12)
  • convention [C0103] Variable name "df" doesn't conform to snake_case naming style (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:126:8)
  • convention [C0103] Variable name "df" doesn't conform to snake_case naming style (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:126:13)
    ===> 1 refactor(s) found:
  • refactor [R0914] Too many local variables (16/15) (pipelines/rj_smtr/br_rj_riodejaneiro_onibus_gps/tasks.py:49: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/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_stpl_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_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/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:
- @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_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/br_rj_riodejaneiro_stpl_gps/flows.py

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

@github-actions
Copy link

===> 1 refactor(s) found:

  • refactor [R0914] Too many local variables (17/15) (pipelines/rj_cor/comando/eventos/tasks.py:85: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/br_rj_riodejaneiro_onibus_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py

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

@github-actions
Copy link

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14: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/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.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:
- @Hellcassius
- @fernandascovino

@fernandascovino
Copy link
Contributor Author

Changelog:

  • Adiciona parâmetro de datetime_filter no flow, para recapturar qualquer timestamp manualmente (nas ultimas 24h)
  • Corrige lógica de catch e upload de erros:
    • bq_upload retorna erro que foi passado das tasks anteriores
    • upload_logs_to_bq recebe erro consolidado da bq_upload
  • Fixa timestamp da captura em pre_treatment_br_rj_riodejaneiro_onibus_gps e upload_logs_to_bq
    • Na captura, é passado como timestamp o retorno da create_current_date_hour_partition (1a task)
    • Na recaptura, é passado como timestamp a lista de timestamps falhadas retornada pela query_logs

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

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/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_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

github-actions bot commented Aug 2, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

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/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
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py

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

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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/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
- 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

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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_stpl_gps/flows.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

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@gabriel-milan gabriel-milan changed the base branch from master to staging August 4, 2022 18:50
@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/materialize_to_datario/flows.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

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

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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_sigmob/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/materialize_to_datario/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.py

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

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Análise da árvore de código

Os seguintes arquivos são afetados diretamente por alterações realizadas nesse pull request:
- pipelines/rj_smtr/materialize_to_datario/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_sigmob/flows.py
- pipelines/rj_smtr/init.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.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

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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_stpl_gps/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

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

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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_stpl_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_brt_gps/flows.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

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

===> 1 convention(s) found:

  • convention [C0411] standard import "import traceback" should be placed before "from pytz import timezone" (pipelines/rj_smtr/tasks.py:14:0)

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

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/br_rj_riodejaneiro_brt_gps/flows.py
- pipelines/rj_smtr/br_rj_riodejaneiro_stpl_gps/flows.py
- pipelines/rj_smtr/init.py

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

@mergify
Copy link
Contributor

mergify bot commented Aug 5, 2022

@fernandascovino esse pull request tem conflitos 😩

@mergify mergify bot added the conflict label Aug 5, 2022
Base automatically changed from staging to master August 10, 2022 19:31
@fernandascovino
Copy link
Contributor Author

closed by #141

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[dados][gps_sppo] Corrrigir código de recaptura
1 participant