Skip to content
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

polynomial_ring cached=false for tropical geometry #3858

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ederc
Copy link
Member

@ederc ederc commented Jun 13, 2024

See #2455.

Tests need to be adjusted, maybe @YueRen can provide some help.

@ederc ederc marked this pull request as draft June 13, 2024 11:08
@YueRen
Copy link
Member

YueRen commented Jun 13, 2024

@ederc Thanks for the changes, much appreciated! I will look over them tomorrow.

@YueRen
Copy link
Member

YueRen commented Jun 18, 2024

(This is more a memo to myself, though comments are welcome) Okay, here is why the change is problematic and what I plan to do to fix it:

Consider a polynomial ring K[x1,..,xn] over a valued field K. In addition to the original polynomial ring K[x1,..,xn], computations in tropical geometry require

  • a polynomial ring R[x1,...,xn] over the valuation ring R (for tropical Groebner bases)
  • a polynomial ring k[x1,...,xn] over the residue field field k (this is where initial ideals live)

Currently, I just (re-)create R[x1,...,xn] and k[x1,...,xn] from K[x1,...,xn] and blindly trust that it always ends up being the same polynomial ring. This is probably not very smart.

What should happen is that K[x1,..,xn] should store R[x1,..,xn] and k[x1,..,xn] as attributes. This will require lots of changes, so it might take a bit of time (ETA August).

@fingolfin
Copy link
Member

Why should K[x1,..,xn] store R[x1,..,xn] and k[x1,..,xn] as attributes? That would be a bit unusual. Perhaps it is unavoidable for some reason, but normally I would expect that there is some kind of parent object (perhaps a "tropical variety" or something like that) which contains these rings.

I had a quick look at the changes in this PR and the tropical code in general but it wasn't apparent to me at all how and when these

@YueRen
Copy link
Member

YueRen commented Jun 25, 2024

Why should K[x1,..,xn] store R[x1,..,xn] and k[x1,..,xn] as attributes? That would be a bit unusual. Perhaps it is unavoidable for some reason, but normally I would expect that there is some kind of parent object (perhaps a "tropical variety" or something like that) which contains these rings.

I had a quick look at the changes in this PR and the tropical code in general but it wasn't apparent to me at all how and when these

I should probably save it in TropicalSemiringMap then. TropicalSemiringMap's main job is to encode a valuation on K and a choice between min-convention and max-convention, and it is always passed along in tropical (algebraic) computations. It could also store a dictionary, which maps K[x] to R[x] and k[x] (in case multiple polynomial rings have the same K).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants