-
Notifications
You must be signed in to change notification settings - Fork 129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
either characteristic or equality test in localizations of quotients is lying #4324
Comments
I guess the multiplicative set contains zero, so the characteristic is wrong. |
yeah, this method is overly optimistic: characteristic(W::AbsLocalizedRing) = characteristic(base_ring(W)) |
Fixing it means that we will see that during the Oscar tests, a multivariate polynomial ring over a genuine zero ring is constructed. More precisely, this happens in the
we need to discuss how to deal with this:
|
Sorry for not getting back to this yet! I didn't have this on the radar when implementing |
Stupid question: Shouldn't the localization know about an element of the modulus being inverted, when it creates the localized ring? |
I think, it does not make sense to have the constructor look for this, really. Why would we do this extra check?
At any point one can check whether or not a ring is the zero ring and get the correct answer. If this can cheaply be cached, then one could do it, e.g. by setting an attribute. But even if one doesn't, one can still do the check with no harm. |
@HechtiDerLachs For the ring
I get
Any idea what could be wrong?
P.S.: I got this from an example in the geometry tests, while looking for usages of the zero ring.
The text was updated successfully, but these errors were encountered: