Opened 3 years ago

Last modified 12 months ago

#21869 closed enhancement

A framework for discrete valuations in Sage — at Version 302

Reported by: saraedum Owned by:
Priority: major Milestone: sage-7.5
Component: commutative algebra Keywords: discrete valuations, valuations, p-adics, function fields, number fields, smooth projective curves, Mac Lane algorithm, Montes algorithm, sd87
Cc: Merged in:
Authors: Julian Rüth Reviewers: GaYee Park, Stefan Wewers, David Roe, Padmavathi Srinivasan, Shiva Chidambaram
Report Upstream: N/A Work issues:
Branch: u/saraedum/a_framework_for_discrete_valuations_in_sage (Commits) Commit: 24807e364bb0ac04e998465ad7c25ee28d16fbf8
Dependencies: #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620, #23642, #23965, #23966 Stopgaps:

Change History (302)

comment:1 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:2 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:3 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:4 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:5 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:6 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:7 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:8 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:9 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:10 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:11 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:12 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:13 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:14 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:15 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:16 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:17 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:18 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:19 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:20 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:21 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:22 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:23 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:24 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:25 Changed 3 years ago by saraedum

  • Description modified (diff)

comment:26 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:27 Changed 2 years ago by saraedum

  • Branch set to u/saraedum/a_framework_for_discrete_valuations_in_sage

comment:28 Changed 2 years ago by roed

  • Commit set to 0f615c771cac39a2cce4d54b4fd190f1c84992de
  • Keywords sd87 added

Last 10 new commits:

925d8fafix typo in comment
c4a7b1aAdded a tutorial in the README
20e6110move to subdirectory for merging with sage tree
ed206ecremoving gitignore for merge with sage tree
14799b0Merge mac_lane infrastructure for discrete valuations into sage
0767201remove mac_lane LICENSE
bd1dcbfremove obsolete TODOs
0eb3fffremove monkey patches
6c3a301move valuation code to valuation/
0f615c7remove specific valuation code out of valuation/

comment:29 Changed 2 years ago by git

  • Commit changed from 0f615c771cac39a2cce4d54b4fd190f1c84992de to 4153ef9b3a920d079437825960d904205fc2ae53

Branch pushed to git repo; I updated commit sha1. New commits:

8afc361remove standalone import commands
4153ef9Wired up valuations in the Sage library

comment:30 Changed 2 years ago by git

  • Commit changed from 4153ef9b3a920d079437825960d904205fc2ae53 to c0a81c8285b47f6fc89aa34bc125ac474c75f2e9

Branch pushed to git repo; I updated commit sha1. New commits:

c0a81c8fix function lookup

comment:31 Changed 2 years ago by saraedum

  • Description modified (diff)

New commits:

c0a81c8fix function lookup

comment:32 Changed 2 years ago by saraedum

  • Authors set to Julian Rüth
  • Description modified (diff)
  • Work issues set to 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

comment:33 Changed 2 years ago by saraedum

  • Status changed from new to needs_review

comment:34 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:35 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:36 follow-up: Changed 2 years ago by padma_sk

I was reviewing gauss_valuation.py and tried reducing a polynomial using the Gauss valuation induced by the 2-adic valuation, and got an unexpected error message. I thought it would tell me that the polynomial wasn't integral, and instead it gave me a coercion error.

sage: v
Gauss valuation induced by 2-adic valuation
sage: v.domain()
Univariate Polynomial Ring in y over Integer Ring
sage: h
1/2*y^2
sage: v.reduce(h)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-108-4d78566b468b> in <module>()
----> 1 v.reduce(h)
 
/projects/da1818ed-996d-4de6-acc6-361415b7725d/user/padma_sk/mac_lane/gauss_valuation.py in reduce(self, f, check, degree_bound, coefficients, valuations)
    360
    361         """
--> 362         f = self.domain().coerce(f)
    363
    364         if degree_bound is not None:
 
/usr/local/sage/src/sage/structure/parent.pyx in sage.structure.parent.Parent.coerce (/usr/local/sage/src/build/cythonized/sage/structure/parent.c:11229)()
   1166             return False
   1167
-> 1168     cpdef coerce(self, x):
   1169         """
   1170         Return x as an element of self, if and only if there is a canonical
 
/usr/local/sage/src/sage/structure/parent.pyx in sage.structure.parent.Parent.coerce (/usr/local/sage/src/build/cythonized/sage/structure/parent.c:11158)()
   1193                 except Exception:
   1194                     _record_exception()
-> 1195             raise TypeError("no canonical coercion from %s to %s" % (parent(x), self))
   1196         else:
   1197             return (<map.Map>mor)._call_(x)
 
TypeError: no canonical coercion from Univariate Polynomial Ring in y over Rational Field to Univariate Polynomial Ring in y over Integer Ring


comment:37 follow-up: Changed 2 years ago by padma_sk

equivalence_unit from gauss_valuation.py isn't outputting the results shown in the example input -2 as shown in the file.

sage: v
Gauss valuation induced by 2-adic valuation
sage: v.domain()
Univariate Polynomial Ring in y over Integer Ring
sage: v.equivalence_unit(2)
4
sage: v.equivalence_unit(-2)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-128-71aadb857663> in <module>()
----> 1 v.equivalence_unit(-Integer(2))
 
/usr/local/sage/src/sage/misc/cachefunc.pyx in sage.misc.cachefunc.CachedMethodCaller.__call__ (/usr/local/sage/src/build/cythonized/sage/misc/cachefunc.c:1079
2)()
   2036                 return cache[k]
   2037         except KeyError:
-> 2038             w = self._instance_call(*args, **kwds)
   2039             cache[k] = w
   2040             return w
 
/usr/local/sage/src/sage/misc/cachefunc.pyx in sage.misc.cachefunc.CachedMethodCaller._instance_call (/usr/local/sage/src/build/cythonized/sage/misc/cachefunc.
c:10238)()
   1912             True
   1913         """
-> 1914         return self.f(self._instance, *args, **kwds)
   1915
   1916     cdef fix_args_kwds(self, tuple args, dict kwds):
 
/projects/da1818ed-996d-4de6-acc6-361415b7725d/user/padma_sk/mac_lane/gauss_valuation.py in equivalence_unit(self, s, reciprocal)
    474             return self.equivalence_reciprocal(self.equivalence_unit(-s))
    475
--> 476         ret = self._base_valuation.element_with_valuation(s)
    477         return self.domain()(ret)
    478
 
/projects/da1818ed-996d-4de6-acc6-361415b7725d/user/padma_sk/mac_lane/valuation_space.py in element_with_valuation(self, s)
    465             s = QQ.coerce(s)
    466             if s not in self.value_semigroup():
--> 467                 raise ValueError("s must be in the value semigroup of this valuation but %r is not in %r"%(s, self.value_semigroup()))
    468             if s == 0:
    469                 return self.domain().one()
 
ValueError: s must be in the value semigroup of this valuation but -2 is not in Additive Abelian Semigroup generated by 1

comment:38 in reply to: ↑ 36 Changed 2 years ago by saraedum

Thanks for reporting this. I think this is ok. The problem here is not that the polynomial is not integral but that the polynomial is not in the domain.

This works:

sage: R.<x> = QQ[]
sage: v = valuations.GaussValuation(R, QQ.valuation(2))
sage: v.reduce(x + 1/2)
ValueError: reduction not defined for non-integral elements and x + 1/2 is not integral over Gauss valuation induced by 2-adic valuation

This does not:

sage: R.<x> = ZZ[]
sage: v = valuations.GaussValuation(R, ZZ.valuation(2))
sage: v.reduce(x + 1/2)
TypeError: no canonical coercion from Univariate Polynomial Ring in x over Rational Field to Univariate Polynomial Ring in x over Integer Ring

Replying to padma_sk:

I was reviewing gauss_valuation.py and tried reducing a polynomial using the Gauss valuation induced by the 2-adic valuation, and got an unexpected error message. I thought it would tell me that the polynomial wasn't integral, and instead it gave me a coercion error.

sage: v
Gauss valuation induced by 2-adic valuation
sage: v.domain()
Univariate Polynomial Ring in y over Integer Ring
sage: h
1/2*y^2
sage: v.reduce(h)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-108-4d78566b468b> in <module>()
----> 1 v.reduce(h)
 
/projects/da1818ed-996d-4de6-acc6-361415b7725d/user/padma_sk/mac_lane/gauss_valuation.py in reduce(self, f, check, degree_bound, coefficients, valuations)
    360
    361         """
--> 362         f = self.domain().coerce(f)
    363
    364         if degree_bound is not None:
 
/usr/local/sage/src/sage/structure/parent.pyx in sage.structure.parent.Parent.coerce (/usr/local/sage/src/build/cythonized/sage/structure/parent.c:11229)()
   1166             return False
   1167
-> 1168     cpdef coerce(self, x):
   1169         """
   1170         Return x as an element of self, if and only if there is a canonical
 
/usr/local/sage/src/sage/structure/parent.pyx in sage.structure.parent.Parent.coerce (/usr/local/sage/src/build/cythonized/sage/structure/parent.c:11158)()
   1193                 except Exception:
   1194                     _record_exception()
-> 1195             raise TypeError("no canonical coercion from %s to %s" % (parent(x), self))
   1196         else:
   1197             return (<map.Map>mor)._call_(x)
 
TypeError: no canonical coercion from Univariate Polynomial Ring in y over Rational Field to Univariate Polynomial Ring in y over Integer Ring


comment:39 in reply to: ↑ 37 Changed 2 years ago by saraedum

You are in the wrong domain in this example I think: This is the error you are seeing because there is no element of valuation -2 in ZZ[x].

sage: R.<x> = ZZ[]
sage: v = valuations.GaussValuation(R, ZZ.valuation(2))
sage: v.element_with_valuation(-2)
ValueError: s must be in the value semigroup of this valuation but -2 is not in Additive Abelian Semigroup generated by 1

However, this works in QQ[x].

sage: sage: R.<x> = QQ[]
sage: v = valuations.GaussValuation(R, QQ.valuation(2))
sage: v.element_with_valuation(-2)
1/4

Does that make sense? (Should it be documented more explicitly?)

Replying to padma_sk:

equivalence_unit from gauss_valuation.py isn't outputting the results shown in the example input -2 as shown in the file.

sage: v
Gauss valuation induced by 2-adic valuation
sage: v.domain()
Univariate Polynomial Ring in y over Integer Ring
sage: v.equivalence_unit(2)
4
sage: v.equivalence_unit(-2)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-128-71aadb857663> in <module>()
----> 1 v.equivalence_unit(-Integer(2))
 
/usr/local/sage/src/sage/misc/cachefunc.pyx in sage.misc.cachefunc.CachedMethodCaller.__call__ (/usr/local/sage/src/build/cythonized/sage/misc/cachefunc.c:1079
2)()
   2036                 return cache[k]
   2037         except KeyError:
-> 2038             w = self._instance_call(*args, **kwds)
   2039             cache[k] = w
   2040             return w
 
/usr/local/sage/src/sage/misc/cachefunc.pyx in sage.misc.cachefunc.CachedMethodCaller._instance_call (/usr/local/sage/src/build/cythonized/sage/misc/cachefunc.
c:10238)()
   1912             True
   1913         """
