Spec: Introduce Community Health Score panel - #4367
Open
functionzz wants to merge 24 commits into
Open
Conversation
mathjazz
changed the title
mathjazz
changed the title
mathjazz
changed the title
mathjazz
reviewed
Aug 5, 2026
Collaborator
There was a problem hiding this comment.
This sadly still needs quite a lot of work.
Lots of content is out of date and incoherent.
I would also find it much easier to read in prose, than in several nesting levels of bullet points. See other specs for reference.
Should we cover the the report system (PR #4336) here as well?
Please also add screenshots.
mathjazz
changed the title
Collaborator
Author
|
Phew, lots of reviews :/
Let's not do that, I was thinking of making a seperate spec for that |
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
mathjazz
reviewed
Aug 6, 2026
Comment on lines
+28
to
+34
| - `active_managers_score`: score representation of `active_managers` | ||
| - `active_translators_score` score representation of `active_translators` | ||
| - `active_contributors_score` score representation of `active_contributors` | ||
| - `all_contributors_score` score representation of `all_contributors` | ||
| - `new_signups_score` score representation of `new_signups` | ||
| - `key_projects_enabled_score` score representation of `key_projects_enabled` | ||
| - `completion_score` score representation of `completion` |
Collaborator
There was a problem hiding this comment.
This paragraphs doesn't add much value. Instead, we should explain the formula for calculating each of these scores.
Think of the spec as the containing sufficient information to develop the feature. It will also be helpul in the future when we'll need to understand how the score works, so we don't need to decypher the code.
| - `key_projects_enabled_score` score representation of `key_projects_enabled` | ||
| - `completion_score` score representation of `completion` | ||
|
|
||
| Finally the scores are combined into a **Community health score** for that locale on that month which is used for measuring locale health. |
Collaborator
There was a problem hiding this comment.
How excatly (mathematically) are they combined?
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
mathjazz
reviewed
Aug 7, 2026
mathjazz
left a comment
mathjazz
left a comment
Collaborator
There was a problem hiding this comment.
Left some more inline comments.
We should fix that reference to the "following scores" ans "thresholds" and we can land this.
| - `key_projects_enabled`: number of projects that are classified as key projects via the `is_chs_project` flag | ||
| - `completion`: locale completion, as a percentage of (**approved strings** + **strings with warnings**) out of **total strings** | ||
|
|
||
| The data used is then computed into the following scores based on predetermined thresholds. Finally the scores are summated to a **Community health score** for that locale on that month which is used for measuring locale health. |
Collaborator
There was a problem hiding this comment.
What following scores and what thresholds?
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
Co-authored-by: Matjaž Horvat <matjaz.horvat@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR copies internal specs for the Insights dashboard for github access.