Merge

Author: roed314

lmfdb/groups/abstract/main.py

@@ -1969,14 +1969,17 @@ def render_abstract_group(label, data=None):
 
     bread = get_bread([(gp.label_compress(), "")])
     learnmore_gp_picture = ('Picture description', url_for(".picture_page"))
+    code = gp.code_snippets()
+    if code and len(code['prompt']) == 0:  # no codes
+        code = None
 
     return render_template(
         "abstract-show-group.html",
         title=title,
         bread=bread,
         info=info,
         gp=gp,
-        code=gp.code_snippets(),
+        code=code,
         properties=gp.properties(),
         friends=friends,
         learnmore=learnmore_list_add(*learnmore_gp_picture),

lmfdb/groups/abstract/templates/abstract-show-group.html

@@ -22,7 +22,9 @@
   </p>
 {% endif %}
 
-<p>{{ place_code('code_description', is_top_snippet=True) }}</p>
+<p>
+{{ place_code('code_description', is_top_snippet=True) }}
+</p>
 
 <h2>Group information</h2>
 <p>
@@ -204,7 +206,9 @@ <h2>Minimal presentations</h2>
 {% if not (gp.live() and gp.abelian) %}
 
 <h2>Constructions</h2>
-  {% if code %}<div align="right" style="float: left;">
+
+{% if code %}
+<div align="right" style="float: left;">
   {{CodeSnippet(code).show_commands_box() | safe}}
 </div><br><br>
 {% endif %}

lmfdb/groups/abstract/web_groups.py

@@ -2999,7 +2999,7 @@ def code_snippets(self):
                 if lang not in code['code_description']:
                     code['code_description'][lang] = 'G := SmallGroup('+gap_id[0]+', '+gap_id[1]+');'
             if 'sage_gap' not in code['code_description']:
-                code['code_description']['sage_gap'] = 'G = gap.SmallGroup('+gap_id[0]+', '+gap_id[1]+')'
+                code['code_description']['sage_gap'] = 'G = libgap.SmallGroup('+gap_id[0]+', '+gap_id[1]+')'
         # Otherwise, check if group is abelian (then can define as product of cyclic groups from its primary decomposition)
         if self.abelian:
             for lang in ['magma', 'gap']:
@@ -3033,30 +3033,35 @@ def code_snippets(self):
         # then unfortunately not all the default code snippets in Sage and Magma will work correctly!
         # As a (hopefully temporary) solution, we hide the code snippets which will not work with our implemention of G in the top code snippet
         # TODO: Find a better solution for this (would it be worth converting G to a permutation group within the top code snippet?)
-        if ("MatrixGroup" in code['code_description']['sage']) and ("permutation" not in code['code_description']['sage']):
+        if 'sage' in code['code_description'] and "MatrixGroup" in code['code_description']['sage'] and "permutation" not in code['code_description']['sage']:
             # Must disable all code snippets which do not work with MatrixGroup in Sage
             for c in ['composition_factors', 'is_cyclic', 'is_elementary_abelian', 'is_pgroup', 'abelianization', 'schur_multiplier',
                       'commutator', 'frattini_subgroup', 'fitting_subgroup', 'socle', 'derived_series', 'lower_central_series', 'upper_central_series']:
                 if c in code:
                     code[c].pop('sage', None)
-        if ("AbelianGroup" in code['code_description']['sage']) and ("permutation" not in code['code_description']['sage']):
+        if 'sage' in code['code_description'] and "AbelianGroup" in code['code_description']['sage'] and "permutation" not in code['code_description']['sage']:
             # Must disable all code snippets which do not work with AbelianGroup in Sage
             for c in ['composition_factors', 'is_elementary_abelian', 'is_nilpotent', 'is_perfect', 'is_pgroup', 'is_polycyclic', 'is_solvable', 'is_supersolvable',
                       'abelianization', 'schur_multiplier', 'center', 'commutator', 'frattini_subgroup', 'fitting_subgroup', 'socle',
                       'derived_series', 'lower_central_series', 'upper_central_series']:
                 if c in code:
                     code[c].pop('sage', None)
-        if ("MatrixGroup" in code['code_description']['magma']) and ("permutation" not in code['code_description']['magma']):
+        if 'magma' in code['code_description'] and "MatrixGroup" in code['code_description']['magma'] and "permutation" not in code['code_description']['magma']:
             # Must disable all code snippets which do not work with MatrixGroup in Magma
             for c in ['socle']:
                 if c in code:
                     code[c].pop('magma', None)
-        if ("AbelianGroup" in code['code_description']['magma']) and ("permutation" not in code['code_description']['magma']):
+        if 'magma' in code['code_description'] and "AbelianGroup" in code['code_description']['magma'] and "permutation" not in code['code_description']['magma']:
             # Must disable all code snippets which do not work with AbelianGroup in Magma
             for c in ['radical', 'socle']:
                 if c in code:
                     code[c].pop('magma', None)
 
+        # Hides code snippets for language lang if no top code snippet has been defined for lang
+        for lang in list(code['prompt']):
+            if lang not in code['code_description']:
+                code['prompt'].pop(lang, None)
+
         for prop in code:
             for lang in code[prop]:
                 code[prop][lang] = code[prop][lang].format(**data)