-> 1914         return self.f(self._instance, *args, **kwds)
   1915
   1916     cdef fix_args_kwds(self, tuple args, dict kwds):
 
/projects/da1818ed-996d-4de6-acc6-361415b7725d/user/padma_sk/mac_lane/gauss_valuation.py in equivalence_unit(self, s, reciprocal)
    474             return self.equivalence_reciprocal(self.equivalence_unit(-s))
    475
--> 476         ret = self._base_valuation.element_with_valuation(s)
    477         return self.domain()(ret)
    478
 
/projects/da1818ed-996d-4de6-acc6-361415b7725d/user/padma_sk/mac_lane/valuation_space.py in element_with_valuation(self, s)
    465             s = QQ.coerce(s)
    466             if s not in self.value_semigroup():
--> 467                 raise ValueError("s must be in the value semigroup of this valuation but %r is not in %r"%(s, self.value_semigroup()))
    468             if s == 0:
    469                 return self.domain().one()
 
ValueError: s must be in the value semigroup of this valuation but -2 is not in Additive Abelian Semigroup generated by 1

comment:40 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:41 Changed 2 years ago by saraedum

  • Dependencies set to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996

comment:42 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190

comment:43 Changed 2 years ago by padma_sk

  • Description modified (diff)

comment:44 Changed 2 years ago by padma_sk

  • Description modified (diff)

comment:45 follow-up: Changed 2 years ago by gpark

  • Reviewers set to GaYee Park
  • Status changed from needs_review to needs_work

Ran the test and found the error:

sage -t src/sage/rings/function_field/function_field.py # 7 doctests failed

comment:46 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:47 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 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 to 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, check that discrete_value_group is still available the same way it was under padics

comment:48 in reply to: ↑ 45 Changed 2 years ago by saraedum

Replying to gpark:

Ran the test and found the error:

sage -t src/sage/rings/function_field/function_field.py # 7 doctests failed

Did you run the tests with all the dependencies applied? About 50 tests fail for me. That's expected (since the dependencies are not in yet.)

comment:49 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Reviewers changed from GaYee Park to GaYee Park, Stefan Wewers

comment:50 Changed 2 years ago by saraedum

  • Status changed from needs_work to needs_review

comment:51 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:52 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:53 Changed 2 years ago by saraedum

The following errors are expected. I am working on them.

----------------------------------------------------------------------
sage -t --warn-long 39.1 src/sage/rings/valuation/limit_valuation.py  # 52 doctests failed
sage -t --warn-long 39.1 src/sage/rings/valuation/gauss_valuation.py  # 1 doctest failed
sage -t --warn-long 39.1 src/sage/rings/valuation/valuation_space.py  # 2 doctests failed
sage -t --warn-long 39.1 src/sage/rings/valuation/mapped_valuation.py  # 42 doctests failed
sage -t --warn-long 39.1 src/sage/rings/valuation/valuation.py  # 9 doctests failed
sage -t --warn-long 39.1 src/sage/rings/valuation/inductive_valuation.py  # 13 doctests failed
sage -t --warn-long 39.1 src/sage/rings/valuation/augmented_valuation.py  # 16 doctests failed
----------------------------------------------------------------------

comment:54 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics to 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, check that discrete_value_group is still available the same way it was under padics, 135 failing doctests

comment:55 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495
  • Description modified (diff)
  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 135 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics

comment:56 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics to 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, check that discrete_value_group is still available the same way it was under padics, 135 failing doctests

comment:57 Changed 2 years ago by saraedum

  • Description modified (diff)

Sorry, I somehow reverted some changes to the ticket description. Not sure how this happened.

comment:58 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 135 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 252 failing doctests

comment:59 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:60 Changed 2 years ago by roed

  • Description modified (diff)

comment:61 Changed 2 years ago by swewers

  • Description modified (diff)

I have had a look at function_field_valuation. I have already worked with the code a lot, and it is working fine, and the documentation is extensive and clear. But I have only tested the latest standalone version.

comment:62 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 252 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 288 failing doctests

comment:63 Changed 2 years ago by saraedum

  • Reviewers changed from GaYee Park, Stefan Wewers to GaYee Park, Stefan Wewers, David Roe

comment:64 Changed 2 years ago by git

  • Commit changed from c0a81c8285b47f6fc89aa34bc125ac474c75f2e9 to 9fd87332c734899d860972d836d479283b5ab62f

Branch pushed to git repo; I updated commit sha1. New commits:

9fd8733fix import

comment:65 Changed 2 years ago by git

  • Commit changed from 9fd87332c734899d860972d836d479283b5ab62f to e1e65a5d7514ebe51365bdf1da402d2054f02654

Branch pushed to git repo; I updated commit sha1. New commits:

e1e65a5fixed typo

comment:66 Changed 2 years ago by git

  • Commit changed from e1e65a5d7514ebe51365bdf1da402d2054f02654 to 9c16a1bc642498daaa1c17f61a874cbd42f4063b

Branch pushed to git repo; I updated commit sha1. New commits:

9c16a1bundo .valuation() shortcut

comment:67 Changed 2 years ago by git

  • Commit changed from 9c16a1bc642498daaa1c17f61a874cbd42f4063b to 119f2454cb4100c62300121a7df024a8b4dead9e

Branch pushed to git repo; I updated commit sha1. New commits:

119f245It is not relevant whether a valuation converts into the valuation space but whether it is already in the valuation space

comment:68 Changed 2 years ago by git

  • Commit changed from 119f2454cb4100c62300121a7df024a8b4dead9e to 7ea987498d844fb5aa28c4023cc8f4350d48a20a

Branch pushed to git repo; I updated commit sha1. New commits:

7ea9874fix doctests

comment:69 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 288 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 36 failing doctests

New commits:

9fd8733fix import
e1e65a5fixed typo
9c16a1bundo .valuation() shortcut
119f245It is not relevant whether a valuation converts into the valuation space but whether it is already in the valuation space
7ea9874fix doctests

comment:70 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483

comment:71 Changed 2 years ago by git

  • Commit changed from 7ea987498d844fb5aa28c4023cc8f4350d48a20a to 0c60cb7006afcd336963fdcc7af5cab510cd796b

Branch pushed to git repo; I updated commit sha1. New commits:

3fd4140add doctest output
0c60cb7fix valuation parameters

comment:72 Changed 2 years ago by git

  • Commit changed from 0c60cb7006afcd336963fdcc7af5cab510cd796b to 20100145cbdac34f8f3f021f597f750541c60d11

Branch pushed to git repo; I updated commit sha1. New commits:

2010014restricting to ZZ, we need to know the prime

comment:73 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 36 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests

New commits:

3fd4140add doctest output
0c60cb7fix valuation parameters
2010014restricting to ZZ, we need to know the prime

comment:74 Changed 2 years ago by git

  • Commit changed from 20100145cbdac34f8f3f021f597f750541c60d11 to efe24904ec35606462a941a47fee496ebf463491

Branch pushed to git repo; I updated commit sha1. New commits:

e5ef4d8do not catch everything
efe2490Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:75 Changed 2 years ago by roed

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495
  • Description modified (diff)
  • Reviewers changed from GaYee Park, Stefan Wewers, David Roe to GaYee Park, Stefan Wewers
  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 252 failing doctests

New commits:

e5ef4d8do not catch everything
9fd8733fix import
e1e65a5fixed typo
9c16a1bundo .valuation() shortcut
119f245It is not relevant whether a valuation converts into the valuation space but whether it is already in the valuation space
7ea9874fix doctests
3fd4140add doctest output
0c60cb7fix valuation parameters
2010014restricting to ZZ, we need to know the prime
efe2490Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:76 Changed 2 years ago by roed

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483
  • Description modified (diff)
  • Reviewers changed from GaYee Park, Stefan Wewers to GaYee Park, Stefan Wewers, David Roe
  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 252 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests

Oops. Reverting.

comment:77 Changed 2 years ago by roed

  • Description modified (diff)

comment:78 Changed 2 years ago by git

  • Commit changed from efe24904ec35606462a941a47fee496ebf463491 to 0fd986f1773f30ea1f89b01198892fd32c122882

Branch pushed to git repo; I updated commit sha1. New commits:

1eace28Update valuation_space.py
21b0f68Update valuation_space.py
38f3815Update valuation_space.py
7c57f68Update valuation_space.py
d90afa8Update valuation_space.py
1cb8178Update valuation_space.py
39cb55bUpdate valuation_space.py
a474ed5Update valuation_space.py
ca2089fUpdate valuation_space.py
0fd986fMerge remote-tracking branch 'remotes/github/sage' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:79 Changed 2 years ago by roed

  • Description modified (diff)

comment:80 Changed 2 years ago by roed

  • Description modified (diff)

comment:81 Changed 2 years ago by roed

  • Description modified (diff)

comment:82 Changed 2 years ago by roed

  • Description modified (diff)

comment:83 Changed 2 years ago by roed

  • Description modified (diff)

comment:84 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:85 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?

comment:86 Changed 2 years ago by shivac

  • Description modified (diff)

comment:87 Changed 2 years ago by roed

  • Description modified (diff)

comment:88 Changed 2 years ago by roed

  • Description modified (diff)

comment:89 Changed 2 years ago by roed

  • Description modified (diff)

comment:90 Changed 2 years ago by roed

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism? to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, make sure there are no more optional:integrateds

comment:91 Changed 2 years ago by saraedum

  • Reviewers changed from GaYee Park, Stefan Wewers, David Roe to GaYee Park, Stefan Wewers, David Roe, Padmavathi Srinivasan, Shiva Chidambaram
  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, make sure there are no more optional:integrateds to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?

