fix: no duplicates use random.shuffle instead of random.choices

.github/config.ini

@@ -4,3 +4,5 @@ download = https://download-directory.github.io
 spacing = 1
 choose = 3
 dry = false
+browse = true
+exclude = tile:devicons

.github/docgen.py

@@ -5,7 +5,7 @@
 from os import listdir
 from os.path import isfile
 from pathlib import Path
-from random import choices
+from random import shuffle
 from typing import Callable
 
 
@@ -15,13 +15,14 @@ def get_config(config_path: Path = Path("./.github/config.ini")) -> dict[str, st
     return dict(parser.defaults())
 
 
-def categorical_wallpapers() -> dict[str, list[Path]]:
+def categorical_wallpapers(exclude: str | list[str] = []) -> dict[str, list[Path]]:
+    exclude = exclude.split(":") if type(exclude) is str else exclude
     return {
         # exclude categorical README.md
         str(category): [Path(picture) for picture in listdir(category) if picture != "README.md"]
         # exclude hidden directories and README.md
         for category in listdir(".")
-        if not category.startswith(".") and not isfile(category)
+        if not category.startswith(".") and not isfile(category) and category not in exclude
     }
 
 
@@ -31,9 +32,14 @@ def get_templates() -> dict[str, str]:
 
 def generate_shuffled(
     config: dict[str, str],
-    categories: dict[str, list[Path]] = categorical_wallpapers()
+    categories: dict[str, list[Path]],
 ) -> dict[str, list[Path]]:
-    return {category: choices(pictures, k=int(config["choose"])) for category, pictures in categories.items()}
+    results = {}
+    choose = int(config["choose"])
+    for category, pictures in categories.items():
+        shuffle(pictures)
+        results[category] = pictures[:choose]
+    return results
 
 
 def prime_templates(
@@ -47,23 +53,21 @@ def prime_templates(
     }
 
 
-def transform_shuffled(category: str, shuffled_paths: list[Path]) -> dict[str, str]:
-    results = {}
-    for index in range(len(shuffled_paths)):
-        results[f"shuffled_{index}"] = str(f"../{category}" / shuffled_paths[index])
-        results[f"shuffled_{index}_stem"] = shuffled_paths[index].stem
-    return results
-
-
 # Handlers {{{
 def handle_body(_, string: str, config: dict[str, str]) -> str:
-    shuffled = generate_shuffled(config)
+    shuffled = generate_shuffled(config, categorical_wallpapers(config["exclude"]))
     results = []
+    spacing = "\n" * int(config["spacing"])
     for category, pictures in shuffled.items():
-        merged = {"category": category}
-        merged = merged | config | transform_shuffled(category, pictures)
-        results.append(string.format(**merged))
-    return ("\n" * int(config["spacing"])).join(results)
+        merged = {"category": category} | config
+        results.append(f"## {category}{spacing}")
+        for picture in pictures:
+            merged["random"] = str(picture)
+            merged["random_stem"] = picture.stem
+            results.append(string.format(**merged))
+        if config["browse"].casefold() == "True".casefold():
+            results.append(f"[Browse](../{category}/README.md){spacing}")
+    return spacing.join(results)
 
 
 def handle_category(_, string: str, config: dict[str, str]) -> dict[str, str]:
@@ -76,8 +80,6 @@ def handle_category(_, string: str, config: dict[str, str]) -> dict[str, str]:
             merged = config | {"filepath": str(picture), "filename": picture.stem}
             results[readme] = f"{results[readme]}{string.format(**merged)}{spacing}"
     return results
-
-
 # }}}
 
 

.github/templates/body.category.md

@@ -1,9 +1 @@
-## {category}
-
-<a href="{shuffled_0}"><img alt="{shuffled_0_stem}" src="{shuffled_0}"></a><br/><br/>
-
-<a href="{shuffled_1}"><img alt="{shuffled_1_stem}" src="{shuffled_1}"></a><br/><br/>
-
-<a href="{shuffled_2}"><img alt="{shuffled_2_stem}" src="{shuffled_2}"></a><br/><br/>
-
-[Browse](../{category}/README.md)
+<a href="../{category}/{random}"><img alt="{random_stem}" src="../{category}/{random}"></a><br/><br/>