New top language algorithm implementation #1732

kitswas · 2022-05-03T10:30:24Z

Old Card

New Card

The tests have been updated.

I am not very familiar with Javascript so I made only minimal changes.
Some refactoring might be required.

Implements #1600

vercel · 2022-05-03T10:30:29Z

@kitswas is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

kitswas · 2022-05-05T01:19:53Z

There was an error in my earlier implementation.

See the differences below:

Has error -
Current -

Preview Deployment URL - https://github-readme-stats-l34h9pi63-kitswas.vercel.app

anuraghazra · 2022-05-05T14:12:30Z

@rickstaa whats your thoughts on this?

rickstaa · 2022-05-07T08:59:04Z

@anuraghazra, I agree with @kitswas that the language counting algorithm could be improved. Due to some pressing deadlines, I did not yet have time to go deep into the proposed calculation. I will therefore give you my first thoughts.

From my first impression, the y variable (i.e. the number of repositories) influences the percentages. By running an optimisation on the p and q factors, we might get better results, but I'm afraid that judging the accuracy of the result might be very subjective. Currently, I am inclined to go with the old behaviour since it is similar to what GitHub uses in their repo language overview (i.e. https://github.com/github/linguist/). I, however, need to dive in deeper to give a detailed conclusion.

kitswas · 2022-05-07T09:18:27Z

From my first impression, the y variable (i.e. the number of repositories) influences the percentages. By running an optimisation on the p and q factors, we might get better results, but I'm afraid that judging the accuracy of the result might be very subjective. Currently, I am inclined to go with the old behaviour since it is similar to what GitHub uses in their repo language overview (i.e. https://github.com/github/linguist/). I, however, need to dive in deeper to give a detailed conclusion.

The influence of y is felt only when it is greater than 1, i.e. when a language appears in multiple repositories.
The numbers p and q could be optimised. Allowing the users to specify these might be a possibility.
The lower the exponents, the more stable the percentages are.

rickstaa · 2022-05-07T10:05:51Z

Without diving deeper into the formula. If we decide to implement this option the following would make sense to me:

Rewrite the k = x^p * y^q in a way that allows us to disable the new behaviour by default.
By default disabling the influence of y.
Giving users the ability to change the p and q values.
Documenting this new behaviour in the README.

src/fetchers/top-languages-fetcher.js

rickstaa · 2022-05-07T10:57:53Z

Please be aware that I did not do any research on metrics that could give a better 'used languages' representation. I, therefore, can only comment on the metric you proposed in #1732 and #1600. It could be that there are better statistical indexes for solving the problem. Nevertheless, I don't think we need to overthink this since your metric does what it is supposed to do.

kitswas · 2022-05-07T12:49:55Z

The new commits -

Rewrite the program in a way that disables the new behaviour by default.
By default disable the influence of y.
Give users the ability to change the p and q values
Document this new behaviour in the README.

The preview deployment URL is https://github-readme-stats-7vlsc5n0x-kitswas.vercel.app

This will prevent users from seeing their lang-cards suddenly change percentages if and when this is deployed.

rickstaa · 2022-05-08T09:20:52Z

@kitswas, thanks a lot for creating the new commits. It shows you are devoted to improving this project! I think people like you make a strong community found behind this project! 🚀

I, however, have to say I'm still not 100% on board with this. When I created #1732 (comment) I overlooked the case where p is 0 and q is 1. I don't think this version makes sense with the conception people might have with the language card. Namely, that is similar to the one shown by GitHub. I, however, understand that I'm also biased since I'm happy with the old behaviour and, therefore, likely would not use #1732 myself. I changed #1600 to a proposal so we can get more feedback from the community.

@anuraghazra, what do you think? Another way to implement this would be to create a switching variable rank_method, allowing users to switch from the old behaviour to the new behaviour. In that case, we can fix the p and q to more optimal values. However, this will not make everybody happy and wants more control over the language card and gives us more optimisation work. I, therefore, think there are two paths: keep the old behaviour since it is more consistent with what people expect to see or add the behaviour where people can change the p and q values.

rickstaa · 2022-05-08T09:43:39Z

@kitswas For now, #1732 looks good. Let's wait on what @anuraghazra says. I will review this once more if we decide to merge it.

This reverts commit b0bc626.

This was overwritten in the merge commit.

kitswas · 2023-04-19T16:37:51Z

Pulled latest changes from anuraghazra/github-readme-stats/master.

rickstaa · 2023-04-24T13:42:44Z

Pulled latest changes from anuraghazra/github-readme-stats/master.

Thanks, I made some small changes (see 9ce9bde) but I think this PR is good to be merged now 👍🏻.

codecov · 2023-04-24T13:43:48Z

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (4d1d83d) 97.31% compared to head (44b6ada) 97.32%.

❗ Current head 44b6ada differs from pull request most recent head 7aa3546. Consider uploading reports for the commit 7aa3546 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1732      +/-   ##
==========================================
+ Coverage   97.31%   97.32%   +0.01%     
==========================================
  Files          24       24              
  Lines        4249     4267      +18     
  Branches      385      387       +2     
==========================================
+ Hits         4135     4153      +18     
  Misses        112      112              
  Partials        2        2

Impacted Files	Coverage Δ
api/top-langs.js	`95.65% <100.00%> (+0.19%)`	⬆️
src/fetchers/top-languages-fetcher.js	`88.23% <100.00%> (+1.18%)`	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

kitswas · 2023-04-24T17:07:39Z

@rickstaa Thanks for your continuous support.
I wonder if we should change the names p and q to size_weight and count_weight, respectively.
p and q are not exactly intuitive.

kitswas · 2023-04-24T17:15:25Z

Users can also list by commit count and by size separately.
This effectively removes ambiguity and confusion.
For reference, here's how mine looks:

rickstaa · 2023-04-24T18:10:25Z

count_weight

Sounds like a plan! If you commit these changes, both in the code and the README, I will merge the pull request :).

Renamed `p` and `q` to `size_weight` and `count_weight`, respectively.

Changes introduced in f2516d6

kitswas · 2023-04-25T04:39:03Z

@rickstaa Updated.

rickstaa · 2023-04-25T06:19:01Z

Great! Merged into the main. Thanks for your contribution.

