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

🇨🇦 Create new model for Quebec - Canada #2000

Merged
merged 7 commits into from Jul 31, 2023

Conversation

davidson-canada-devops
Copy link
Contributor

Ajout initial d'un modèle pour le Québec.
Pour le moment seul 2 facteurs d'émission sont adaptés, le mixe électrique et le poids des services publics.
L'adaptation Québécoise va nécessiter plus de modification depuis le modèle FR pour etre cohérent, l'empreinte moyenne d'un Québécois étant d'environ 15 tonnes.
Jusqu'à ce que l'on puisse sélectionner une région au siens d'un pays, la sélection du Canada utilise les facteurs d'émissions spécifiques au Québec.

@Clemog Clemog changed the title Create new model for Quebec - Canada 🇨🇦 Create new model for Quebec - Canada Jul 25, 2023
@Clemog
Copy link
Contributor

Clemog commented Jul 25, 2023

Hello @davidson-canada-devops !

Merci pour la contribution ! On regarde ça au plus vite ! :)

@Clemog
Copy link
Contributor

Clemog commented Jul 25, 2023

@davidson-canada-devops je peux te laisser ajouter la modification des suggestions pour les transports (au moins train et avion mais il y a aussi le ferry qui pourrait être adapté), exemple pour la Belgique:

transport . avion . court courrier . heures de vol:
  suggestions:
    🚫✈️: 0
    Bruxelles ⇄ Paris: 2
    Bruxelles ⇄ Londres: 2.5
    Bruxelles ⇄ Berlin: 3
    Bruxelles ⇄ Rome: 4

transport . avion . moyen courrier . heures de vol:
  suggestions:
    🚫✈️: 0
    Bruxelles ⇄ Madrid: 5
    Bruxelles ⇄ Marrakech: 7
    Bruxelles ⇄ Le Caire: 8

transport . avion . long courrier . heures de vol:
  suggestions:
    🚫✈️: 0
    Bruxelles ⇄ New York: 16
    Bruxelles ⇄ Lima: 36
    Bruxelles ⇄ Tokyo: 40
    Bruxelles ⇄ Sydney: 60

transport . train . km:
  suggestions:
    traversée d'une région: 300
    Bruxelles ⇄ Amsterdam: 350
    Bruxelles ⇄ Paris: 500
    💳 grand voyageur: 10000
    💳 grand voyageur plus ultra: 25000

@Clemog
Copy link
Contributor

Clemog commented Jul 25, 2023

Autre remarque, je me suis permis de reprendre un mix électrique lissé sur l'année.

Je trouve que le mix québecois n'est sans doute pas représentatif du mix canadien (très bas), surtout si la moyenne est 15 tonnes...

@github-actions
Copy link

github-actions bot commented Jul 25, 2023

Report for the pull request #2000

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


🔧 Model compilation status

Task Status Message
Supported regions ✔️ Ø
Successfully compiled and optimized rules:
Expand
  • FR-fr
  • BE-fr
  • CA-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
  • CA-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) Δ (%)

Test model optimisation

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

@Clemog
Copy link
Contributor

Clemog commented Jul 25, 2023

Normalement, le lien de démo fonctionne, tu es bien détecté au canada avec la bonne version ?

@davidson-canada-devops
Copy link
Contributor Author

Merci @Clemog
La détection fonctionne très bien de mon coté (depuis Montréal)
Je vais faire une passe sur les suggestions de transport et j'aimerais bien soumettre un personna Qc

@ogourment
Copy link

Qu'est-ce que ça prendrait pour que ça fonctionne aussi en anglais puisqu'on vise le Canada ? Je peux ajouter des traductions.

@davidson-canada-devops
Copy link
Contributor Author

Qu'est-ce que ça prendrait pour que ça fonctionne aussi en anglais puisqu'on vise le Canada ? Je peux ajouter des traductions.

La traduction se passe dans ce fichier, je viens d'y rajouter les suggestions de transport : e971e7b

Pour le moment je me concentre uniquement sur le Québec car les empreintes carbone entre provinces sont vraiment trop distinctes et je trouve ca difficile et risqué d'en faire des moyennes.

@Clemog Clemog self-assigned this Jul 27, 2023
@Clemog
Copy link
Contributor

Clemog commented Jul 27, 2023

Je pense que c'est bon pour la V1 ! Effectivement, on verra quels sont les retours et si la problématique de la détection à l'échelle de la région remonte, on pourra prioriser cette feature par la suite !

Ok pour toi @davidson-canada-devops ?

@Leicas
Copy link

Leicas commented Jul 29, 2023

Je confirme que ça marche ici aussi, et merci à tous pour le boulot sur la PR !
Je viens de repasser les étapes et la seule partie qui est un peu étrange, c'est sur les carburants, les indices d’octanes sont différents en Amérique du Nord et c’est aussi un impact sur la consommation des véhicules (mais je ne suis pas expert dans le domaine).

@Clemog
Copy link
Contributor

Clemog commented Jul 31, 2023

Je viens de repasser les étapes et la seule partie qui est un peu étrange, c'est sur les carburants, les indices d’octanes sont différents en Amérique du Nord et c’est aussi un impact sur la consommation des véhicules (mais je ne suis pas expert dans le domaine).

Ah yes super remarque !
C'est effectivement une question d'intitulés mais aussi de facteurs d'émission qui se cachent derrière (qui aujourd'hui sont ceux de la Base Empreinte, territoire Français donc), il serait vraiment intéressant que davantage de chiffres (autre que le mix électrique) soient adaptés :)

@Clemog Clemog merged commit ac739dd into incubateur-ademe:master Jul 31, 2023
11 checks passed
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

4 participants