comment:92 Changed 2 years ago by git

  • Commit changed from 0fd986f1773f30ea1f89b01198892fd32c122882 to 33aa99f1aa6c8e4478cd73c1808b3e7918a29402

Branch pushed to git repo; I updated commit sha1. New commits:

bcb78f3Update scaled_valuation.py
be4daabUpdate valuation.py
7a95a9dUpdate augmented_valuation.py
dbf7791Update augmented_valuation.py
58bb2bcUpdate developing_valuation.py
e1c8bd5Update limit_valuation.py
a0f6bd3Update mapped_valuation.py
33aa99fUpdate padic_valuation.py

comment:93 Changed 2 years ago by git

  • Commit changed from 33aa99f1aa6c8e4478cd73c1808b3e7918a29402 to 37d718725faa28404b3db194cc81b9eede19c01e

Branch pushed to git repo; I updated commit sha1. New commits:

b90ba64Fix doctest for left action
37d7187Remove absolute_import

comment:94 Changed 2 years ago by roed

  • Description modified (diff)
  • Type changed from task to enhancement

New commits:

b90ba64Fix doctest for left action
37d7187Remove absolute_import

comment:95 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:96 Changed 2 years ago by git

  • Commit changed from 37d718725faa28404b3db194cc81b9eede19c01e to 8da1ceb17a7bbcfbf174cdb754365d3ddca05cc0

Branch pushed to git repo; I updated commit sha1. New commits:

8da1cebClarify semigroup construction

comment:97 Changed 2 years ago by git

  • Commit changed from 8da1ceb17a7bbcfbf174cdb754365d3ddca05cc0 to 07e40b964b4ab3a9f7cdf448bcd5956065fd787f

Branch pushed to git repo; I updated commit sha1. New commits:

07e40b9add shifting doctest

comment:98 Changed 2 years ago by roed

  • Description modified (diff)

New commits:

07e40b9add shifting doctest

comment:99 Changed 2 years ago by git

  • Commit changed from 07e40b964b4ab3a9f7cdf448bcd5956065fd787f to 021b188797aa024ae6f17f4954f679eabcaed5c3

Branch pushed to git repo; I updated commit sha1. New commits:

021b188improve error message

comment:100 Changed 2 years ago by git

  • Commit changed from 021b188797aa024ae6f17f4954f679eabcaed5c3 to a6a798f9e713f7237e81eea92cdaca9bc455eda3

Branch pushed to git repo; I updated commit sha1. New commits:

a6a798fClarified the notion of minimality

comment:101 Changed 2 years ago by git

  • Commit changed from a6a798f9e713f7237e81eea92cdaca9bc455eda3 to ee0db3141cb365e8e7cdf89ec568b3b493a179e9

Branch pushed to git repo; I updated commit sha1. New commits:

ee0db31clarify meaning of effective_degree

comment:102 Changed 2 years ago by git

  • Commit changed from ee0db3141cb365e8e7cdf89ec568b3b493a179e9 to e9243b8ea32bdaba2694bc77071b3ee41b7355ce

Branch pushed to git repo; I updated commit sha1. New commits:

e9243b8bring residue into the right residue ring

comment:103 Changed 2 years ago by git

  • Commit changed from e9243b8ea32bdaba2694bc77071b3ee41b7355ce to 24e68ecc6536c18d63d77d7f56ae4cb27a98ae10

Branch pushed to git repo; I updated commit sha1. New commits:

24e68ecadd to/from residue ring tests

comment:104 Changed 2 years ago by git

  • Commit changed from 24e68ecc6536c18d63d77d7f56ae4cb27a98ae10 to 2bed311a203da9a773b4cefe0bc3c6b5c038394b

Branch pushed to git repo; I updated commit sha1. New commits:

791879ause more robust r strings
2bed311No workaround for number field equality necessary

comment:105 Changed 2 years ago by git

  • Commit changed from 2bed311a203da9a773b4cefe0bc3c6b5c038394b to 544977a3161e2f86d1e924b7403fd9709c77620a

Branch pushed to git repo; I updated commit sha1. New commits:

c02de30Added an example of a p-adic valuation in an order
e3fb245trivial extensions are totally ramified
e959a8bcheck for subrings so extensions() is not bound to fail
544977aadd doctest to element_with_valuation

comment:106 Changed 2 years ago by git

  • Commit changed from 544977a3161e2f86d1e924b7403fd9709c77620a to a0880bd55bb78b4c738917211adf2d0892a33038

Branch pushed to git repo; I updated commit sha1. New commits:

a0880bdFix bug in p-adic shift

comment:107 Changed 2 years ago by git

  • Commit changed from a0880bd55bb78b4c738917211adf2d0892a33038 to e9eddbd7dc92017f1113612dace917d562c55eaf

Branch pushed to git repo; I updated commit sha1. New commits:

e9eddbdbetter size estimate for large primes

comment:108 Changed 2 years ago by git

  • Commit changed from e9eddbd7dc92017f1113612dace917d562c55eaf to d0ebc9bc5ba1f1f76476c9cc2c69f39f26a9e1d0

Branch pushed to git repo; I updated commit sha1. New commits:

6a1e0f1fixed docstring
45286d7Update function_field.py
d0ebc9bMerge branch 'sage' of github.com:saraedum/mac_lane into t/21869/a_framework_for_discrete_valuations_in_sage

comment:109 Changed 2 years ago by git

  • Commit changed from d0ebc9bc5ba1f1f76476c9cc2c69f39f26a9e1d0 to 7c1bf020ef92e91440d97a3cc5a7354483ef36c0

Branch pushed to git repo; I updated commit sha1. New commits:

7c1bf02Key has changed

comment:110 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:111 Changed 2 years ago by git

  • Commit changed from 7c1bf020ef92e91440d97a3cc5a7354483ef36c0 to a93ce9ae2b875091b6e3d3b2079835b69026525f

Branch pushed to git repo; I updated commit sha1. New commits:

a93ce9aSome examples that came up at Sage Days 87

comment:112 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism? to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation

New commits:

a93ce9aSome examples that came up at Sage Days 87

comment:113 Changed 2 years ago by git

  • Commit changed from a93ce9ae2b875091b6e3d3b2079835b69026525f to 7bc7f170e970245c65c39e379a84d099234e8c83

Branch pushed to git repo; I updated commit sha1. New commits:

709e7f6GaussValuation is available on the global scope
a86ed5eadded introductory examples for developing valuations
f305a7fUpdate inductive_valuation.py
b86ddd7Merge branch 'sage' of github.com:saraedum/mac_lane into t/21869/a_framework_for_discrete_valuations_in_sage
7bc7f17Improved intro sections of valuation files

comment:114 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation to 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods

New commits:

709e7f6GaussValuation is available on the global scope
a86ed5eadded introductory examples for developing valuations
f305a7fUpdate inductive_valuation.py
b86ddd7Merge branch 'sage' of github.com:saraedum/mac_lane into t/21869/a_framework_for_discrete_valuations_in_sage
7bc7f17Improved intro sections of valuation files

comment:115 Changed 2 years ago by git

  • Commit changed from 7bc7f170e970245c65c39e379a84d099234e8c83 to 62134d89bd5b2b3f6064e6d9cf48133558bbe91f

Branch pushed to git repo; I updated commit sha1. New commits:

62134d8move README to the reference manual

comment:116 Changed 2 years ago by saraedum

  • Work issues changed from 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, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods to make sure that valuation() has a lot of the documentation and the factory just references it, remove optional: integrated bits, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods

New commits:

62134d8move README to the reference manual

comment:117 Changed 2 years ago by git

  • Commit changed from 62134d89bd5b2b3f6064e6d9cf48133558bbe91f to 39ee4e92895e44d022ecc9aebdf7a4aed2eabbfd

Branch pushed to git repo; I updated commit sha1. New commits:

39ee4e9Move p-adic valuation factory documentation

comment:118 Changed 2 years ago by saraedum

  • Work issues changed from make sure that valuation() has a lot of the documentation and the factory just references it, remove optional: integrated bits, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods to remove optional: integrated bits, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods

New commits:

39ee4e9Move p-adic valuation factory documentation

New commits:

39ee4e9Move p-adic valuation factory documentation

comment:119 Changed 2 years ago by saraedum

  • Work issues changed from remove optional: integrated bits, check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods to check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods

comment:120 Changed 2 years ago by saraedum

  • Work issues changed from check that discrete_value_group is still available the same way it was under padics, 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods to 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods

comment:121 Changed 2 years ago by saraedum

  • Work issues changed from 33 failing doctests, is anything inheriting from Map/DefaultConvertMap instead of Morphism?, check generated documentation, add references to methods to 33 failing doctests, check generated documentation, add references to method

comment:122 Changed 2 years ago by git

  • Commit changed from 39ee4e92895e44d022ecc9aebdf7a4aed2eabbfd to d893c55a222ef01c0c281c7fc9032118003e8dcc

Branch pushed to git repo; I updated commit sha1. New commits:

d893c55remove completed TODO

comment:123 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 33 failing doctests, check generated documentation, add references to method to 33 failing doctests, check generated documentation, add references to method, comments on github

New commits:

d893c55remove completed TODO

comment:124 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:125 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:126 Changed 2 years ago by git

  • Commit changed from d893c55a222ef01c0c281c7fc9032118003e8dcc to e318075c0420a9225010a4b57c66e68d9c052567

Branch pushed to git repo; I updated commit sha1. New commits:

e318075value_group has moved

comment:127 Changed 2 years ago by git

  • Commit changed from e318075c0420a9225010a4b57c66e68d9c052567 to a2d10913245fc598684fe178f3eaf74bac5e5b2e

Branch pushed to git repo; I updated commit sha1. New commits:

a2d1091fix toctree formatting

comment:128 Changed 2 years ago by git

  • Commit changed from a2d10913245fc598684fe178f3eaf74bac5e5b2e to 9b0280042e3e1266bce9c4dee0e92ec207d0205f

Branch pushed to git repo; I updated commit sha1. New commits:

9b02800fix docbuild

comment:129 Changed 2 years ago by git

  • Commit changed from 9b0280042e3e1266bce9c4dee0e92ec207d0205f to 735cd8d901ea93cf7e7d2613e16f9f0514967ec4

