id summary reporter owner description type status priority milestone component resolution keywords cc merged author reviewer upstream work_issues branch commit dependencies stopgaps
21869 A framework for discrete valuations in Sage Julian Rüth "This is a meta-ticket to keep track of the progress of integrating https://github.com/saraedum/mac_lane into Sage.
= Review =
For your convenience you can review this ticket at https://github.com/saraedum/mac_lane/pull/4 (and leave inline comments.)
Please check off `[x]` the following when you think that a file is in good shape (modulo the comments that you made.) Or put a `[-]` if it needs substantial work. You can put your name next to file to tell others that you are already having a look at it.
{{{
[ ] function_field/function_field_valuation.py
[ ] padics/discrete_value_group.py
[ ] padics/padic_valuation.py
[-] valuation/README.md
[x] valuation/__init__.py
[ ] valuation/all.py
[ ] valuation/augmented_valuation.py
[ ] valuation/developing_valuation.py
[ ] valuation/gauss_valuation.py
[ ] valuation/inductive_valuation.py
[ ] valuation/limit_valuation.py
[ ] valuation/mapped_valuation.py
[ ] valuation/scaled_valuation.py
[ ] valuation/trivial_valuation.py
[ ] valuation/valuation.py
[ ] valuation/valuation_space.py (Padmavathi)
[ ] valuation/valuations_catalog.py
[ ] valuation/value_group.py
}}}
= Necessary changes =
== Fix bugs in Sage ==
There are a number of trivial bugs that get fixed by monkey-patches in https://github.com/saraedum/mac_lane/blob/master/__init__.py
1. Conversion from a Function Field to its Constant Field #21872
1. Conversion from a Function Field to its underlying Polynomial Ring #23166
1. Coercions between Function Fields #23167
1. Coercions are injective if the underlying map is #21879
1. Ring homomorphisms from Fields are injective #21879
1. Polynomial rings embed into their fraction fields #23185
1. The embedding of a ring into a polynomial ring over that ring is injective #23203, #23204, #23211
1. p-adic rings embed into their fraction fields #23188
1. Morphisms of number fields are injective #21879
1. ZZ into QQ is injective #21879
1. quotients of polynomial rings are injective/surjective #23190
1. ZZ into a Number Field is injective #21879
1. ZZ into an order of a Number Field is injective #21879
1. ZZ does not map onto QQ #23186
1. ~~ZpCA shifts are broken~~
1. add default implementation of inverse_of_unit() #23191
== Add new features to Sage ==
New features that the code needs to work
1. Factorization over iterated extensions of finite fields. #21996
1. ~~principal_part() and sides() of a Newton Polygon~~ (patch this in the calling code instead.)
1. (cached_in_argument_method #22034)
== Make tests non-trivial ==
1. (some_elements() should be non-trivial for number fields/orders) #23192
1. (some_elements() should be non-trivial/deterministic for rational function fields and their extensions) #23193
1. (some_elements() should be non-trivial for fraction_fields of polynomial rings) #23194
== Add the valuation code to Sage ==
i.e., add these files https://github.com/saraedum/mac_lane to Sage." task needs_review minor sage-7.5 commutative algebra discrete valuations, valuations, p-adics, function fields, number fields, smooth projective curves, Mac Lane algorithm, Montes algorithm,sd87 Julian Rüth N/A move references to references file, move README to sage documentation, make sure that valuation() has a lot of the documentation and the factory just references it, remove optional: integrated bits u/saraedum/a_framework_for_discrete_valuations_in_sage c0a81c8285b47f6fc89aa34bc125ac474c75f2e9