Fixing determinant for the zero matrix over the p-adics

[edgarcosta]

This fixes the bug below:

sage: matrix(Qp(37),[0]).determinant()
---------------------------------------------------------------------------
UnboundLocalError                         Traceback (most recent call last)
<ipython-input-323-f9ac4bb66315> in <module>()
----> 1 matrix(Zp(Integer(37)),[Integer(0)]).determinant()

/Applications/sage-dev/local/lib/python3.7/site-packages/sage/matrix/matrix2.pyx in sage.matrix.matrix2.Matrix.determinant (build/cythonized/sage/matrix/matrix2.c:14548)()
   1640         R = self._base_ring
   1641         if hasattr(R, '_matrix_determinant'):
-> 1642             d = R._matrix_determinant(self)
   1643             self.cache('det', d)
   1644             return d

/Applications/sage-dev/local/lib/python3.7/site-packages/sage/rings/padics/local_generic.py in _matrix_determinant(self, M)
   1587                 break
   1588             val = curval
-> 1589             if S[pivi,pivj] == 0:
   1590                 if track_precision:
   1591                     return R(0, valdet + (n-piv)*val - shift)

UnboundLocalError: local variable 'pivi' referenced before assignment

Component: padics

Author: Edgar Costa

Branch/Commit: d5d16fe

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/29249

[sagetrac-git]

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

6befbbe

fixing determinant for the zero matrix

[sagetrac-git]

Changed commit from 268f41a to 6befbbe

[sagetrac-git]

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

d5d16fe

fixing pivi, pivj

[sagetrac-git]

Changed commit from 6befbbe to d5d16fe

[edgarcosta]

Author: Edgar Costa

[edgarcosta]

New commits:

d5d16fe

fixing pivi, pivj

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:4

LGTM.

[vbraun]

Changed branch from u/edgarcosta/padicdet to d5d16fe