cncf / devstats

📈CNCF-created tool for analyzing and graphing developer contributions
https://devstats.cncf.io
Apache License 2.0
66 stars 24 forks source link

[bug] GitHub account is missing country and location information. #1

Closed ydFu closed 1 year ago

ydFu commented 1 year ago

Problem description:

When using the following search, I found that my contributions are not included in the country statistics.

  1. My contributions cannot be found.
    • github: ydFu
    • country: Taiwan

https://k8s.devstats.cncf.io/d/66/developer-activity-counts-by-companies?orgId=1&var-period_name=Last%20decade&var-metric=contributions&var-repogroup_name=All&var-repo_name=kubernetes%2Fkubernetes&var-country_name=Taiwan%20Province%20of%20China&var-companies=All

image


  1. But my contributions are included in "all".

https://k8s.devstats.cncf.io/d/66/developer-activity-counts-by-companies?orgId=1&var-period_name=Last%20decade&var-metric=contributions&var-repogroup_name=All&var-repo_name=kubernetes%2Fkubernetes&var-country_name=All&var-companies=All

image

Possible reasons:

Currently, when checking my data in github_users.json, I found that the fields "country_id" and "location" are missing

github_users.json https://github.com/cncf/devstats/blob/master/github_users.json

image

Bug fix.

I hope that I can adjust the settings according to my own country on GitHub,And anyone can search for country-specific categories of related metrics within k8s.devstats.cncf.io.

image

lukaszgryglicki commented 1 year ago

I will do this fix next week.

lukaszgryglicki commented 1 year ago

Updated JSON, please note that changes will be visible when the newest affiliations are imported (which happens once per 4 weeks).

Seo-yul commented 1 year ago

@lukaszgryglicki What happens when there is no information in the file? I became a member of Kubernetes last year, but there is no information in this github_users.json file.

lukaszgryglicki commented 1 year ago

Please add your data in cncf/gitdm following this so you data will be added in post-merge scripts.

Seo-yul commented 1 year ago

I will check that page Thank you so much.