Skip to content

Commit

Permalink
sagemathgh-38450: fix annotations in sage/matrix/matrix2.pyx and `s…
Browse files Browse the repository at this point in the history
…age/rings/integer.pyx`

Fix some annotations issues reported by some CI.

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [ ] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->

URL: sagemath#38450
Reported by: David Coudert
Reviewer(s): Kwankyu Lee
  • Loading branch information
Release Manager committed Jul 31, 2024
2 parents 4b20d0c + 5e16982 commit 8655ba2
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 10 deletions.
4 changes: 2 additions & 2 deletions build/pkgs/configure/checksums.ini
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
tarball=configure-VERSION.tar.gz
sha1=d21dcef9ea75fb52a89af8592796f8a2d97d3f2f
sha256=ce97f7eecff470cc7dc4c6dce7f0e5e7e85382c2411b23bc6eb9c6934b002ada
sha1=ace59bffb9de534f3561c0c145cb4e4046cd2360
sha256=c4ff877f3b8b9bb345726bdc0de1198e9e457efceb5ff7d509b5857d6f648c3c
2 changes: 1 addition & 1 deletion build/pkgs/configure/package-version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
f473d0954b37ab99d3530145c0f4536e4f0d88c9
4c1f7b23172f8beeac593ef18a35a5080556727c
2 changes: 0 additions & 2 deletions src/sage/matrix/matrix2.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -10548,7 +10548,6 @@ cdef class Matrix(Matrix1):
[ 0 0 5.444401659866974? 5.468660610611130? -0.6827161852283857?]
[ 0 0 0 1.027626039419836? -3.619300149686620?]
[ 0 0 0 0 0.024551430807012?]

sage: Q.conjugate_transpose()*Q
[1.000000000000000? 0.?e-18 0.?e-17 0.?e-16 0.?e-13]
[ 0.?e-18 1.000000000000000? 0.?e-17 0.?e-16 0.?e-13]
Expand Down Expand Up @@ -10583,7 +10582,6 @@ cdef class Matrix(Matrix1):
[ 0.?e-18 + 0.?e-17*I 1.000000000000000? + 0.?e-17*I 0.?e-17 + 0.?e-17*I 0.?e-16 + 0.?e-16*I]
[ 0.?e-17 + 0.?e-17*I 0.?e-17 + 0.?e-17*I 1.000000000000000? + 0.?e-17*I 0.?e-16 + 0.?e-16*I]
[ 0.?e-16 + 0.?e-16*I 0.?e-16 + 0.?e-16*I 0.?e-16 + 0.?e-16*I 1.000000000000000? + 0.?e-16*I]

sage: Q*R - A
[ 0.?e-17 0.?e-17 + 0.?e-17*I 0.?e-16 + 0.?e-16*I 0.?e-16 + 0.?e-16*I]
[ 0.?e-18 0.?e-17 + 0.?e-17*I 0.?e-16 + 0.?e-16*I 0.?e-16 + 0.?e-16*I]
Expand Down
11 changes: 6 additions & 5 deletions src/sage/rings/integer.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -592,12 +592,13 @@ cdef class Integer(sage.structure.element.EuclideanDomainElement):
Test comparisons with numpy types (see :issue:`13386` and :issue:`18076`)::
sage: import numpy # needs numpy
sage: if int(numpy.version.short_version[0]) > 1: # needs numpy
....: numpy.set_printoptions(legacy="1.25") # needs numpy
sage: numpy.int8('12') == 12 # needs numpy
sage: # needs numpy
sage: import numpy
sage: if int(numpy.version.short_version[0]) > 1:
....: numpy.set_printoptions(legacy="1.25")
sage: numpy.int8('12') == 12
True
sage: 12 == numpy.int8('12') # needs numpy
sage: 12 == numpy.int8('12')
True
sage: float('15') == 15
Expand Down

0 comments on commit 8655ba2

Please sign in to comment.