* refactor: update languages JSON (anuraghazra#2596) Co-authored-by: rickstaa <[email protected]> * Add format stats option (anuraghazra#2155) * feat: added `format_stats` option (anuraghazra#2128) * refactor: change `format_stats` to `short_values` (anuraghazra#2128) * test: create shorten values test (anuraghazra#2128) * Update readme.md Co-authored-by: Rick Staa <[email protected]> * refactor: rename ``short_values`` to ``number_format`` * Update readme.md Co-authored-by: Rick Staa <[email protected]> * Update src/cards/stats-card.js Co-authored-by: Rick Staa <[email protected]> * refactor: format codebase --------- Co-authored-by: Rick Staa <[email protected]> * add github_dark_dimmed theme (anuraghazra#2594) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank_icon option (anuraghazra#2628) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank icon option * feat(RankIcon): extract rankIcon to icons file * feat(RankIcon): update readme * feat(RankIcon): test coverage * Update readme.md Co-authored-by: Rick Staa <[email protected]> --------- Co-authored-by: Rick Staa <[email protected]> * Updated the custom error there was a Typo. (anuraghazra#2618) "Something went while trying to retrieve the stats data using the GraphQL API." -> "Something went wrong while trying to retrieve the stats data using the GraphQL API." * Fix typos inside wakatime test name (anuraghazra#2617) * removed redundant comparison (anuraghazra#957) * dev (anuraghazra#1782) * fix path to powered by vercel image (anuraghazra#1792) * Synchonize cache seconds min value inside docs translations with main readme (anuraghazra#2616) * New top language algorithm implementation (anuraghazra#1732) * Reduced vercel maxDuration * Implemented new algorithm for Top Langs * Revert "Reduced vercel maxDuration" This reverts commit b0bc626. * Added documentation * Fixed broken implementation * Update fetchTopLanguages.test.js Changed tests * Now uses the general formula The parameters p and q can be set by the user. * Updated tests and added new test * Added new test New test for order by repo count. * Updated documentation Added explanation and examples for new options. * Updated documentation This was overwritten in the merge commit. * docs: improve docs and fix tests * Renamed parameters Renamed `p` and `q` to `size_weight` and `count_weight`, respectively. * Updated the documentation Changes introduced in f2516d6 --------- Co-authored-by: rickstaa <[email protected]> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: rickstaa <[email protected]> Co-authored-by: Caeden Perelli-Harris <[email protected]> Co-authored-by: Rick Staa <[email protected]> Co-authored-by: Fabiano Couto <[email protected]> Co-authored-by: LOKESH SINGH <[email protected]> Co-authored-by: Alexandr Garbuzov <[email protected]> Co-authored-by: kitswas <[email protected]>

* Reduced vercel maxDuration * Implemented new algorithm for Top Langs * Revert "Reduced vercel maxDuration" This reverts commit b0bc626. * Added documentation * Fixed broken implementation * Update fetchTopLanguages.test.js Changed tests * Now uses the general formula The parameters p and q can be set by the user. * Updated tests and added new test * Added new test New test for order by repo count. * Updated documentation Added explanation and examples for new options. * Updated documentation This was overwritten in the merge commit. * docs: improve docs and fix tests * Renamed parameters Renamed `p` and `q` to `size_weight` and `count_weight`, respectively. * Updated the documentation Changes introduced in f2516d6 --------- Co-authored-by: rickstaa <[email protected]>

* Include all repo w/ access to * parameterize affiliation * Use GraphQL variable syntax * Custom ownerAffiliations for general stats * Add default affiliation * Apply patches anuraghazra#1122 (comment) * build(deps): bump json5 from 2.2.1 to 2.2.3 (anuraghazra#2373) Bumps [json5](https://github.com/json5/json5) from 2.2.1 to 2.2.3. - [Release notes](https://github.com/json5/json5/releases) - [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md) - [Commits](json5/json5@v2.2.1...v2.2.3) --- updated-dependencies: - dependency-name: json5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: update snapshots * Add loading Animation to Most used Language card (anuraghazra#2197) * add: text animation * compact progress done * cleaned code * non compact animation done * tests fixed + prettier * revert: vercel.json * remove: unnecessary import * added parameter in docs * style: improve syntax Co-authored-by: rickstaa <[email protected]> * ci: fix theme readme generation action (anuraghazra#2271) * Revert "build(deps): bump json5 from 2.2.1 to 2.2.3 (anuraghazra#2373)" (anuraghazra#2393) This reverts commit 42fdff6. * Revert "ci: fix theme readme generation action (anuraghazra#2271)" (anuraghazra#2394) This reverts commit 5ae959b. * Revert "Add loading Animation to Most used Language card (anuraghazra#2197)" (anuraghazra#2396) This reverts commit 77dcdab. * fix: retryer fixed (anuraghazra#2400) * Revert "Revert "ci: fix theme readme generation action (anuraghazra#2271)" (anuraghazra#2394)" (anuraghazra#2418) This reverts commit 70f0264. * refactor: update code formatting * Revert "Revert "Add loading Animation to Most used Language card (anuraghazra#2197)" (anuraghazra#2396)" (anuraghazra#2419) This reverts commit 4b8198f. * Update readme.md (anuraghazra#2442) * test: fix e2e tests * test: update snapshots * test: fix e2e test data * test: update e2e test data * feat: fix e2e tests * fix: fix e2e test data * feat: enable multi page star fetching for private vercel instances (anuraghazra#2159) * feat: enable multi-page stars' fetching for private vercel instances This commit enables multi-page stars' support from fetching on private Vercel instances. This feature can be disabled on the public Vercel instance by adding the `FETCH_SINGLE_PAGE_STARS=true` as an env variable in the public Vercel instance. This variable will not be present when people deploy their own Vercel instance, causing the code to fetch multiple star pages. * fix: improve stats multi-page fetching behavoir This commit makes sure that the GraphQL api is only called one time per 100 repositories. The old method added one unnecesairy GraphQL call. * docs: update documentation * style: improve code syntax Co-authored-by: Matteo Pierro <[email protected]> * lol happy new year * docs: remove rate limit documentation for now Remove the `FETCH_SINGLE_PAGE_STARS` from documentation for now since it might confuse people. * fix: fix error in automatic merge * feat: make sure env variable is read Co-authored-by: Matteo Pierro <[email protected]> Co-authored-by: Anurag <[email protected]> * chore: resolve conflict (anuraghazra#2453) * test: fix e2e tests * test: update snapshots * test: fix e2e test data * test: update e2e test data Co-authored-by: rickstaa <[email protected]> * feat: parse role parameter * fix: change prod deployment branch to vercel branch to fix maxDuration bug (anuraghazra#2424) * Create deploy-prep.yml * Create deploy-prep.py * Update vercel.json * Update deploy-prep.yml * Update vercel.json * Added coauthor Co-authored-by: Dou Xiaobo <[email protected]> * Update deploy-prep.yml * refactor: format code * Added if condition to disable deployments on forks Co-authored-by: Rick Staa <[email protected]> * Update deploy-prep.yml Co-authored-by: Dou Xiaobo <[email protected]> Co-authored-by: Anurag Hazra <[email protected]> Co-authored-by: rickstaa <[email protected]> * fix: fixes card overflow problem anuraghazra#2452 (anuraghazra#2460) This commit makes sure that the card width is formatted correctly. * fix: fix language card ownerAffiliation bug This commit fixes a small error in the stats-fetcher that caused the `exclude_repo` parameter values to be seen as the `ownerAffiliation` inputs. * ci: prevent certain actions from running on forks (anuraghazra#2466) * feat: add PAT monitoring functions (anuraghazra#2178) * feat: add PAT monitoring functions This commit adds two monitoring functions that can be used to check whether the PATs are functioning correctly: - status/up: Returns whether the PATs are rate limited. - status/pat-info: Returns information about the PATs. * feat: add shields.io dynamic badge json response This commit adds the ability to set the return format of the `/api/status/up` cloud function. When this format is set to `shields` a dynamic shields.io badge json is returned. * feat: add 'json' type to up monitor * feat: cleanup status functions * ci: decrease pat-info rate limiting time * feat: decrease monitoring functions rate limits * refactor: pat code * feat: add PAT monitoring functions This commit adds two monitoring functions that can be used to check whether the PATs are functioning correctly: - status/up: Returns whether the PATs are rate limited. - status/pat-info: Returns information about the PATs. * feat: add shields.io dynamic badge json response This commit adds the ability to set the return format of the `/api/status/up` cloud function. When this format is set to `shields` a dynamic shields.io badge json is returned. * feat: add 'json' type to up monitor * feat: cleanup status functions * ci: decrease pat-info rate limiting time * feat: decrease monitoring functions rate limits * refactor: pat code * test: fix pat-info tests * Update api/status/pat-info.js Co-authored-by: Anurag Hazra <[email protected]> * test: fix broken tests * chore: fix suspended account * chore: simplify and refactor * chore: fix test * chore: add resetIn field --------- Co-authored-by: Anurag <[email protected]> * chore: minor changes in pat info (anuraghazra#2481) * ci: add update languages action (anuraghazra#2484) * ci: add update languages action * ci: make sure PR is created when upstream languages are updated * Add `rose` theme (anuraghazra#2480) Co-authored-by: 강민지 <[email protected]> * Support hide_progress for top-langs feature (anuraghazra#2514) * Add support for hide_progress in top languages feature * Fix mistake * Add documents for all languages * Remove unnecessary value check * Update top-languages-card.js * Revert document for all languages except English * Update documentation * Update documentation --------- Co-authored-by: Zohan Subhash <[email protected]> * add holi_theme (anuraghazra#2539) * add holi_theme * add holi_theme * Add one_dark_pro (anuraghazra#2507) This colors were taken from One Dark Pro theme in VSCode extention. Please add it and I will use it in my profile Co-authored-by: Zohan Subhash <[email protected]> * fix: fix JSON themes bug. (anuraghazra#2544) This fixes a JSON bug that was introduced in anuraghazra#2507. * Add auto-labelling for documentation updates (anuraghazra#2526) * ci: fix unsafe directory bug (anuraghazra#2518) This commit fixes a bug that was introduced due to a upstream change in the git package. See https://stackoverflow.com/questions/71849415/i-cannot-add-the-parent-directory-to-safe-directory-in-git/71904131#71904131 for more information. * Update readme.md (anuraghazra#2414) fix: missing "&" in show_icons=true in Showing icons section * test: update snapshots (anuraghazra#2519) * fix: for issue anuraghazra#2534 (anuraghazra#2536) * Add option to deploy using other services (anuraghazra#2525) * Create express.js * Update readme.md * Update readme.md * ci: update e2e tests (anuraghazra#2548) * ci: fix a bug in the theme preview action (anuraghazra#2549) * refactor: format code (anuraghazra#2550) * refactor: update languages JSON (anuraghazra#2554) Co-authored-by: rickstaa <[email protected]> * Preview theme workflow fix (anuraghazra#2557) * Fix octokit error * ci: make octokit instance global --------- Co-authored-by: rickstaa <[email protected]> * ci: preview theme workflow fix (anuraghazra#2559) * Fix octokit error * ci: make octokit instance global * Fix preview theme (move declarations to global) * refactor: make constants uppercase --------- Co-authored-by: rickstaa <[email protected]> * refactor: fix code comments and change 'up' rate limit (anuraghazra#2560) * Preview action fix (anuraghazra#2561) * Fix error * refactor: remove unused code --------- Co-authored-by: Rick Staa <[email protected]> * ci: fix theme preview action (anuraghazra#2563) * fix theme preview (anuraghazra#2564) * ci: fix theme preview action * fix: fix some bugs in the 'theme-preveiw' action * ci: fixes theme preview action (anuraghazra#2566) * ci: improve theme preview action (anuraghazra#2572) * Added "Shadow" set (Red, Green, Blue, transparent BG) (anuraghazra#2529) * Added "Shadow" set (Red, Green, Blue, transparent BG) 3 additional themes sticking primarily to flat colors, which the exception of icons and border being slightly darker. All 3 themes also have transparent backgrounds that will show differently per-user via GiHub's own light and dark themes. Transparency should also still provide easy readability for both. * Test Just want to see if we can make the themes have a transparent background. * Shadows moved under Transparent --------- Co-authored-by: Zohan Subhash <[email protected]> * ci: fix theme docs generate bug (anuraghazra#2573) * update my theme (anuraghazra#2576) * refactor: update languages JSON (anuraghazra#2596) Co-authored-by: rickstaa <[email protected]> * Add format stats option (anuraghazra#2155) * feat: added `format_stats` option (anuraghazra#2128) * refactor: change `format_stats` to `short_values` (anuraghazra#2128) * test: create shorten values test (anuraghazra#2128) * Update readme.md Co-authored-by: Rick Staa <[email protected]> * refactor: rename ``short_values`` to ``number_format`` * Update readme.md Co-authored-by: Rick Staa <[email protected]> * Update src/cards/stats-card.js Co-authored-by: Rick Staa <[email protected]> * refactor: format codebase --------- Co-authored-by: Rick Staa <[email protected]> * add github_dark_dimmed theme (anuraghazra#2594) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank_icon option (anuraghazra#2628) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank icon option * feat(RankIcon): extract rankIcon to icons file * feat(RankIcon): update readme * feat(RankIcon): test coverage * Update readme.md Co-authored-by: Rick Staa <[email protected]> --------- Co-authored-by: Rick Staa <[email protected]> * Updated the custom error there was a Typo. (anuraghazra#2618) "Something went while trying to retrieve the stats data using the GraphQL API." -> "Something went wrong while trying to retrieve the stats data using the GraphQL API." * Fix typos inside wakatime test name (anuraghazra#2617) * removed redundant comparison (anuraghazra#957) * dev (anuraghazra#1782) * fix path to powered by vercel image (anuraghazra#1792) * Synchonize cache seconds min value inside docs translations with main readme (anuraghazra#2616) * New top language algorithm implementation (anuraghazra#1732) * Reduced vercel maxDuration * Implemented new algorithm for Top Langs * Revert "Reduced vercel maxDuration" This reverts commit b0bc626. * Added documentation * Fixed broken implementation * Update fetchTopLanguages.test.js Changed tests * Now uses the general formula The parameters p and q can be set by the user. * Updated tests and added new test * Added new test New test for order by repo count. * Updated documentation Added explanation and examples for new options. * Updated documentation This was overwritten in the merge commit. * docs: improve docs and fix tests * Renamed parameters Renamed `p` and `q` to `size_weight` and `count_weight`, respectively. * Updated the documentation Changes introduced in f2516d6 --------- Co-authored-by: rickstaa <[email protected]> * feat: improve CONTRIBUTING.md (anuraghazra#2609) * docs: update give logo (anuraghazra#2605) * Fixed typo in word color inside README (anuraghazra#2652) * Fixed docs typo inside src/common/utils.js (anuraghazra#2651) * Fixed several typos inside preview theme script (anuraghazra#2650) * Fixed todo inside stats card data fetcher (anuraghazra#2649) * docs: fix README language card usage order (anuraghazra#2659) * Fixed vscode type error in get card colors function (anuraghazra#2660) * Show no activity error in compact layout of wakatime card (anuraghazra#2662) * Fixed card constructor docstring to resolve vscode type errors (anuraghazra#2674) * Fixed wakatime create text node func docstring to resolve vscode type errors (anuraghazra#2678) * Fixed docstring for several wakatime card gen functions to resolve multiple type errors (anuraghazra#2690) * Fixed docstring for stats card create text node function to resolve type errors (anuraghazra#2689) * Update codecov action version to resolve github action deprecation warning (anuraghazra#2677) * Add PRs cache cleaning workflow (anuraghazra#2661) * build(deps): bump yaml from 2.1.3 to 2.2.2 (anuraghazra#2644) Bumps [yaml](https://github.com/eemeli/yaml) from 2.1.3 to 2.2.2. - [Release notes](https://github.com/eemeli/yaml/releases) - [Commits](eemeli/yaml@v2.1.3...v2.2.2) --- updated-dependencies: - dependency-name: yaml dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix(wakatime-fetcher): add default range to prevent user not found error (anuraghazra#2670) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank icon option * feat(RankIcon): extract rankIcon to icons file * feat(RankIcon): update readme * feat(RankIcon): test coverage * Update readme.md Co-authored-by: Rick Staa <[email protected]> * add wakatime default range * update wakatime demos in readme with most active user * update wakatime demo in readme * remove &range=all_time from wakatime demos in readme --------- Co-authored-by: Rick Staa <[email protected]> * Cover with test changes in anuraghazra#2662 pull request (anuraghazra#2673) * test: fix mistaken pageInfo.endCursor keys (anuraghazra#2657) The previous mock logic was too simplistic and has been fixed in the commit. If the mock logic had been properly implemented, then the mistaken pageInfo.endCursor keys should have made the test case "should fetch two pages of stars if 'FETCH_MULTI_PAGE_STARS' env variable is set to `true`" stuck. * docs: update wakatime user (anuraghazra#2697) The old example user doesn't have a WakaTime account anymore. * Fixed docstring for get styles function to resolve vscode type errors (anuraghazra#2700) * Fixed axios imports inside docstrings to resolve vscode type errors (anuraghazra#2699) * Fixed typos inside bug report issue template (anuraghazra#2706) * Update wakatime user inside docs translations (anuraghazra#2705) * Fixed docstring for create language text node function (anuraghazra#2704) * Fixed docstring for wakatime card recalculate percentages function (anuraghazra#2710) * add pie chart layout to language card (anuraghazra#2099) * add pie chart layout to language card * resolve failing top-lang card tests * scale down pie chart * update readme.md * Update readme.md Co-authored-by: Rick Staa <[email protected]> * style: format code * update donut layout to be created without dependencies * minor update * style: format readme * resolve failing tests * refactor: clean up code and add extra tests This commit cleans up the pie chart generation code and adds additional tests. * feat: improve pie chart positioning * rename layout pie to donut * add animation to donut layout * refactor: rename pie and doughnut to donut * feat: decrease donus animation delay --------- Co-authored-by: rickstaa <[email protected]> * Fixed docstring for create donut language node function (anuraghazra#2713) * Add missing argument to create compact lang node function to resolve vscode type error (anuraghazra#2714) * Top languages card pie layout (anuraghazra#2709) * Top languages card donut layout * Top languages card pie layout * renames * dev * docs * dev * dev * animations * dev * handle one language * Docs: add missing top languages card layouts. (anuraghazra#2717) * Top languages card render test: move repeated code into helper function (anuraghazra#2718) * Docs (translations): add missing top languages card layouts. (anuraghazra#2728) * Top languages card donut vertical layout (anuraghazra#2701) * Top languages card donut layout * dev * dev * dev * dev * Docs (translations): add top languages card donut vertical layout (anuraghazra#2739) * Docs: fix top languages card available layouts number (anuraghazra#2738) * Ranking System v2 (anuraghazra#1186) * Revise rank calculation * Replace contributions by commits * Lower average stats and S+ threshold * Fix calculateRank.test.js Missing key in dictionary constructor Co-authored-by: Rick Staa <[email protected]> * refactor: run prettier * feat: change star weight to 0.75 * Separate PRs and issues * Tweak weights * Add count_private back * fix: enable 'count_private' again * test: fix tests * refactor: improve code formatting * Higher targets --------- Co-authored-by: Rick Staa <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Raymond Nook <[email protected]> Co-authored-by: rickstaa <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Huzaifa Khilawala <[email protected]> Co-authored-by: Anurag Hazra <[email protected]> Co-authored-by: Zohan Subhash <[email protected]> Co-authored-by: Matteo Pierro <[email protected]> Co-authored-by: Dou Xiaobo <[email protected]> Co-authored-by: 강민지 <[email protected]> Co-authored-by: 강민지 <[email protected]> Co-authored-by: Amir <[email protected]> Co-authored-by: Mohamed Hassan <[email protected]> Co-authored-by: Oleksandr Perlov <[email protected]> Co-authored-by: Cateline Mnemosyne <[email protected]> Co-authored-by: Raphael Gonçalves <[email protected]> Co-authored-by: Rehman <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: rickstaa <[email protected]> Co-authored-by: Etanarvazac Revorix <[email protected]> Co-authored-by: Eduardo Zaniboni <[email protected]> Co-authored-by: Caeden Perelli-Harris <[email protected]> Co-authored-by: Fabiano Couto <[email protected]> Co-authored-by: LOKESH SINGH <[email protected]> Co-authored-by: Alexandr Garbuzov <[email protected]> Co-authored-by: kitswas <[email protected]> Co-authored-by: Rongrong <[email protected]> Co-authored-by: Nabil Alamin <[email protected]> Co-authored-by: Alexandr Garbuzov <[email protected]> Co-authored-by: François Rozet <[email protected]>

kitswas changed the title ~~Kitswas new top language detection algorithm~~ New top language algorithm implementation May 3, 2022

rickstaa added the enhancement New feature or request. label May 3, 2022

kitswas marked this pull request as draft May 4, 2022 13:22

kitswas force-pushed the kitswas-New-Top-Language-Detection-Algorithm branch from 10be39a to 1fef47b Compare May 4, 2022 14:36

kitswas marked this pull request as ready for review May 4, 2022 14:43

rickstaa requested changes May 7, 2022

View reviewed changes

src/fetchers/top-languages-fetcher.js Outdated Show resolved Hide resolved

rickstaa mentioned this pull request May 7, 2022

[Proposal] New Top Language Detection Algorithm #1600

Closed

kitswas requested a review from rickstaa May 7, 2022 12:52

Repository owner deleted a comment from kitswas May 8, 2022

rickstaa approved these changes May 8, 2022

View reviewed changes

rickstaa added the popular label Jun 10, 2022

rickstaa mentioned this pull request Jun 10, 2022

Customization of ownerAffiliations with parameter #1122

Draft

This was referenced Jul 6, 2022

Top Languages card results are incorrect #1801

Open

[Feature Request] Add organization stats #1

Open

rickstaa removed the popular label Sep 22, 2022

kitswas added 4 commits September 30, 2022 21:16

Reduced vercel maxDuration

22adf69

Implemented new algorithm for Top Langs

42da11b

Revert "Reduced vercel maxDuration"

407406f

This reverts commit b0bc626.

Added documentation

576fd51

rickstaa force-pushed the master branch from 86aafe8 to 8bc69e7 Compare January 21, 2023 16:47

kitswas added 2 commits April 19, 2023 21:57

Merge branch 'master' into kitswas-New-Top-Language-Detection-Algorithm

5e24e98

Updated documentation

aac13f5

This was overwritten in the merge commit.

github-actions bot added the documentation Improvements or additions to documentation. label Apr 19, 2023

rickstaa approved these changes Apr 24, 2023

View reviewed changes

rickstaa requested review from anuraghazra and Zo-Bro-23 and removed request for rickstaa April 24, 2023 13:41

docs: improve docs and fix tests

9ce9bde

rickstaa force-pushed the kitswas-New-Top-Language-Detection-Algorithm branch from 44b6ada to 9ce9bde Compare April 24, 2023 13:41

rickstaa mentioned this pull request Apr 24, 2023

Cached language stats #2638

Closed

kitswas added 2 commits April 25, 2023 09:51

Renamed parameters

f2516d6

Renamed `p` and `q` to `size_weight` and `count_weight`, respectively.

Updated the documentation

7aa3546

Changes introduced in f2516d6

rickstaa approved these changes Apr 25, 2023

View reviewed changes

rickstaa merged commit 5577bbf into anuraghazra:master Apr 25, 2023

kitswas deleted the kitswas-New-Top-Language-Detection-Algorithm branch April 25, 2023 06:38

Uh oh!

New top language algorithm implementation #1732

New top language algorithm implementation #1732

Uh oh!

Conversation

kitswas commented May 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Old Card

New Card

Uh oh!

vercel bot commented May 3, 2022

Uh oh!

kitswas commented May 5, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

anuraghazra commented May 5, 2022

Uh oh!

rickstaa commented May 7, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kitswas commented May 7, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rickstaa commented May 7, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

rickstaa commented May 7, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kitswas commented May 7, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rickstaa commented May 8, 2022

Uh oh!

rickstaa commented May 8, 2022

Uh oh!

kitswas commented Apr 19, 2023

Uh oh!

rickstaa commented Apr 24, 2023

Uh oh!

codecov bot commented Apr 24, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

kitswas commented Apr 24, 2023

Uh oh!

kitswas commented Apr 24, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rickstaa commented Apr 24, 2023

Uh oh!

kitswas commented Apr 25, 2023

Uh oh!

rickstaa commented Apr 25, 2023

Uh oh!

Uh oh!

kitswas commented May 3, 2022 •

edited

Loading

kitswas commented May 5, 2022 •

edited

Loading

rickstaa commented May 7, 2022 •

edited

Loading

kitswas commented May 7, 2022 •

edited

Loading

rickstaa commented May 7, 2022 •

edited

Loading

rickstaa commented May 7, 2022 •

edited

Loading

kitswas commented May 7, 2022 •

edited

Loading

codecov bot commented Apr 24, 2023 •

edited

Loading

kitswas commented Apr 24, 2023 •

edited

Loading