Adapt unitary burau matrix to Sage convention

[soehms]

The unitary Burau matrix has been implemented in #25760. The choice between the two adjoined possibilities has been made according to the used literature ([Squ1984]_).

Unfortunately, this is opposite to the convention used in Sage for example in :meth:_check_matrix of :class:UnitaryMatrixGroup_generic.

This discrepancy is corrected by the ticket. A deprecation of the former behavior doesn't seem to be necessary, since the new functionality has been released just a couple of months ago.

CC: @tscrim

Component: group theory

Keywords: braid, burau, unitary

Author: Sebastian Oehms

Branch/Commit: a05e3e4

Reviewer: Travis Scrimshaw

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

[soehms]

Branch: u/soehms/adapt_unitary_burau-26657

[soehms]

Commit: be4674b

[soehms]

New commits:

be4674b

initial implementation

[tscrim]

comment:3

Sorry, I lost track of this ticket. LGTM.

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:4

One little fix needed: :trac:`25760 -> :trac:`25760`. Once done, you can set to a positive review.

[soehms]

Changed branch from u/soehms/adapt_unitary_burau-26657 to u/soehms/adapt_unitary_burau_26657

[soehms]

Changed commit from be4674b to none

[sagetrac-git]

Commit: b36eca1

[sagetrac-git]

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

[sagetrac-git]

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

a05e3e4

correct typset error in new branch to avoid merge error

[sagetrac-git]

Changed commit from b36eca1 to a05e3e4

[soehms]

comment:8

Replying to @tscrim:

Sorry, I lost track of this ticket. LGTM.

No Problem!

Thank you, Travis! I will be waiting for the patchbot to see if I got rid of that merge error.

[vbraun]

Changed branch from u/soehms/adapt_unitary_burau_26657 to a05e3e4