From 13aaa42908435f691299e5873b738eaaf8752a33 Mon Sep 17 00:00:00 2001 From: Jean-Baptiste Holcroft Date: Sep 09 2023 09:16:19 +0000 Subject: website: display progress as percents let's use human percents everywhere --- diff --git a/build_stats.py b/build_stats.py index 8a6f471..afa5305 100755 --- a/build_stats.py +++ b/build_stats.py @@ -53,12 +53,21 @@ def compute_language_statistics(languages_stats: dict, total_release_source_word result["totalsourcewordssum"] = result["translatedsourcewords"] + result["fuzzysourcewords"] + result[ "untranslatedsourcewords"] result["totalsourcewords_d"] = total_release_source_words - result["progress"] = result["translatedsourcewords"] / result["totalsourcewordssum"] - result["progress_d"] = result["translatedsourcewords"] / result["totalsourcewords_d"] + + # prevent a Runtime warning for languages with no content + if result["totalsourcewordssum"] > 0: + result["progress"] = (result["translatedsourcewords"] / result["totalsourcewordssum"]) * 100 + else: + result["progress"] = 0.0 + + result["progress_d"] = (result["translatedsourcewords"] / result["totalsourcewords_d"]) * 100 packages_stats = df[package_fields].groupby("package").sum() packages_stats["totalsourcewordssum"] = packages_stats["translatedsourcewords"] + packages_stats["fuzzysourcewords"] + packages_stats["untranslatedsourcewords"] - packages_stats["progress"] = packages_stats["translatedsourcewords"] / packages_stats["totalsourcewordssum"] + + packages_stats["progress"] = (packages_stats["translatedsourcewords"] / packages_stats["totalsourcewordssum"]) * 100 + # prevent NaN values when a package have total source words = 0 + packages_stats.fillna(0, inplace=True) packages_stats["team"] = df[["metadata_language_team", "package"]].groupby("package").first() result["packages"] = packages_stats.to_dict(orient="index") @@ -85,7 +94,9 @@ def compute_package_statistics(df: pd.DataFrame) -> dict: stats = df[po_fields].groupby(index).sum() stats["totalsourcewordssum"] = stats["translatedsourcewords"] + stats["fuzzysourcewords"] + stats["untranslatedsourcewords"] - stats["progress"] = stats["translatedsourcewords"] / stats["totalsourcewordssum"] + stats["progress"] = (stats["translatedsourcewords"] / stats["totalsourcewordssum"]) * 100 + # prevent NaN values when a package have total source words = 0 + stats.fillna(0, inplace=True) stats["team"] = df[["metadata_language_team", index]].groupby(index).first() df['filename'] = df.index stats["filename"] = df[["filename", index]].groupby(index).sum() diff --git a/templates/language.adoc b/templates/language.adoc index 68af78e..55e6b33 100644 --- a/templates/language.adoc +++ b/templates/language.adoc @@ -17,8 +17,8 @@ territories: Language progress for {{ lang_name_en }} ({{ lang_code }}) in Fedora {{ results }} is: -* {{ '{:.2f}'.format(progress) }} when we only look on started packages for this language. -* {{ '{:.2f}'.format(progress_d) }} when we compare to every single translatable string in Fedora {{ results }}. +* {{ '{:.2f}'.format(progress) }}% when we only look on started packages for this language. +* {{ '{:.2f}'.format(progress_d) }}% when we compare to every single translatable string in Fedora {{ results }}. Possible scripts are: {% for script in scripts -%}{{ script }} {%- endfor %}