Commit 41b6d824 authored by Manuel Schrauth's avatar Manuel Schrauth
Browse files

Merge branch '52-kerneldunham-doesn-t-know-vertex-centered-tilings' into 'master'

Resolve "Kerneldunham doesn't know vertex-centered tilings"

Closes #52

See merge request !42
parents 5fe40b4a f23de35a
Pipeline #15739 passed with stages
in 14 minutes and 28 seconds
......@@ -39,5 +39,8 @@ def HyperbolicTiling(p, q, n, center="cell", kernel="flo"):
if kernel not in kernels:
raise KeyError("[hypertiling] Error: No valid kernel specified")
if kernel == "dunham":
raise NotImplementedError("[hypertiling] Error: Dunham kernel is currently broken (fixme!)")
print("Caution: This kernel is slow and error-prone. Use at own risk!")
if center == "vertex":
print("KernelDunham doesn't support vertex-centered tilings yet. A cell-centered tiling will be generated instead...")
# raise NotImplementedError("[hypertiling] Error: Dunham kernel is currently broken (fixme!)")
return kernels[kernel](p, q, n, center)
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment