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

FIX timeout report de retencio d'irpf sobre el rendiment de GenerationkWh #23

Merged
merged 6 commits into from
Apr 27, 2020

Conversation

susu105
Copy link
Contributor

@susu105 susu105 commented Apr 22, 2020

Objectiu

Reduïr el temps en generar-se el report de retencio d'irpf sobre el rendiment de GenerationkWh

Comportament antic

  • El report triga en quasi tots els casos, a producció, més de 180 segons en generar-se i el worker que de render acaba matant la tasca.

Nou comportament

  • En comptes de fer servir la funció get_irpf_amounts, es fa servir get_total_saving_partner la qual és molt més lleugera de processar
  • Update: s'utilitza una sentencia SQL

@oriolpiera oriolpiera force-pushed the NEW_report_retencions_generationkwh branch from 1addd23 to 23608e4 Compare April 27, 2020 12:09
@oriolpiera
Copy link
Member

@susu105 he fet un:

git checkout NEW_report_retencions_generationkwh
git rebase master

La teva branca partia d'un master antic. Amb aquests comenades, he fet que es basi amb l0últim commit de master, atualitzant els últims canvis, abans de fer el Rebase d'aquesta PR. Això ens ajuda a minimitzar conflictes i en aquest cas, que passin els testos (vaig fer un commit a master per que no petessin els testos al CI).

Exemple visual ;)

      A---B---C NEW_report_retencions_generationkwh
     /
D---E---F---G master


              A'--B'--C' NEW_report_retencions_generationkwh
             /
D---E---F---G master

@oriolpiera oriolpiera merged commit 3a98225 into master Apr 27, 2020
@oriolpiera oriolpiera deleted the NEW_report_retencions_generationkwh branch January 21, 2021 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants