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

Suite services societaux #1638

Merged
merged 15 commits into from Jan 12, 2023
Merged

Suite services societaux #1638

merged 15 commits into from Jan 12, 2023

Conversation

Clemog
Copy link
Contributor

@Clemog Clemog commented Jan 4, 2023

#1624

Démo : https://deploy-preview-797--nosgestesclimat.netlify.app/?PR=1638

@github-actions
Copy link

github-actions bot commented Jan 11, 2023

Report for the pull request #1638

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


🔧 Model compilation status

Task Status Message
Rules evaluation ✔️ Ø
Rules evaluation
See error:

TypeError: Cannot read properties of undefined (reading 'suggestions')
TypeError: Cannot read properties of undefined (reading 'suggestions')
at /home/runner/work/nosgestesclimat/nosgestesclimat/scripts/i18n/addTranslationToBaseRules.js:41:24
at Array.forEach (<anonymous>)
at /home/runner/work/nosgestesclimat/nosgestesclimat/scripts/i18n/addTranslationToBaseRules.js:36:5
at Array.forEach (<anonymous>)
at addTranslationToBaseRules (/home/runner/work/nosgestesclimat/nosgestesclimat/scripts/i18n/addTranslationToBaseRules.js:33:34)
at /home/runner/work/nosgestesclimat/nosgestesclimat/scripts/rulesToJSON.js:94:28
at Array.forEach (<anonymous>)
at /home/runner/work/nosgestesclimat/nosgestesclimat/scripts/rulesToJSON.js:88:13
at f (/home/runner/work/nosgestesclimat/nosgestesclimat/node_modules/once/once.js:25:25)
at Glob.<anonymous> (/home/runner/work/nosgestesclimat/nosgestesclimat/node_modules/glob/glob.js:148:7)

| Rules compilation to JSON for en-us | ✔️ | Ø |
| Rules compilation to JSON for fr | ✔️ | Ø |
| Personas compilation to JSON for fr | ✔️ | Ø |
| Personas compilation to JSON for es | ✔️ | Ø |
| Personas compilation to JSON for it | ✔️ | Ø |
| Personas compilation to JSON for en-us | ✔️ | Ø |
| Documentation compilation to JSON | ✔️ | Ø |


🌐 Translation status

Rules

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

Personas

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

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

@Clemog
Copy link
Contributor Author

Clemog commented Jan 11, 2023

Je pense qu'on est bon pour merge cette v1 ! Rien de cassant à priori !

@EmileRolley @laem n'hésitez pas à relire https://github.com/datagir/nosgestesclimat/blob/eb1655897ac67429615b71bd5de8d646383c42c6/docs/guide-technique-services-societaux.md

Il reste éventuellement la description des scripts, vous pensez que c'est utile ?

@Clemog
Copy link
Contributor Author

Clemog commented Jan 11, 2023

Ah il y a peut-être des conflits au niveau de la trad ?

Edit: J'ai mis à jour le fichier de trad, je suis étonné de la diff de ce commit

Copy link
Contributor

@EmileRolley EmileRolley left a comment

Choose a reason for hiding this comment

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

Le guide me semble très bien ! Je t'ai laissé quelques commentaires.

>
> **Un pré-requis pour vous lancer dans ce guide est la lecture de notre article sur [l'implémentation des "services scoiétaux" dans Nos Gestes Climat](https://nosgestesclimat.fr/nouveaut%C3%A9s/l'empreinte-climat%20des%20%22services%20soci%C3%A9taux%22).**

**En bref**, certains postes constituant l'empreinte carbone individuelle sont inhérents à la société à laquelle nous appartenons et ne peuvent pas être captés autrement que via une approche macro-économique (ie l'approche "montante" utilisée dans le reste du test NGC et permettant de reconstituer l'empreinte individuelle via les données physiques de consommation n'est pas suffisante). Ils correspondent à l'empreinte des services publics français, et des services marchands que l'on peut considérer comme étant essentiels à la vie de chacun, divisée par la population du pays.
Copy link
Contributor

Choose a reason for hiding this comment

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

Ce serait intéressant de rediriger vers une explication de l'approche bottom-up, je ne sais pas si elle existe déjà ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

L'approche "bottum-up" c'est NGC du coup, on a pas vraiment (et pas encore) écrit sur le sujet

docs/guide-technique-services-societaux.md Outdated Show resolved Hide resolved
docs/guide-technique-services-societaux.md Outdated Show resolved Hide resolved
docs/guide-technique-services-societaux.md Outdated Show resolved Hide resolved
docs/guide-technique-services-societaux.md Show resolved Hide resolved
docs/guide-technique-services-societaux.md Show resolved Hide resolved

#### Cas des valeurs "secrètes"

Certains chiffres d'affaire sont marquées "S" (ie, "soumises au secret statistique") : c'est le cas pour l'extraction de gaz naturel, l'industrie du tabac ou encore les engins militaires. Dans la suite nous prendrons l'exemple du groupement SDES C10_12 comprenant C10, C11 dont le CA est connu et C12 inconnu.
Copy link
Contributor

Choose a reason for hiding this comment

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

C'est fou, pourquoi les chiffres de l'extraction de gaz naturel et de l'industrie du tabac sont secrets ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bonne question, lobby ? données sensibles pour l'état ?

docs/guide-technique-services-societaux.md Outdated Show resolved Hide resolved
docs/guide-technique-services-societaux.md Outdated Show resolved Hide resolved
docs/guide-technique-services-societaux.md Outdated Show resolved Hide resolved
@Clemog Clemog merged commit deaa6b8 into master Jan 12, 2023
@Clemog Clemog deleted the suite-services-societaux branch January 12, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants