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

Rename employment variables to "Labor Force" category #94

Merged

Conversation

danielhuppmann
Copy link
Member

As a follow-up to #57, this PR renames the Employment/Unemployment variables from the population-category to a new top-level category "Labor Force".

@IAMconsortium/common-definitions-macro-economy, looking at the description of the variables introduce in #57, it said that "Not Active" are people outside of the working-age range. However, this definition states that "Outside the Labor Force" includes all people not actively looking for work, which may include working-age people.

Please clarify or make your categorization complete by adding a category for inactive people of working age.

@danielhuppmann
Copy link
Member Author

Ping @FlorianLeblancDr, please double-check whether the description of "Not Active" should be changed to be in line with ILO definitions.

@FlorianLeblancDr
Copy link
Contributor

Hi Daniel,
Sorry for the long delay.
Thanks for checking, indeed Inactive does not only means not being at the age of working according to the ILO.

I made some edits.

I will see later if those definitions are robust to several definitions of unemployment (more for tier 2 / 3 var.)

@danielhuppmann
Copy link
Member Author

Thanks @FlorianLeblancDr - just to be clear, you wrote "I made some edits", but it's not clear to me where you made them?

@FlorianLeblancDr
Copy link
Contributor

FlorianLeblancDr commented May 21, 2024

@danielhuppmann Directly on Github here:
https://github.com/IAMconsortium/common-definitions/pull/94/files
File labor-force.yaml

Edit: I did not click on 'submit review' sorry for this.

@danielhuppmann
Copy link
Member Author

Thanks @FlorianLeblancDr, there was some formatting mix-up in your suggestions, so I revised them a bit further. Please approve if this is ready to be merged (a.k.a. good enough for now).

@FlorianLeblancDr
Copy link
Contributor

All good to me, thanks for the edits @danielhuppmann

@danielhuppmann danielhuppmann merged commit 4049b47 into IAMconsortium:main May 22, 2024
3 checks passed
@danielhuppmann danielhuppmann deleted the definitions/labour-market branch May 22, 2024 08:40
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.

2 participants