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

Correction application TVA #1914

Merged
merged 4 commits into from Jun 6, 2023
Merged

Correction application TVA #1914

merged 4 commits into from Jun 6, 2023

Conversation

Benjamin-Boisserie-ABC
Copy link
Collaborator

La TVA s'applique sur le montant des dépenses fait en TTC (pour repasser en HT la dépense) et non pas sur la valeur du ratio monétaire

La TVA s'applique sur le montant des dépenses fait en TTC (pour repasser en HT la dépense) et non pas sur la valeur du ratio monétaire
@Clemog
Copy link
Contributor

Clemog commented Jun 6, 2023

Pourquoi ne pas corriger le ratio monétaire ? Pour moi c'est plus simple

Dans ce cas là, ton point c'est qu'il y a un erreur au niveau de la conversion ?

Tu veux dire que 1kgCO2/€TTC = 1kgCO2e/(€HT*1.2) = 1kgCO2e/€HT / 1.2 ? Et non 1kgCO2/€TTC = 1kgCO2e/€HT * 0.8 ?

@Clemog
Copy link
Contributor

Clemog commented Jun 6, 2023

C'est important de garder les variables séparée (et là ça ne compile pas, on a mis en place les commentaires dans la PR pour justement signaler un pb dans le code)

@Clemog
Copy link
Contributor

Clemog commented Jun 6, 2023

Si on est d'accord sur mon commentaire, c'est une grosse erreur de ma part, à répercuter sur les résultats de l'enquête !

@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Report for the pull request #1914

🚀 Test the model from the website: https://nosgestesclimat.fr?PR=1914


🔧 Model compilation status

Task Status Message
Supported regions ✔️ Ø
Successfully compiled and optimized rules:
Expand
  • FR-fr
  • BE-fr
  • CH-fr
  • DE-fr
  • GF-fr
  • GP-fr
  • IT-fr
  • LU-fr
  • MQ-fr
  • PF-fr
  • PL-fr
  • PT-fr
  • RE-fr
  • TN-fr
  • TR-fr
  • UK-fr
  • YT-fr
  • FR-en-us
  • BE-en-us
  • CH-en-us
  • DE-en-us
  • GF-en-us
  • GP-en-us
  • IT-en-us
  • LU-en-us
  • MQ-en-us
  • PF-en-us
  • PL-en-us
  • PT-en-us
  • RE-en-us
  • TN-en-us
  • TR-en-us
  • UK-en-us
  • YT-en-us
✔️ Ø
Personas compilation to JSON for fr ✔️ Ø
Personas compilation to JSON for en-us ✔️ Ø
Documentation compilation to JSON ✔️ Ø

🌐 Translation status

Rules

Language Nb. missing translations Status
en-us Ø ✔️

Personas

Language Nb. missing translations Status
en-us Ø ✔️

You will find more information about the translation in the dedicated file.


👫 Personas changes

Test personas regression

Persona Total PR (kg CO2e) Total in prod. (kg CO2e) Δ (%)
8,374 8,424 -1%
2,933 2,934 0%
5,002 5,012 0%
11,547 11,556 0%
8,637 8,652 0%
13,425 13,523 -1%
8,559 8,609 -1%
8,532 8,581 -1%
9,157 9,206 -1%
14,155 14,204 0%
11,886 11,984 -1%
20,078 20,226 -1%
8,330 8,380 -1%

Test model optimisation

Persona Total PR with optim. (kg CO2e) Total PR without optim. (kg CO2e) Δ (%)

@Clemog
Copy link
Contributor

Clemog commented Jun 6, 2023

@Benjamin-Boisserie-ABC merci pour le retour, je merge
Comment on fait remonter ça au niveau de OW et l'enquête ? Ça me semble très important

@Clemog Clemog merged commit 0c60424 into master Jun 6, 2023
11 checks passed
@Clemog Clemog deleted the correction-RM-divers-1 branch June 6, 2023 15:19
Clemog added a commit that referenced this pull request Jun 6, 2023
@Clemog Clemog restored the correction-RM-divers-1 branch June 6, 2023 16:35
@@ -49,18 +49,17 @@ divers . autres produits:
Par exemple, il n'est pas du tout évident qu'un livre qui coûte plus cher ait une plus grosse empreinte.
De même, un ensemble de draps en lin pourrait très bien avoir une empreinte 5 fois plus faible qu'un ensemble en coton
(la culture du lin demande très peu de ressources), et pourtant ce dernier est susceptible de coûter jusqu'à 10 fois plus cher.
formule: ratio monétaire * montant / amortissement
formule: ratio monétaire TTC / amortissement
Copy link
Contributor

Choose a reason for hiding this comment

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

@Clemog @Benjamin-Boisserie-ABC c'est moi ou votre PR transforme cette variable en constante ? Il y a erreur dans la suppression de "montant" non ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh oui grosse grosse erreur, vraiment désolé !

@Benjamin-Boisserie-ABC
Copy link
Collaborator Author

Benjamin-Boisserie-ABC commented Jun 7, 2023

Alors pas sûr de bien suivre ce que tu fais ici

Tu veux dire que 1kgCO2/€TTC = 1kgCO2e/(€HT*1.2) = 1kgCO2e/€HT / 1.2 ? Et non 1kgCO2/€TTC = 1kgCO2e/€HT * 0.8 ?

Mais pour passer du HT au TTC on fait en effet HT x 1.2 et non HT x 0,8

Après j'ai pas bien compris pourquoi ma modif faisait foirer la compilation. J'ai voulu corriger pour faire comme on fait habituellement lorsqu'on utilise les ratios monétaires : montant de la dépense HT (ou TTC que l'on ramène en HT en divisant par 1,2) * ratio monétaire HT plutôt que l'inverse.

@Benjamin-Boisserie-ABC
Copy link
Collaborator Author

Comment on fait remonter ça au niveau de OW et l'enquête ? Ça me semble très important

Pour l'enquête c'est pas très grave à partir du moment ou l'on a la valeur brute renseignée., ce qui nous permet de corriger la quantification GES à posteriori en repartant de cette valeur et de la nouvelle "formule"

@Clemog
Copy link
Contributor

Clemog commented Jun 7, 2023

Pour l'enquête c'est pas très grave à partir du moment ou l'on a la valeur brute renseignée., ce qui nous permet de corriger la quantification GES à posteriori en repartant de cette valeur et de la nouvelle "formule"

J'entends bien mais il faut être sur que ce soit bien remonté auprès de OW et qu'on ait une base commune de données

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

Successfully merging this pull request may close these issues.

None yet

3 participants