Branch pushed to git repo; I updated commit sha1. New commits:

735cd8dfix typo

comment:130 Changed 2 years ago by git

  • Commit changed from 735cd8d901ea93cf7e7d2613e16f9f0514967ec4 to 04dd3954fc4969dab2049e689453ddb606a5419e

Branch pushed to git repo; I updated commit sha1. New commits:

04dd395fix reference link

comment:131 Changed 2 years ago by git

  • Commit changed from 04dd3954fc4969dab2049e689453ddb606a5419e to 36f33e6cbd3d8535f8c1da12af0e9f549df8f602

Branch pushed to git repo; I updated commit sha1. New commits:

75ccb0efix typo
36f33e6remove commented out code

comment:132 Changed 2 years ago by git

  • Commit changed from 36f33e6cbd3d8535f8c1da12af0e9f549df8f602 to 4630b93916bf0c5277c63eef537019e6ce3c28fe

Branch pushed to git repo; I updated commit sha1. New commits:

13f7d6cfix grammar
39b0792Fix MacLane references
4630b93fix LaTeX

comment:133 Changed 2 years ago by git

  • Commit changed from 4630b93916bf0c5277c63eef537019e6ce3c28fe to cc9607fbf5d5e836a20f97117991886d3d27aab1

Branch pushed to git repo; I updated commit sha1. New commits:

cc9607ffix docbuild

comment:134 Changed 2 years ago by git

  • Commit changed from cc9607fbf5d5e836a20f97117991886d3d27aab1 to 2b767b709f4cf5e11e4d1bdf946e726a0e5865df

Branch pushed to git repo; I updated commit sha1. New commits:

2b767b7fix reference key

comment:135 Changed 2 years ago by git

  • Commit changed from 2b767b709f4cf5e11e4d1bdf946e726a0e5865df to cdf89aef1bf3a1b6f7c98a650ce8669dd6c1becb

Branch pushed to git repo; I updated commit sha1. New commits:

cdf89aerenamed reference manual section

comment:136 Changed 2 years ago by saraedum

  • Description modified (diff)

Last 10 new commits:

735cd8dfix typo
04dd395fix reference link
75ccb0efix typo
36f33e6remove commented out code
13f7d6cfix grammar
39b0792Fix MacLane references
4630b93fix LaTeX
cc9607ffix docbuild
2b767b7fix reference key
cdf89aerenamed reference manual section

comment:137 Changed 2 years ago by git

  • Commit changed from cdf89aef1bf3a1b6f7c98a650ce8669dd6c1becb to c011093d3b22d3826a51c28f48bf81cef79063fd

Branch pushed to git repo; I updated commit sha1. New commits:

c011093reformat headers for reference manual

comment:138 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 33 failing doctests, check generated documentation, add references to method, comments on github to 121 failing doctests, check generated documentation, add references to method, comments on github

New commits:

c011093reformat headers for reference manual

comment:139 Changed 2 years ago by git

  • Commit changed from c011093d3b22d3826a51c28f48bf81cef79063fd to 8022ed196995b2e7b1bc93c9dcd5f0d93fd3bfd4

Branch pushed to git repo; I updated commit sha1. New commits:

8022ed1fix docbuild

comment:140 Changed 2 years ago by git

  • Commit changed from 8022ed196995b2e7b1bc93c9dcd5f0d93fd3bfd4 to 14e034e46f0834f42f0ddd84f74a8b2814dbe809

Branch pushed to git repo; I updated commit sha1. New commits:

14e034efix value_group documentation

comment:141 Changed 2 years ago by git

  • Commit changed from 14e034e46f0834f42f0ddd84f74a8b2814dbe809 to 3bff1b961ca0104b5cb70415b2100bd4c83d1ae8

Branch pushed to git repo; I updated commit sha1. New commits:

3bff1b9fix documentation in valuation

comment:142 Changed 2 years ago by git

  • Commit changed from 3bff1b961ca0104b5cb70415b2100bd4c83d1ae8 to 7db0f057258c627ac6e648cebd67536569500cd6

Branch pushed to git repo; I updated commit sha1. New commits:

ed13821fix documentation of valuation_space
7db0f05fix documentation of gauss_valuation

comment:143 Changed 2 years ago by git

  • Commit changed from 7db0f057258c627ac6e648cebd67536569500cd6 to 2f6dd581cc8768f4cfa467629df2fe50773758b2

Branch pushed to git repo; I updated commit sha1. New commits:

2f6dd58fix MacLane references

comment:144 Changed 2 years ago by git

  • Commit changed from 2f6dd581cc8768f4cfa467629df2fe50773758b2 to 007499ebc0532f8c9fb63767d359b9dcbf582cac

Branch pushed to git repo; I updated commit sha1. New commits:

738904btrying to fix augmented valuation links
007499eshorten links

comment:145 Changed 2 years ago by git

  • Commit changed from 007499ebc0532f8c9fb63767d359b9dcbf582cac to a105a8f43dc3bce318bb373d9fccf0a0ec34948b

Branch pushed to git repo; I updated commit sha1. New commits:

a105a8ffixed links in documentation

comment:146 Changed 2 years ago by git

  • Commit changed from a105a8f43dc3bce318bb373d9fccf0a0ec34948b to 3b91735e9718a4b000a627a451ddc6aedef61ac2

Branch pushed to git repo; I updated commit sha1. New commits:

3b91735fixing links in developing valuation

comment:147 Changed 2 years ago by git

  • Commit changed from 3b91735e9718a4b000a627a451ddc6aedef61ac2 to ee5abb213abfaa78d2ebe82f839c5fe312b54146

Branch pushed to git repo; I updated commit sha1. New commits:

ee5abb2Fix links in developing valuation

comment:148 Changed 2 years ago by git

  • Commit changed from ee5abb213abfaa78d2ebe82f839c5fe312b54146 to 548e879415fb1a128531f4961f51cc9e88a3e313

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

51e89dbMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
0f66745Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
984cc75Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
35b62e6Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
c50f6ecMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
93757efMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
5c62063Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
f173cafMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
013292fMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
548e879Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:149 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 121 failing doctests, check generated documentation, add references to method, comments on github to 121 failing doctests, check generated documentation, add references to method, comments on github, check docbuild output for broken links

comment:150 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 121 failing doctests, check generated documentation, add references to method, comments on github, check docbuild output for broken links to 121 failing doctests, comments on github, check docbuild output for broken links

comment:151 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 121 failing doctests, comments on github, check docbuild output for broken links to 120 failing doctests, comments on github, check docbuild output for broken links

comment:152 Changed 2 years ago by git

  • Commit changed from 548e879415fb1a128531f4961f51cc9e88a3e313 to 2ca7a7c934463219690fd31d5ff4f4c3ee341c31

Branch pushed to git repo; I updated commit sha1. New commits:

6da7536fix broken documentation links
6aa9f65Fixed documentation
d27262aFix _richcmp_
6509f94Merge remote-tracking branch 'trac/u/saraedum/23190' into t/21869/a_framework_for_discrete_valuations_in_sage
fb66f9fFix richcmp
e3fb631Merge branch 'u/saraedum/a_polynomial_ring_embeds_into_its_fraction_field' of git://trac.sagemath.org/sage into t/23185/a_polynomial_ring_embeds_into_its_fraction_field
f3ecd0eadded missing doctest
5093894Merge remote-tracking branch 'trac/u/saraedum/a_polynomial_ring_embeds_into_its_fraction_field' into t/21869/a_framework_for_discrete_valuations_in_sage
2ca7a7cfix pickling of valuations

comment:153 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 120 failing doctests, comments on github, check docbuild output for broken links to 67 failing doctests, comments on github, check docbuild output for broken links

New commits:

6da7536fix broken documentation links
6aa9f65Fixed documentation
d27262aFix _richcmp_
6509f94Merge remote-tracking branch 'trac/u/saraedum/23190' into t/21869/a_framework_for_discrete_valuations_in_sage
fb66f9fFix richcmp
e3fb631Merge branch 'u/saraedum/a_polynomial_ring_embeds_into_its_fraction_field' of git://trac.sagemath.org/sage into t/23185/a_polynomial_ring_embeds_into_its_fraction_field
f3ecd0eadded missing doctest
5093894Merge remote-tracking branch 'trac/u/saraedum/a_polynomial_ring_embeds_into_its_fraction_field' into t/21869/a_framework_for_discrete_valuations_in_sage
2ca7a7cfix pickling of valuations

comment:154 Changed 2 years ago by git

  • Commit changed from 2ca7a7c934463219690fd31d5ff4f4c3ee341c31 to 45e0e8f7cb9b0b6b8aaa1d3af8e4c4a761fb0cca

Branch pushed to git repo; I updated commit sha1. New commits:

b105f14Disable function field test
50d6090fix _relative_size output
9bf9e75fill in missing doctest output
45e0e8ffix copy&paste error

comment:155 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 67 failing doctests, comments on github, check docbuild output for broken links to 53 failing doctests, comments on github, check docbuild output for broken links

New commits:

b105f14Disable function field test
50d6090fix _relative_size output
9bf9e75fill in missing doctest output
45e0e8ffix copy&paste error

comment:156 Changed 2 years ago by git

  • Commit changed from 45e0e8f7cb9b0b6b8aaa1d3af8e4c4a761fb0cca to 6de28680e68ec96f826efca092dc39b8e032158c

Branch pushed to git repo; I updated commit sha1. New commits:

a4ac152Call pAdicValuation instead of .valuation
6de2868fix _relative_size output

comment:157 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 53 failing doctests, comments on github, check docbuild output for broken links to 46 failing doctests, comments on github, check docbuild output for broken links

New commits:

a4ac152Call pAdicValuation instead of .valuation
6de2868fix _relative_size output

comment:158 Changed 2 years ago by git

  • Commit changed from 6de28680e68ec96f826efca092dc39b8e032158c to 249bdc0743c0c79ffd6cec510db0d9c7bbec49e2

Branch pushed to git repo; I updated commit sha1. New commits:

cf61932Fix doctest
1329738Allow p-adic rings
249bdc0Update doctest output

comment:159 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 46 failing doctests, comments on github, check docbuild output for broken links to 36 failing doctests, comments on github, check docbuild output for broken links

New commits:

cf61932Fix doctest
1329738Allow p-adic rings
249bdc0Update doctest output

comment:160 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510

comment:161 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525

