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

Provincial and Districts data of NRB #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nikeshbalami
Copy link
Contributor

No description provided.

Copy link
Member

@cliftonmcintosh cliftonmcintosh left a comment

Choose a reason for hiding this comment

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

The percentages for the "All" rows are incorrect.

@@ -0,0 +1,9 @@
State/Province,Total number of active male accounts,Total number of active female accounts,Total population,Percentage of active male accounts,Percentage of active female accounts
(All),9168740,5816647,22183283,259,169
Copy link
Member

Choose a reason for hiding this comment

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

I believe the percentages for the first column are incorrect. They are both over 100%. It looks like they were derived from adding the percentages in the provinces together rather than calculating percentages for the full population.

What do "Percentage of active male accounts" and "Percentage of active male accounts" mean in these data? Is it:
percent of accounts that are held by men and by women and that are "active"
percent of men (or women) in the total population who own a savings account?

What does "active" mean in these data? That isn't something that would be useful to answer when the data are used.

@@ -0,0 +1,79 @@
District,Total number of active male accounts,Total number of active female accounts,Total population,Percentage of active male accounts,Percentage of active female accounts
(All),9168740,5816647,22183283,2770,1714
Copy link
Member

Choose a reason for hiding this comment

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

Percentages for the first column both exceed 100%.

@@ -0,0 +1,79 @@
District,Total number of registered saving accounts,Total population,Registered saving accounts penetration (%)
(All),18446149,22183283,5404
Copy link
Member

Choose a reason for hiding this comment

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

Percentage exceeds 100%!

@@ -0,0 +1,79 @@
District,Total number of registered saving accounts,Total population,Registered saving accounts penetration (%)
Copy link
Member

Choose a reason for hiding this comment

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

Can we have a gloss somewhere, perhaps in a README, that describes the meaning of terms like what the "penetration" of saving accounts means, and what counts as active, etc.? That will make it easier to consume these data. Could we also have link to the original data set to make verification and review easy?

@@ -0,0 +1,9 @@
State/Province,Total number of registered saving accounts,Total population,Registered saving accounts penetration (%)
(All),18446149,22183283,534
Copy link
Member

Choose a reason for hiding this comment

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

Same here -- percentage exceeds 100%

@ravinepal
Copy link
Member

@nikeshbalami - can you pls advise? If the data is in accurate we don't have to use it I believe..

@nikeshbalami
Copy link
Contributor Author

Although it comes from the official source, NRB hasn't described the metadata and probably they have their own way of calculation. Let leave this issue for a while, I will have a look when I am free.

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.

3 participants