github-readme-stats icon indicating copy to clipboard operation
github-readme-stats copied to clipboard

Show progress rank icon (100-percentile)

Open Madhav-MKNC opened this issue 2 years ago • 10 comments

Is your feature request related to a problem? Please describe.

Currently, the GitHub stats card displays the percentile in the form of the "in the top X percentage".

Describe the solution you'd like

It would be better if it could also display the percentile in the form "Y percentile" where Y = (100-X).

Describe alternatives you've considered

No response

Additional context

No response

Madhav-MKNC avatar Jun 21 '23 09:06 Madhav-MKNC

@Madhav-MKNC, thanks for your feature request. We briefly merged what you want in https://github.com/anuraghazra/github-readme-stats/pull/2858, but we reverted it since the percentile was more apparent. Can you checkout if this is what you had in mind:

![Anurag's GitHub stats](https://github-readme-stats-git-addprogressrankicon-rickstaa.vercel.app/api?username=anuraghazra\&rank_icon=progress)

Anurag's GitHub stats

rickstaa avatar Jun 21 '23 09:06 rickstaa

Yes, It's exactly what I was asking for, thanks for sharing.

Madhav-MKNC avatar Jun 21 '23 10:06 Madhav-MKNC

@rickstaa, Can we have a separate card for just the rank_icon, I think many users like me like to show all of followings on their profile: grade, percentile, progress

Madhav-MKNC avatar Jun 21 '23 10:06 Madhav-MKNC

Yes, It's exactly what I was asking for, thanks for sharing.

Thanks! @qwerty541, what are your thoughts about adding the progress code back in the master branch?

rickstaa avatar Jun 21 '23 14:06 rickstaa

@rickstaa, Can we have a separate card for just the rank_icon, I think many users like me like to show all of following on their profile: grade, percentile, progress

You can create as many copies of the stats card as possible. When https://github.com/anuraghazra/github-readme-stats/pull/2868 is merged, you can remove all the stats and show the rank_progress and grade next to each other.

rickstaa avatar Jun 21 '23 14:06 rickstaa

@rickstaa, Can we have a separate card for just the rank_icon, I think many users like me like to show all of the following on their profile: grade, percentile, progress

You can create as many copies of the stats card as possible. When https://github.com/anuraghazra/github-readme-stats/pull/2868 is merged, you can remove all the stats and show the rank_progress and grade next to each other.

rickstaa avatar Jun 21 '23 14:06 rickstaa

I am not able to figure out the url for this #2868

Madhav-MKNC avatar Jun 21 '23 16:06 Madhav-MKNC

Yes, It's exactly what I was asking for, thanks for sharing.

Thanks! @qwerty541, what are your thoughts about adding the progress code back in the master branch?

As i wrote in https://github.com/anuraghazra/github-readme-stats/pull/2859#pullrequestreview-1487346398 i think that it is impossible to understand what the 100 - percentile (i.e. 80%) means for random profile visitor. Because of this, i think that it is better to not return this functionality to master branch. But we can keep this issue open and if it receive a lot of community support (#1935) may be we can still add it later.

@Madhav-MKNC you are able to deploy GRS at this point https://github.com/anuraghazra/github-readme-stats/tree/a362af0ebc7bbcd6821e70ba559fecde4ce3bf2e on your own vercel instance if you need this right now. If you need something for own use it does not means that it should be publicly deployed since it may turn out that no one needs it except you.

qwerty541 avatar Jun 22 '23 05:06 qwerty541

Yes, It's exactly what I was asking for, thanks for sharing.

Thanks! @qwerty541, what are your thoughts about adding the progress code back in the master branch?

As i wrote in #2859 (review) i think that it is impossible to understand what the 100 - percentile (i.e. 80%) means for random profile visitor. Because of this, i think that it is better to not return this functionality to master branch. But we can keep this issue open and if it receive a lot of community support (#1935) may be we can still add it later.

@Madhav-MKNC you are able to deploy GRS at this point https://github.com/anuraghazra/github-readme-stats/tree/a362af0ebc7bbcd6821e70ba559fecde4ce3bf2e on your own vercel instance if you need this right now. If you need something for own use it does not means that it should be publicly deployed since it may turn out that no one needs it except you.

@Madhav-MKNC I have to agree with @qwerty541 that since the stat is not apparent, it is better to not merge it until there is more demand. Let's see if this feature request gets enough 👍🏻 to merge it in the future. Neverteless, I created https://github.com/anuraghazra/github-readme-stats/pull/2875, which you can use on your own Vercel instance if you already want to use this feature.

rickstaa avatar Jun 22 '23 08:06 rickstaa

@rickstaa @qwerty541 Fine if that is not what majority then you don't have to merge this functionality to the master branch.

Madhav-MKNC avatar Jun 23 '23 16:06 Madhav-MKNC