comment:162 Changed 2 years ago by git

  • Commit changed from 249bdc0743c0c79ffd6cec510db0d9c7bbec49e2 to 005e7ab0f7740c533e13de1c5f9555cf1575fb72

Branch pushed to git repo; I updated commit sha1. New commits:

a92f47fmake sure that arguments to add_bigoh are correctly normalized
aa1709dcomment on stagnating degrees
005e7abMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:163 Changed 2 years ago by git

  • Commit changed from 005e7ab0f7740c533e13de1c5f9555cf1575fb72 to 2d1cb90b3deffd9210a9d6366cdf8666c38acbdc

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

bd15d71add exact keyword
99dccf6Merge branch 't/23331/allow_exact_defining_polynomials_for_p_adic_extensions' into t/20310/change_precision
ef4ed33Fix SEEALSO:
1754b44Fix exact=True errors in the right branch
7926b41Moving code for fraction_field and integer_ring, and enabling fraction fields for fixed-mod
7712a86Merge remote-tracking branch 'trac/u/roed/fixed_mod_frac_field' into t/21869/a_framework_for_discrete_valuations_in_sage
276e4f7fix and fill in doctest output
9b03cf8compute fraction field only when necessary
a886c42fix pickling of nodes
2d1cb90Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:164 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 36 failing doctests, comments on github, check docbuild output for broken links to 7 failing doctests, check docbuild output for broken links

Last 10 new commits:

1754b44Fix exact=True errors in the right branch
7926b41Moving code for fraction_field and integer_ring, and enabling fraction fields for fixed-mod
7712a86Merge remote-tracking branch 'trac/u/roed/fixed_mod_frac_field' into t/21869/a_framework_for_discrete_valuations_in_sage
276e4f7fix and fill in doctest output
9b03cf8compute fraction field only when necessary
a886c42fix pickling of nodes
a92f47fmake sure that arguments to add_bigoh are correctly normalized
aa1709dcomment on stagnating degrees
005e7abMerge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
2d1cb90Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:165 Changed 2 years ago by git

  • Commit changed from 2d1cb90b3deffd9210a9d6366cdf8666c38acbdc to 282c95ea845009b91e5db57e9b6f76de562f64ab

Branch pushed to git repo; I updated commit sha1. New commits:

282c95eRemove "not slopes"

comment:166 Changed 2 years ago by saraedum

  • Work issues changed from 7 failing doctests, check docbuild output for broken links to 7 failing doctests, check docbuild output for broken links, check _richcmp_ implementations

comment:167 Changed 2 years ago by roed

  • Priority changed from minor to major

Doesn't seem "minor"....

comment:168 Changed 2 years ago by git

  • Commit changed from 282c95ea845009b91e5db57e9b6f76de562f64ab to ad9adb1ab69e0bec23ce6a87a4f6f106e85b78f1

Branch pushed to git repo; I updated commit sha1. New commits:

7b44fb3A test case that failed in an older version
ad9adb1Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:169 Changed 2 years ago by git

  • Commit changed from ad9adb1ab69e0bec23ce6a87a4f6f106e85b78f1 to 7660eab0134f488f18572d2b4a2a856d43641f33

Branch pushed to git repo; I updated commit sha1. New commits:

28fbf35Check that extensions are computed correctly in towers of number fields
101502eMerge branch 'develop' into t/21869/a_framework_for_discrete_valuations_in_sage
1221125remove TABs
7660eabMapped valuations should know whether they are discrete

comment:170 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620

comment:171 Changed 2 years ago by git

  • Commit changed from 7660eab0134f488f18572d2b4a2a856d43641f33 to 51f7ed09ea118b0d05c6bc124c159b321c89a90c

Branch pushed to git repo; I updated commit sha1. New commits:

4e46d34Added another failing case
2bbcbb9Merge branch 't/23525/implement_is_squarefree_for_p_adic_rings' into t/21869/a_framework_for_discrete_valuations_in_sage
51f7ed0Add test case

comment:172 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620, #23642

comment:173 Changed 2 years ago by git

  • Commit changed from 51f7ed09ea118b0d05c6bc124c159b321c89a90c to 7c915f1213eb08f9b5bb487b2d21b1755ff2fff4

Branch pushed to git repo; I updated commit sha1. New commits:

7c915f1Merge branch 'develop' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:174 Changed 2 years ago by mderickx

  • Status changed from needs_review to needs_work

comment:175 Changed 2 years ago by git

  • Commit changed from 7c915f1213eb08f9b5bb487b2d21b1755ff2fff4 to 29feccb3d72a85cd739d09faa621d4690798ad7d

Branch pushed to git repo; I updated commit sha1. New commits:

82338dcFix factorization over fraction fields that are isomorphic to function fields
64e6546Merge branch 'develop' into t/23642/factorization_over_some_quotient_rings_incorrect
f6dd0cdBetter isomorphisms between fraction fields and function fields
38f18b8minor docstring change
8bfe326Merge branch 'develop' into t/23660/better_isomorphisms_between_function_fields_and_fraction_fields
746eac0fix docstring
4bddab1Merge branch 't/23660/better_isomorphisms_between_function_fields_and_fraction_fields' into t/23642/factorization_over_some_quotient_rings_incorrect
29feccbMerge branch 't/23642/factorization_over_some_quotient_rings_incorrect' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:176 Changed 2 years ago by git

  • Commit changed from 29feccb3d72a85cd739d09faa621d4690798ad7d to b3c8ac3730edd5b5977fbaa93c1cca1096df6dda

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

7e62f70Fixing errors in the coercion to the fraction field for fixed-mod p-adics
1a4333aMerge branch 't/13591/add_bigoh' into t/23510/fixed_mod_frac_field
cdeb376Merge branch 't/13591/add_bigoh' into t/23510/fixed_mod_frac_field
cb882fdUpdate add_bigoh to account for fixed-mod having a fraction field
a826978Fix _tester_add_bigoh
883e6b5Fix doctest in local_generic_element.pyx
52cbd2aMerge branch 'u/roed/fixed_mod_frac_field' of git://trac.sagemath.org/sage into t/23510/fixed_mod_frac_field
bc59ffaMerge branch 't/14825/polynomial_representation_of_a_padic_number' into t/23510/fixed_mod_frac_field
3292259Merge branch 'u/saraedum/fixed_mod_frac_field' of git://trac.sagemath.org/sage into t/23510/fixed_mod_frac_field
b3c8ac3Merge branch 't/23510/fixed_mod_frac_field' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:177 Changed 2 years ago by git

  • Commit changed from b3c8ac3730edd5b5977fbaa93c1cca1096df6dda to 608c5c3dda3ae4560bc0e88eeb440233ab5aa4bc

Branch pushed to git repo; I updated commit sha1. New commits:

608c5c3Use names for richcmp operators

comment:178 Changed 2 years ago by saraedum

  • Work issues changed from 7 failing doctests, check docbuild output for broken links, check _richcmp_ implementations to 7 failing doctests, check docbuild output for broken links

David: I made a trivial change. I don't think it needs review.

comment:179 Changed 2 years ago by roed

Yeah, that change seems fine.

comment:180 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 7 failing doctests, check docbuild output for broken links to 25 failing doctests, check docbuild output for broken links

comment:181 Changed 2 years ago by git

  • Commit changed from 608c5c3dda3ae4560bc0e88eeb440233ab5aa4bc to a63c1ffa21e57a3db4866793fee42740e01f0fbf

Branch pushed to git repo; I updated commit sha1. New commits:

b81b722Remove use of depraceted list()
2904b76Merge branch 't/14825/polynomial_representation_of_a_padic_number' into t/21869/a_framework_for_discrete_valuations_in_sage
5739955Base change factorization to the right parent
c17adf7Merge branch 'u/saraedum/factorization_over_some_quotient_rings_incorrect' of git://trac.sagemath.org/sage into t/23642/factorization_over_some_quotient_rings_incorrect
a63c1ffMerge branch 't/23642/factorization_over_some_quotient_rings_incorrect' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:182 Changed 2 years ago by git

  • Commit changed from a63c1ffa21e57a3db4866793fee42740e01f0fbf to 226518c0b8e1b01e9f047cb42d5420556d01f7e6

Branch pushed to git repo; I updated commit sha1. New commits:

28b36f8Fix ScaleAction
b6c62d0Fix doctests
99b3ab2Merge branch 't/23642/factorization_over_some_quotient_rings_incorrect' into t/21869/a_framework_for_discrete_valuations_in_sage
dad02c7Merge branch 'u/saraedum/fixed_mod_frac_field' of git://trac.sagemath.org/sage into t/23510/fixed_mod_frac_field
189ac2bAdd _test_fraction_field to the coercion tutorial
9ecff79Merge branch 't/23510/fixed_mod_frac_field' into t/21869/a_framework_for_discrete_valuations_in_sage
226518cMerge remote-tracking branch 'trac/develop' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:183 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 25 failing doctests, check docbuild output for broken links to 17 failing doctests, check docbuild output for broken links

comment:184 Changed 2 years ago by git

  • Commit changed from 226518c0b8e1b01e9f047cb42d5420556d01f7e6 to 271f6ec79d1dd5e700816a0a43dd3ed3751ecf92

Branch pushed to git repo; I updated commit sha1. New commits:

271f6ecUse a more specialized class for non-classical discrete valuations on function fields

comment:185 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 17 failing doctests, check docbuild output for broken links to 16 failing doctests, check docbuild output for broken links

New commits:

271f6ecUse a more specialized class for non-classical discrete valuations on function fields

comment:186 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 16 failing doctests, check docbuild output for broken links to 10 failing doctests, check docbuild output for broken links

comment:187 Changed 2 years ago by git

  • Commit changed from 271f6ec79d1dd5e700816a0a43dd3ed3751ecf92 to 20868e9de4fbaa180a0581f1552c02e199ee4b49

Branch pushed to git repo; I updated commit sha1. New commits:

bbca3cefill in doctest output
20868e9bring equivalence_decompositions back to non-fields

comment:188 Changed 2 years ago by saraedum

  • Work issues changed from 10 failing doctests, check docbuild output for broken links to 9 failing doctests, check docbuild output for broken links

New commits:

bbca3cefill in doctest output
20868e9bring equivalence_decompositions back to non-fields

New commits:

bbca3cefill in doctest output
20868e9bring equivalence_decompositions back to non-fields

