Ticket #3601 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

[with patch, positive review] Reimplementation of tensor products

Reported by: aschilling Owned by: Mike Hansen
Priority: minor Milestone: sage-3.0.6
Component: combinatorics Keywords: tensor products of crystals
Cc: sage-combinat-commits@… Author(s):
Report Upstream: Reviewer(s):
Merged in: Work issues:

Description

I split TensorProductOfCrystals? into TensorProductOfCrystalsWithGenerators? and FullTensorProductOfCrystals? (with or without the option of being a classical crystal). This makes it possible to have a more efficient way to access list, count, etc for tensor products that do not have module generators.

Also, the distinction between the specifications of module_generators and highest_weight_vectors is made more precise (which made it necessary to slightly change the implementation of Daniel Bump on characters).

This change is necessary for the upcoming implementation of affine crystals.

Attachments

tensor_product-for_trac.patch Download (9.4 KB) - added by aschilling 2 years ago.
tensor_product-3601-review.patch Download (5.5 KB) - added by mhansen 2 years ago.

Change History

Changed 2 years ago by aschilling

Changed 2 years ago by mhansen

Changed 2 years ago by mhansen

  • summary changed from Reimplementation of tensor products to [with patch, under review by mhansen] Reimplementation of tensor products

Hi Anne,

I added a few doctests and fixed some whitespace issues (there were tabs instead of spaces). If you're fine with these changes, I'll go ahead and give it a positive review so it can get merged.

--Mike

Changed 2 years ago by aschilling

Hi Mike,

Yes, your changes look fine to me (I wonder how the tabs instead of spaces happened to be wrong?).

Anne

Changed 2 years ago by mhansen

  • summary changed from [with patch, under review by mhansen] Reimplementation of tensor products to [with patch, positive by mhansen] Reimplementation of tensor products

I'm not sure. Usually the editor picks that up just fine.

Changed 2 years ago by mabshoff

Anne, Mike,

to be 100% clear: both patches should be applied in this case?

Cheers,

Michael

Changed 2 years ago by mhansen

Yes, both patches.

Changed 2 years ago by mabshoff

  • status changed from new to closed
  • resolution set to fixed
  • summary changed from [with patch, positive by mhansen] Reimplementation of tensor products to [with patch, positive review] Reimplementation of tensor products

Merged in Sage 3.0.6.alpha0

Note: See TracTickets for help on using tickets.