Opened 11 years ago

Last modified 11 years ago

#10536 closed enhancement

Add vector conjugation — at Version 6

Reported by: rbeezer Owned by: jason, was
Priority: major Milestone: sage-4.7
Component: linear algebra Keywords:
Cc: kcrisman Merged in:
Authors: Rob Beezer Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by flawrence)

As for matrices, this method conjugates every element of a vector, by appealing to the conjugate() method of the base ring. This is a prelude to a proper inner product for vectors over the complex numbers.

For the record, making a list of new entries and creating a new vector, is much faster than making a copy and changing it in-place.

To the release manager: apply trac_10536-vector-conjugation-v3.patch only.

Change History (9)

Changed 11 years ago by rbeezer

comment:1 Changed 11 years ago by rbeezer

  • Authors set to Rob Beezer
  • Status changed from new to needs_review

comment:2 Changed 11 years ago by rbeezer

  • Status changed from needs_review to needs_work

Parents aren't being preserved correctly. New patch in just a bit.

Changed 11 years ago by rbeezer

comment:3 Changed 11 years ago by rbeezer

  • Status changed from needs_work to needs_review

This now depends on #10537, so that sparse vectors behave properly once rebuilt after being manipulated.

Depends on #10537

Apply trac_10536-vector-conjugation-v2.patch

comment:4 Changed 11 years ago by kcrisman

  • Cc kcrisman added

Changed 11 years ago by rbeezer

comment:5 Changed 11 years ago by rbeezer

A block of doctests slipped through without any output. v3 patch corrects that, and is the only patch necessary to apply.

Apply: trac_10536-vector-conjugation-v3.patch

comment:6 Changed 11 years ago by flawrence

  • Description modified (diff)
  • Status changed from needs_review to positive_review

It makes sense, works, passes doctests, and is well documented. Positive review.

Note: See TracTickets for help on using tickets.