comment:189 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:190 Changed 2 years ago by git

  • Commit changed from 20868e9de4fbaa180a0581f1552c02e199ee4b49 to 27351d9440667c18e64c5baa7e26c26e24e07918

Branch pushed to git repo; I updated commit sha1. New commits:

27351d9Bring back equivalence_decompositions into local rings

comment:191 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 9 failing doctests, check docbuild output for broken links to 8 failing doctests, check docbuild output for broken links

New commits:

27351d9Bring back equivalence_decompositions into local rings

comment:192 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620, #23642 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620, #23642, #23965

comment:193 Changed 2 years ago by git

  • Commit changed from 27351d9440667c18e64c5baa7e26c26e24e07918 to 76df6901142e9484c6969d3b4d5afeed81482982

Branch pushed to git repo; I updated commit sha1. New commits:

55b33e4Coercions to the fraction field are injective
ac081b0Merge branch 't/23965/coercion_to_fraction_field_is_injective' into t/21869/a_framework_for_discrete_valuations_in_sage
76df690Fill in doctest output

comment:194 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 8 failing doctests, check docbuild output for broken links to 7 failing doctests, check docbuild output for broken links

New commits:

55b33e4Coercions to the fraction field are injective
ac081b0Merge branch 't/23965/coercion_to_fraction_field_is_injective' into t/21869/a_framework_for_discrete_valuations_in_sage
76df690Fill in doctest output

New commits:

55b33e4Coercions to the fraction field are injective
ac081b0Merge branch 't/23965/coercion_to_fraction_field_is_injective' into t/21869/a_framework_for_discrete_valuations_in_sage
76df690Fill in doctest output

comment:195 Changed 2 years ago by saraedum

  • Dependencies changed from #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620, #23642, #23965 to #21782, #23166, #23167, #21879, #23185, #23203, #23204, #23211, #23188, #21879, #23186, #23191, #21996, #23190, #23495, #23483, #23510, #23525, #23620, #23642, #23965, #23966

comment:196 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 7 failing doctests, check docbuild output for broken links to 6 failing doctests, check docbuild output for broken links

comment:197 Changed 2 years ago by git

  • Commit changed from 76df6901142e9484c6969d3b4d5afeed81482982 to c7ac63228cc88d0e03b4cc87ed89bb5db8a16344

Branch pushed to git repo; I updated commit sha1. New commits:

87c900cnormalize() FP elements
c7ac632Merge branch 't/23966/conversion_from_zpfm_to_its_fraction_field_incorrect' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:198 Changed 2 years ago by git

  • Commit changed from c7ac63228cc88d0e03b4cc87ed89bb5db8a16344 to d9b94870dad0e5079a6eef07533af6514cdc0ca4

Branch pushed to git repo; I updated commit sha1. New commits:

a29d30aFill in correct doctest output
d9b9487Fix computations in iterated extensions of number fields

comment:199 Changed 2 years ago by git

  • Commit changed from d9b94870dad0e5079a6eef07533af6514cdc0ca4 to 6c5c12b4e18253522eb6ea52f106f7601de27b0e

Branch pushed to git repo; I updated commit sha1. New commits:

6c5c12bfill in doctest output

comment:200 Changed 2 years ago by git

  • Commit changed from 6c5c12b4e18253522eb6ea52f106f7601de27b0e to 41277b3f9070fe51be20cdc05f15f4c852e12205

Branch pushed to git repo; I updated commit sha1. New commits:

41277b3Fix factorization doctests

comment:201 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 6 failing doctests, check docbuild output for broken links to 1 failing doctest, check docbuild output for broken links

New commits:

6c5c12bfill in doctest output
41277b3Fix factorization doctests

New commits:

6c5c12bfill in doctest output
41277b3Fix factorization doctests

comment:202 Changed 2 years ago by git

  • Commit changed from 41277b3f9070fe51be20cdc05f15f4c852e12205 to 09b64022a765aa6116f10aaf846f410a074d4821

Branch pushed to git repo; I updated commit sha1. New commits:

09b6402stable factorization output

comment:203 Changed 2 years ago by git

  • Commit changed from 09b64022a765aa6116f10aaf846f410a074d4821 to dfad6f47d2e8edf2e7cb694f29cc0c4316a4dc1c

Branch pushed to git repo; I updated commit sha1. New commits:

dfad6f4Fix valuation.lift(report_coefficients=True)

comment:204 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Status changed from needs_work to needs_review
  • Work issues changed from 1 failing doctest, check docbuild output for broken links to check docbuild output for broken links

New commits:

dfad6f4Fix valuation.lift(report_coefficients=True)

New commits:

dfad6f4Fix valuation.lift(report_coefficients=True)

comment:205 Changed 2 years ago by git

  • Commit changed from dfad6f47d2e8edf2e7cb694f29cc0c4316a4dc1c to 8767fdc5d81f83172e6daac6fd1aa143fa7cb41c

Branch pushed to git repo; I updated commit sha1. New commits:

8767fdcfix docbuild

comment:206 Changed 2 years ago by git

  • Commit changed from 8767fdc5d81f83172e6daac6fd1aa143fa7cb41c to ecfe1bc9d6b021d0066269a9c21d7022e9e28e97

Branch pushed to git repo; I updated commit sha1. New commits:

ecfe1bcfix doc references

comment:207 Changed 2 years ago by saraedum

  • Work issues check docbuild output for broken links deleted

New commits:

ecfe1bcfix doc references

New commits:

ecfe1bcfix doc references

comment:208 Changed 2 years ago by git

  • Commit changed from ecfe1bc9d6b021d0066269a9c21d7022e9e28e97 to 775e093eb3076c1ebf44bd2867340605700c2a35

Branch pushed to git repo; I updated commit sha1. New commits:

775e093Remove Python2 code from valuations

comment:209 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Status changed from needs_review to needs_work
  • Work issues set to 55 failing long doctests

New commits:

775e093Remove Python2 code from valuations

comment:210 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:211 Changed 2 years ago by git

  • Commit changed from 775e093eb3076c1ebf44bd2867340605700c2a35 to 5a555ada7232c5a9ea28f073b8f6d0f26024de83

Branch pushed to git repo; I updated commit sha1. New commits:

32c4528Fix INPUT:: blocks
5a555adCatch multiple exceptions in _test_inverse

comment:212 Changed 2 years ago by git

  • Commit changed from 5a555ada7232c5a9ea28f073b8f6d0f26024de83 to 46b8c2ea13cf0e0d1288076b2d6345c71d812ab4

Branch pushed to git repo; I updated commit sha1. New commits:

46b8c2eFix _test_inverse handling of NotImplementedError

comment:213 Changed 2 years ago by saraedum

  • Description modified (diff)
  • Work issues changed from 55 failing long doctests to 8 failing long doctests

New commits:

32c4528Fix INPUT:: blocks
5a555adCatch multiple exceptions in _test_inverse
46b8c2eFix _test_inverse handling of NotImplementedError

comment:214 Changed 2 years ago by git

  • Commit changed from 46b8c2ea13cf0e0d1288076b2d6345c71d812ab4 to 716d66bd30adab7b25fb85a5dfa3e78014b243b2

Branch pushed to git repo; I updated commit sha1. New commits:

0a87df3Fix inverse() checks for ZZ
14e8f28Fix _test_mul for valuations
a1d7cd9There's only one valuation implemented on Zp(5)
377d87aFix _test_residue()
716d66bDo not test shifts for broken orders

comment:215 Changed 2 years ago by saraedum

  • Status changed from needs_work to needs_review
  • Work issues 8 failing long doctests deleted

comment:216 Changed 2 years ago by git

  • Commit changed from 716d66bd30adab7b25fb85a5dfa3e78014b243b2 to 5578cae3051b2637aa31b06a28958b8a5d739a14

Branch pushed to git repo; I updated commit sha1. New commits:

cc35bcdthere is a new section in the reference manual
5578caefix doctests in valuation/index.rst

comment:217 Changed 2 years ago by saraedum

  • Description modified (diff)

comment:218 follow-up: Changed 2 years ago by roed

In _test_shift in valuation_space.py, you're not actually doing tests, just having lines with boolean statements.

comment:219 follow-up: Changed 2 years ago by roed

In index.rst there are some \QQs that should probably be \Qs. Have you tried building the documentation recently? I'm surprised this didn't cause problems....

Is there a place I can look at the generated documentation online? It might be easier to catch some kinds of problems there.

comment:220 follow-up: Changed 2 years ago by roed

In function_field.py there's a typo in We create a valuation that correspond (should be plural).

comment:221 follow-up: Changed 2 years ago by roed

In padic_valuation.py you have self.domain()(~ZpFM(self.p(), ZZ(precision).ceil())(x)). This is probably better done with the inverse_mod method in integers, since then you don't have to construct a new parent. Also, if precision isn't integral, won't ZZ(precision) fail before you can call ceil() on it?

comment:222 follow-up: Changed 2 years ago by roed

There are a few really long lines. I don't really mind them, but they might be worth thinking about breaking up. For example,

return self.domain().change_ring(self.domain())([0 if valuations[i] > error else self._base_valuation.simplify(c, error=error-i*self._mu, force=force, phiadic=True) for (i,c) in enumerate(coefficients)])(self.phi())

comment:223 follow-up: Changed 2 years ago by roed

In the montes_factorization tests, why do you specify required precision be 0? Runtime? Feasibility of printing the result?

comment:224 follow-up: Changed 2 years ago by roed

You should probably not delete 'reference/tensor_free_modules' from sage_setup/docbuild/__init__.py.

comment:225 follow-up: Changed 2 years ago by roed

In index.rst, where you do say random output (order may vary), why not just use sorted?

comment:226 Changed 2 years ago by roed

Tests pass though! I think we're almost there.

comment:227 in reply to: ↑ 218 Changed 2 years ago by saraedum

Replying to roed:

In _test_shift in valuation_space.py, you're not actually doing tests, just having lines with boolean statements.

Yes, fixed.

comment:228 in reply to: ↑ 219 Changed 2 years ago by saraedum

Replying to roed:

In index.rst there are some \QQs that should probably be \Qs. Have you tried building the documentation recently? I'm surprised this didn't cause problems....

It did not seem to cause problems. But I changed it anyway.

Is there a place I can look at the generated documentation online? It might be easier to catch some kinds of problems there.

https://drugtestbots.info.tm/reference/valuations/index.html (still uploading)

