improve scripts

* move more generic code handling data to data.py
* fix markdown generation in generate_deck.py
* make gen_hoverable_link more flexible by introducing callbacks
  to generate the relevant link components.
* fix the link targets for hoverable links in the generated card markdown

1 files changed, 7 insertions, 4 deletions

diff --git a/scripts/generate_card.py b/scripts/generate_card.py
index 973377a1..6500a480 100755
--- a/scripts/generate_card.py
+++ b/scripts/generate_card.py
@@ -1,4 +1,6 @@
 #!/usr/bin/env python3
+"""Generate latex or markdown from yaml card definitions"""
+
 import argparse
 import pathlib
 from pathlib import Path
@@ -124,11 +126,11 @@ def get_latex_field(card: MutableMapping,
     # latex_* values are just concatenated.
 
     # NOTE: it is allowed for a latex_ field to be sparse.
-    # Missing entries in a latex_ field list are populated from the corrsponding
+    # Missing entries in a latex_ field list are populated from the corresponding
     # entry of the normal field
     assert len(latex_value) > 1
-    for i in range(0, len(latex_value)):
-        if latex_value[i] is None:
+    for i, l_val in enumerate(latex_value):
+        if l_val is None:
             latex_value[i] = f'{fmt_prefix}{normal_value[i]}// '
 
     return ''.join(latex_value), True
@@ -149,7 +151,8 @@ def generate_markdown(card: MutableMapping, language='en', indentation=3):
     gen_png_link_text = lambda _: 'png'
     gen_png_link_target = lambda c, l: f'{built_card_path}.png'
     png_hover_link = generate_card_hover_links.gen_hoverable_link(
-        str(file_name), gen_png_link_text, gen_png_link_target, language)
+        str(file_name), gen_png_link_text, gen_png_link_target,
+        '../latex-build', language)
 
     print(
         f'\n[pdf]({built_card_path}.pdf) {png_hover_link} [yml]({yml_card_path})\n'