M2 icon indicating copy to clipboard operation
M2 copied to clipboard

add CoefficientRing to smoothFanoVariety(d,0)

Open Devlin-Mallory opened this issue 1 year ago • 2 comments

In NormalToricVarieties, smoothFanoToricVariety(d,i) took a CoefficientRing option, which was previously erroneously ignored when i = 0 (i.e., for P^d). This just adds CoefficientRing => opts.CoefficientRing in that case.

Devlin-Mallory avatar May 22 '24 23:05 Devlin-Mallory

Why not also pass the variable option? You can just pass opts which includes both.

mahrud avatar May 23 '24 00:05 mahrud

Why not also pass the variable option? You can just pass opts which includes both.

Ah, good call - added that now.

Devlin-Mallory avatar May 23 '24 02:05 Devlin-Mallory