comment:229 in reply to: ↑ 220 Changed 2 years ago by saraedum

Replying to roed:

In function_field.py there's a typo in We create a valuation that correspond (should be plural).

Ok. Fixed

comment:230 Changed 2 years ago by git

  • Commit changed from 5578cae3051b2637aa31b06a28958b8a5d739a14 to 159491957d15df472d93f9b4505d237fbb2d43c7

Branch pushed to git repo; I updated commit sha1. New commits:

175d9e1fix LaTeX errors in reference manual
0e9fa0cfix grammar in docstring
3afa634Fix handling of rationals in inverse()
b457126comment on required_precision=0
6b1f708assert something in _test_shift
1594919Fix _test_shift()

comment:231 in reply to: ↑ 221 Changed 2 years ago by saraedum

Replying to roed:

In padic_valuation.py you have self.domain()(~ZpFM(self.p(), ZZ(precision).ceil())(x)). This is probably better done with the inverse_mod method in integers, since then you don't have to construct a new parent. Also, if precision isn't integral, won't ZZ(precision) fail before you can call ceil() on it?

Thanks, all fixed.


New commits:

175d9e1fix LaTeX errors in reference manual
0e9fa0cfix grammar in docstring
3afa634Fix handling of rationals in inverse()
b457126comment on required_precision=0
6b1f708assert something in _test_shift
1594919Fix _test_shift()

New commits:

175d9e1fix LaTeX errors in reference manual
0e9fa0cfix grammar in docstring
3afa634Fix handling of rationals in inverse()
b457126comment on required_precision=0
6b1f708assert something in _test_shift
1594919Fix _test_shift()

comment:232 in reply to: ↑ 222 Changed 2 years ago by saraedum

Replying to roed:

There are a few really long lines. I don't really mind them, but they might be worth thinking about breaking up. For example,

return self.domain().change_ring(self.domain())([0 if valuations[i] > error else self._base_valuation.simplify(c, error=error-i*self._mu, force=force, phiadic=True) for (i,c) in enumerate(coefficients)])(self.phi())

I tried to fix some of these.

comment:233 in reply to: ↑ 223 Changed 2 years ago by saraedum

Replying to roed:

In the montes_factorization tests, why do you specify required precision be 0? Runtime? Feasibility of printing the result?

Because of speed. It finds an exact factorization without which takes forever (I did not wait for it to terminate) as it is very slowly improving the quality of the approximation. I added a comment.

comment:234 in reply to: ↑ 224 ; follow-up: Changed 2 years ago by saraedum

Replying to roed:

You should probably not delete 'reference/tensor_free_modules' from sage_setup/docbuild/__init__.py.

Why not? It used to be the last item in this list, now valuations has taken its spot.

Last edited 2 years ago by roed (previous) (diff)

comment:235 in reply to: ↑ 225 Changed 2 years ago by saraedum

Replying to roed:

In index.rst, where you do say random output (order may vary), why not just use sorted?

Because I want this to be a readable introduction. I think that sorting would make it much more cryptic to people who are not so fluent in python.

comment:236 Changed 2 years ago by saraedum

I think I addressed all your comments.

comment:237 Changed 2 years ago by git

  • Commit changed from 159491957d15df472d93f9b4505d237fbb2d43c7 to 20d3ad9f62502bb77b54d3b468128ba1f2ff8269

Branch pushed to git repo; I updated commit sha1. New commits:

20d3ad9broke some long lines

comment:238 in reply to: ↑ 234 Changed 2 years ago by roed

Replying to saraedum:

Replying to roed:

You should probably not delete 'reference/tensor_free_modules' from sage_setup/docbuild/__init__.py.

Why not? It used to be the last item in this list, now valuations has taken its spot.

I was confused; it's okay.

comment:239 Changed 2 years ago by roed

  • Branch changed from u/saraedum/a_framework_for_discrete_valuations_in_sage to u/roed/a_framework_for_discrete_valuations_in_sage

comment:240 Changed 2 years ago by roed

  • Commit changed from 20d3ad9f62502bb77b54d3b468128ba1f2ff8269 to 3c0dd8ff18284135983926f7c73db688a6dd4c1d
  • Status changed from needs_review to positive_review

I just merged in the last few dependencies and changed some whitespace in valuation_space.py. Looks good to me!


New commits:

3447e80Changing whitespace
a323641Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/discval
df62c48Merge branch 'u/roed/fixed_mod_frac_field' of git://trac.sagemath.org/sage into t/23965/fracfield_coercion_injective
a4d9ad6Catch exceptions raised by attempting to find fraction field
bd7d54eMerge branch 't/23965/fracfield_coercion_injective' into t/21869/discval
98d8256Merge branch 'u/roed/fixed_mod_frac_field' of git://trac.sagemath.org/sage into develop
b8de189Different fix for 23966, add doctest
3c0dd8fMerge branch 't/23966/ZpFM_to_fracfield' into t/21869/discval

comment:241 Changed 2 years ago by chapoton

Do not implement both _cmp_ and _richcmp_. The second one only please.

Last edited 2 years ago by chapoton (previous) (diff)

comment:242 Changed 2 years ago by saraedum

  • Branch changed from u/roed/a_framework_for_discrete_valuations_in_sage to u/saraedum/a_framework_for_discrete_valuations_in_sage

comment:243 Changed 2 years ago by saraedum

  • Commit changed from 3c0dd8ff18284135983926f7c73db688a6dd4c1d to bb86ca0fd179a17464c240ff703827355b1b2d27

Ok. I removed the _cmp_ implementation.


New commits:

bb86ca0Do not implement Python2 _cmp_

comment:244 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work

PDF docs don't build

comment:245 Changed 2 years ago by saraedum

  • Work issues set to PDF documentation does not build

I can not build PDF docs locally (not sure why but it's not related to this ticket.) Can someone else try to fix this?

comment:246 Changed 2 years ago by roed

I'm giving it a shot.

comment:247 Changed 2 years ago by roed

  • Branch changed from u/saraedum/a_framework_for_discrete_valuations_in_sage to u/roed/a_framework_for_discrete_valuations_in_sage

comment:248 Changed 2 years ago by roed

  • Commit changed from bb86ca0fd179a17464c240ff703827355b1b2d27 to 8ed55a6baf4f9bf1f5d250b27ccffd216c46d634

I fixed some problems, but I'm not sure that I got them all since make doc-clean && sage -docbuild reference pdf doesn't seem to see my changes (and still complains about the unicode <= symbol). I'm not sure why....


New commits:

3447e80Changing whitespace
a323641Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/discval
bd7d54eMerge branch 't/23965/fracfield_coercion_injective' into t/21869/discval
3c0dd8fMerge branch 't/23966/ZpFM_to_fracfield' into t/21869/discval
8ed55a6Fixing some pdf docbuild issues.

comment:249 Changed 2 years ago by git

  • Commit changed from 8ed55a6baf4f9bf1f5d250b27ccffd216c46d634 to e8cfff08cdad8d07b38da97e995dd32dc4f4c284

Branch pushed to git repo; I updated commit sha1. New commits:

bb2f7c2Merge commit '7c915f1213' into t/21869/a_framework_for_discrete_valuations_in_sage
4b468bcFix errors that caused trouble for docbuild
e8cfff0Merge branch 'u/roed/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:250 Changed 2 years ago by roed

  • Status changed from needs_work to needs_review

Okay, documentation builds. Julian, can you take a look at my changes?

comment:251 Changed 2 years ago by saraedum

  • Status changed from needs_review to positive_review
  • Work issues PDF documentation does not build deleted

comment:252 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work

Merge conflict

comment:253 Changed 2 years ago by saraedum

  • Branch changed from u/roed/a_framework_for_discrete_valuations_in_sage to u/saraedum/a_framework_for_discrete_valuations_in_sage

comment:254 Changed 2 years ago by saraedum

  • Commit changed from e8cfff08cdad8d07b38da97e995dd32dc4f4c284 to 6f0f9e26ae8dddd256b536c41b71c6e8ad45a2cf
  • Status changed from needs_work to positive_review

New commits:

bb86ca0Do not implement Python2 _cmp_
6f0f9e2Merge remote-tracking branch 'trac/develop' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:255 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work
[docpdf] ! Undefined control sequence.
[docpdf] l.309 induced Gauss valuation on \(\Q
[docpdf]                                      [x]\), i.e., the valuation that assigns...

comment:256 Changed 2 years ago by roed

  • Branch changed from u/saraedum/a_framework_for_discrete_valuations_in_sage to u/roed/a_framework_for_discrete_valuations_in_sage

comment:257 Changed 2 years ago by roed

  • Commit changed from 6f0f9e26ae8dddd256b536c41b71c6e8ad45a2cf to 25e48096e0237f2d00ed288689b4c2d0652d587e

Oops, I thought I had fixed that but it looks like it didn't get pushed.


New commits:

bb2f7c2Merge commit '7c915f1213' into t/21869/a_framework_for_discrete_valuations_in_sage
4b468bcFix errors that caused trouble for docbuild
3447e80Changing whitespace
a323641Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/discval
bd7d54eMerge branch 't/23965/fracfield_coercion_injective' into t/21869/discval
3c0dd8fMerge branch 't/23966/ZpFM_to_fracfield' into t/21869/discval
8ed55a6Fixing some pdf docbuild issues.
e8cfff0Merge branch 'u/roed/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage
25e4809Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:258 Changed 2 years ago by roed

  • Status changed from needs_work to needs_review

comment:259 Changed 2 years ago by roed

Julian, let me know if you're happy with these fixes. I'm currently building and running sage -docbuild reference pdf.

comment:260 Changed 2 years ago by saraedum

Looks good. Positive review if it now actually builds :)

comment:261 Changed 2 years ago by roed

  • Status changed from needs_review to positive_review

Yep, pdf documentation builds.

comment:262 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work

On some 32-bit buildbots I get

**********************************************************************
File "src/doc/en/reference/valuations/index.rst", line 36, in doc.en.reference.valuations.index
Failed example:
    w(a + 3), ww(a + 3)
Expected:
    (1, 0)
Got:
    (0, 1)
**********************************************************************
File "src/doc/en/reference/valuations/index.rst", line 38, in doc.en.reference.valuations.index
Failed example:
    w(a + 5), ww(a + 5)
Expected:
    (0, 1)
Got:
    (1, 0)
**********************************************************************
1 item had failures:
   2 of  54 in doc.en.reference.valuations.index
    [53 tests, 2 failures, 0.92 s]

