Skip to content

Commit d3ca875

Browse files
roed314claude
andcommitted
CI fixes for shimura_curves: lint, template collision, test wiring
- web_curve.py: drop f-prefix on placeholder-less f-string (pyflakes) - rename lat_diagram_page.html -> shimcurve_lat_diagram_page.html (collided with modular_curves' template of the same name in the shared jinja namespace) - test_home: browse page notation changed from X_0(N) to X(D;N) - workflow: test-file count 44 -> 45; run shimura tests on devmirror in CI Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 2bb5c48 commit d3ca875

6 files changed

Lines changed: 9 additions & 4 deletions

File tree

.github/workflows/matrix_includes.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,5 +108,10 @@
108108
"files": "lmfdb/modular_curves/test_modular_curves.py",
109109
"folders": "modular_curves",
110110
"server": "devmirror"
111+
},
112+
{
113+
"files": "lmfdb/shimura_curves/test_shimura_curves.py",
114+
"folders": "shimura_curves",
115+
"server": "devmirror"
111116
}
112117
]

.github/workflows/python-package.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ jobs:
9090
- name: checking that we didn't miss any test files
9191
shell: bash -l {0}
9292
# If this fails you need to update the file list above and file count
93-
run: test $(find lmfdb -name 'test_*.py' -or -name '*_test.py' | wc -l) -eq 44
93+
run: test $(find lmfdb -name 'test_*.py' -or -name '*_test.py' | wc -l) -eq 45
9494

9595
- name: Config LMFDB to run tests against proddb
9696
if: matrix.files != 'lint' && matrix.server == 'proddb'

lmfdb/shimura_curves/main.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ def lat_diagram(label):
162162
info = {"dojs": dojs}
163163
info.update(display_opts)
164164
return render_template(
165-
"lat_diagram_page.html",
165+
"shimcurve_lat_diagram_page.html",
166166
dojs=dojs,
167167
info=info,
168168
title="Diagram of nearby Shimura curves for %s" % label,

lmfdb/shimura_curves/templates/lat_diagram_page.html renamed to lmfdb/shimura_curves/templates/shimcurve_lat_diagram_page.html

File renamed without changes.

lmfdb/shimura_curves/test_shimura_curves.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ def test_home(self):
99
assert 'Browse' in L.get_data(as_text=True)
1010
assert 'Search' in L.get_data(as_text=True)
1111
assert 'Find' in L.get_data(as_text=True)
12-
assert 'X_0(N)' in L.get_data(as_text=True)
12+
assert 'X(D;N)' in L.get_data(as_text=True)

lmfdb/shimura_curves/web_curve.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -938,7 +938,7 @@ def rational_points_description(self):
938938
elif curve.genus > 1 or (curve.genus == 1 and curve.rank == 0):
939939
desc = "This Shimura curve has finitely many rational points."
940940
else:
941-
desc = fr'Local obstructions for rational points on this curve are not known.'
941+
desc = 'Local obstructions for rational points on this curve are not known.'
942942
if (self.genus > 1 or self.genus == 1 and self.rank == 0) and self.db_rational_points:
943943
desc += " The following are the known rational points on this Shimura curve (one row per $j$-invariant)."
944944
return desc

0 commit comments

Comments
 (0)