comment:263 Changed 2 years ago by saraedum

  • Branch changed from u/roed/a_framework_for_discrete_valuations_in_sage to u/saraedum/a_framework_for_discrete_valuations_in_sage

comment:264 Changed 2 years ago by roed

  • Commit changed from 25e48096e0237f2d00ed288689b4c2d0652d587e to b4756f5a04be2b56735972078238283b1372c322

Is this ready for review again?


New commits:

b4756f5Make mac_lane_approximants output more stable

comment:265 Changed 2 years ago by roed

Two comments on

try:
    ret = sorted(leafs, key=lambda v: str(v))
except:
    # if for some reason the valuation can not be printed, we leave them unsorted
    ret = list(leafs)

You can just use key=str, and you shouldn't have a bare except (except Exception is probably fine).

comment:266 Changed 2 years ago by saraedum

  • Work issues set to fix bare except

Yes, you are right. I should actually know about both ;) This is not ready for review. I can't get sage to build from develop locally at all at the moment, so I wanted to see how bad such a change is (by having it be tested by the patchbots).


New commits:

b4756f5Make mac_lane_approximants output more stable

comment:267 Changed 2 years ago by roed

If you want, there's a copy in Src/sage-roed in the sd88 project on k8s; I've added you to the project so that you can access it. I'm running doctests there now.

comment:268 Changed 2 years ago by saraedum

  • Work issues fix bare except deleted

comment:269 Changed 2 years ago by git

  • Commit changed from b4756f5a04be2b56735972078238283b1372c322 to c4a9900c04eeffeb083ec59e6556a354981acaa1

Branch pushed to git repo; I updated commit sha1. New commits:

51a96efdo not catch bare exceptions
c51fb43shorten code
c4a9900Adapt doctests to sorting of mac_lane_approximants()

comment:270 Changed 2 years ago by saraedum

  • Status changed from needs_work to needs_review

comment:271 Changed 2 years ago by roed

  • Status changed from needs_review to needs_work
  • Work issues set to doctest output order

There's a doctest failure because of output ordering. I'll try to fix it, but may not have time to do it today....

comment:272 Changed 2 years ago by saraedum

  • Status changed from needs_work to needs_review
  • Work issues doctest output order deleted

comment:273 Changed 2 years ago by git

  • Commit changed from c4a9900c04eeffeb083ec59e6556a354981acaa1 to 4583f6557566fea5c01f62fab6e8d05a993da832

Branch pushed to git repo; I updated commit sha1. New commits:

4583f65fix output order

comment:274 Changed 2 years ago by roed

There seems to be a nondeterministic failure (it occurred on one of the three patchbot runs for this commit):

sage -t --long src/sage/rings/valuation/mapped_valuation.py
**********************************************************************
File "src/sage/rings/valuation/mapped_valuation.py", line 507, in sage.rings.valuation.mapped_valuation.FiniteExtensionFromInfiniteValuation.upper_bound
Failed example:
    u.upper_bound(t + 2)
Expected:
    3
Got:
    2
**********************************************************************
1 item had failures:
   1 of   8 in sage.rings.valuation.mapped_valuation.FiniteExtensionFromInfiniteValuation.upper_bound

Any ideas?

comment:275 Changed 2 years ago by saraedum

The upper bound is not non-deterministic itself. I don't remember really but I don't think that there is anything non-deterministic in my code.

comment:276 Changed 2 years ago by saraedum

The upper_bound depends on the precision of the underlying approximation, so the upper_bound can change between calls. As long as the doctests run in a deterministic order, I don't see why this should happen.

comment:277 follow-up: Changed 2 years ago by saraedum

We now have 13 good patchbot runs and one that failed. Would it be acceptable to ignore this?

comment:278 Changed 2 years ago by saraedum

I started some patchbots on my own and kicked the old results to get some more data here.

comment:279 Changed 2 years ago by git

  • Commit changed from 4583f6557566fea5c01f62fab6e8d05a993da832 to 7b19f8a8bd683ecdaf7055577753c8e3e31d2b3d

Branch pushed to git repo; I updated commit sha1. New commits:

7b19f8aMerge remote-tracking branch 'origin/develop' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:280 in reply to: ↑ 277 Changed 2 years ago by roed

  • Status changed from needs_review to positive_review

Replying to saraedum:

We now have 13 good patchbot runs and one that failed. Would it be acceptable to ignore this?

I think that's reasonable. I'm happy with this ticket, and there are lots of patchbot runs where all tests pass. Positive review from me.

comment:281 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work

I'm getting a bunch of failures of the form

File "src/sage/rings/valuation/valuation_space.py", line 506, in sage.rings.valuation.valuation_space.DiscretePseudoValuationSpace.ElementMethods.residue_ring
Failed example:
    GaussValuation(ZZ['x'], ZZ.valuation(2)).residue_ring()
Expected:
    Univariate Polynomial Ring in x over Finite Field of size 2 (using NTL)
Got:
    Univariate Polynomial Ring in x over Finite Field of size 2 (using GF2X)
**********************************************************************
1 item had failures:
   1 of   6 in sage.rings.valuation.valuation_space.DiscretePseudoValuationSpace.ElementMethods.residue_ring
    [201 tests, 1 failure, 0.56 s]

comment:282 Changed 2 years ago by saraedum

Ok. I'll try to reproduce this locally…but I have to rebuild Sage of course…

comment:283 Changed 2 years ago by saraedum

  • Status changed from needs_work to positive_review

I don't see this problem with the latest develop merged in. Could you tell us more about the machines and base revision where this is failing?

comment:284 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work

Try with the next beta

comment:285 Changed 2 years ago by git

  • Commit changed from 7b19f8a8bd683ecdaf7055577753c8e3e31d2b3d to c53f45c7fe243ba5ef2e53dd54b428d3d8218ee7

Branch pushed to git repo; I updated commit sha1. New commits:

613a524Merge remote-tracking branch 'origin/develop' into t/21869/a_framework_for_discrete_valuations_in_sage
e05c325Merge remote-tracking branch 'trac/develop' into t/21869/a_framework_for_discrete_valuations_in_sage
c53f45cFix doctests

comment:286 Changed 2 years ago by saraedum

I fixed the trivial doctest errors.

There are also a bunch of doctest errors in the patchbots that do not seem to be related to what we are doing here. I am rebuilding sage locally and report back once I have checked locally.

comment:287 Changed 2 years ago by saraedum

  • Status changed from needs_work to positive_review

comment:288 Changed 2 years ago by git

  • Commit changed from c53f45c7fe243ba5ef2e53dd54b428d3d8218ee7 to 6892bdda7b4df40403dea51a1ff31f47dba0fc0d
  • Status changed from positive_review to needs_review

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

6892bddWorkaround a noisy doctest

comment:289 Changed 2 years ago by roed

  • Status changed from needs_review to positive_review

Looks good to me.

comment:290 Changed 2 years ago by vbraun

  • Status changed from positive_review to needs_work
[docpdf] ! Missing $ inserted.
[docpdf] <inserted text> 
[docpdf]                 $
[docpdf] l.1203 relation \sphinxcode{P.v ≤ V.v}
[docpdf]                                          (pointwise on the polynomial ring K...

comment:291 Changed 2 years ago by git

  • Commit changed from 6892bdda7b4df40403dea51a1ff31f47dba0fc0d to e78174232093e0931514f6fc6038be22a2bc4e0c

Branch pushed to git repo; I updated commit sha1. New commits:

e781742Fix documentation build issues

comment:292 Changed 2 years ago by saraedum

  • Status changed from needs_work to needs_review

comment:293 Changed 2 years ago by git

  • Commit changed from e78174232093e0931514f6fc6038be22a2bc4e0c to e01a3ab0fbd7f40532395063e2cc7e9defd579a0

Branch pushed to git repo; I updated commit sha1. New commits:

e01a3abRequire incomparability

comment:294 Changed 2 years ago by saraedum

The changes could make some outputs gain precision. Let's see what the patchbots think…

comment:295 Changed 2 years ago by roed

There's an extra space in the new commit: 10*x^1 2.

comment:296 Changed 2 years ago by git

  • Commit changed from e01a3ab0fbd7f40532395063e2cc7e9defd579a0 to 21428911816e42073feb06eaa3867b55bfd2e31a

Branch pushed to git repo; I updated commit sha1. New commits:

2142891fix copy & paste mistake

comment:297 Changed 2 years ago by roed

I'm trying to build the documentation now to check the previous change.


New commits:

2142891fix copy & paste mistake

comment:298 Changed 23 months ago by saraedum

  • Status changed from needs_review to needs_work
File "src/sage/rings/padics/padic_valuation.py", line 726, in sage.rings.padics.padic_valuation.pAdicValuation_base.extensions
Failed example:
    QQ.valuation(2).extensions(L)
Expected:
    Traceback (most recent call last):
    ...
    ValueError: The valuation [ Gauss valuation induced by 2-adic valuation, v(x) = 1/2 ] does not approximate a unique extension of 2-adic valuation with respect to x^4 + 2*x^3 + 2*x^2 + 8
Got:
    [[ 2-adic valuation, v(x + 2) = 3/2 ]-adic valuation,
     [ 2-adic valuation, v(x) = 1/2 ]-adic valuation]

comment:299 Changed 23 months ago by git

  • Commit changed from 21428911816e42073feb06eaa3867b55bfd2e31a to 4900f56945720dba8eeaff3be5d5d4c18e443b0f

Branch pushed to git repo; I updated commit sha1. New commits:

963ceb7Fix incorrect doctest output
4900f56Merge branch 'u/saraedum/a_framework_for_discrete_valuations_in_sage' of git://trac.sagemath.org/sage into t/21869/a_framework_for_discrete_valuations_in_sage

comment:300 Changed 23 months ago by saraedum

  • Status changed from needs_work to needs_review

comment:301 Changed 23 months ago by git

  • Commit changed from 4900f56945720dba8eeaff3be5d5d4c18e443b0f to 24807e364bb0ac04e998465ad7c25ee28d16fbf8

Branch pushed to git repo; I updated commit sha1. New commits:

24807e3Merge remote-tracking branch 'trac/develop' into t/21869/a_framework_for_discrete_valuations_in_sage

comment:302 Changed 23 months ago by saraedum

  • Description modified (diff)
Note: See TracTickets for help on using tickets.