Opened 7 years ago

Closed 7 years ago

#16876 closed defect (fixed)

Fix cocharge on A2Dual rigged configurations and better support for D_4^(3) and relabelled types

Reported by: tscrim Owned by: sage-combinat
Priority: critical Milestone: sage-6.4
Component: combinatorics Keywords: rigged configurations
Cc: sage-combinat, aschilling Merged in:
Authors: Travis Scrimshaw Reviewers: Anne Schilling
Report Upstream: N/A Work issues:
Branch: 208ee57 (Commits, GitHub, GitLab) Commit: 208ee57b45eb850917a9f55200d8d70622168117
Dependencies: Stopgaps:

Status badges

Description (last modified by tscrim)

Currently, cocharge is not calculated correctly for type A2n(2)\dagger. The cocharge should be half of the virtual. 

sage: RC = RiggedConfigurations(CartanType(['A',4,2]).dual(), [[1,1],[2,2]])
sage: for mg in RC.module_generators:
....:     mg.cocharge()
....:     mg.to_virtual_configuration().cocharge()
....:     print ""
0
0

1/2
2

0
2

1/2
2

1/2
4

0
4

1
6

Also rigged configurations for type D4(3) and other relabelled types aren't well supported, but is solved with better handling of the index set. 

sage: RC = RiggedConfigurations(['D',4,3], [[1,1]])
sage: RC.module_generators
(
   
(/)
   
(/)
   
)

There should be another generator.

This also makes use_half_width_boxes_type_B into a global option (as it should've been done in #16324, but I forgot). It also makes the latex of a rigged configuration respect the French convention.

Change History (10)

comment:1 Changed 7 years ago by tscrim

  • Branch set to public/rigged_configurations/fixes-16876
  • Commit set to 3614310c58e08b05b9e1190f69f07d7e9cf92857
  • Status changed from new to needs_review

New commits:

​3614310Fixed cocharge for A2Dual and better support for D_4^(3) and relabelled types.

comment:2 Changed 7 years ago by git

  • Commit changed from 3614310c58e08b05b9e1190f69f07d7e9cf92857 to bdd64c38e2c6ec4565240d7796e00b6092f35185

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

​bdd64c3Made half with type B into a global option and French convention latex fix.

comment:3 Changed 7 years ago by tscrim

  • Description modified (diff)

comment:4 follow-up: Changed 7 years ago by git

  • Commit changed from bdd64c38e2c6ec4565240d7796e00b6092f35185 to 9ead54d1287a9fa0a807471b1d06633a62b782dd

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

​e33ecf2Merge branch 'develop' into public/rigged_configurations/fixes-16876
​25ed446Merge branch 'develop' into public/rigged_configurations/fixes-16876
​9ead54dUpdating references.

comment:5 in reply to: ↑ 4 Changed 7 years ago by aschilling

Hi Travis,

Here is a first comment:

Please provide the formula you are using for cocharge in the cocharge method in /src/sage/combinat/rigged_configurations/rigged_configuration_element.py or provide a reference!

Anne

comment:6 Changed 7 years ago by git

  • Commit changed from 9ead54d1287a9fa0a807471b1d06633a62b782dd to a0f9517360ceb18aea9dcdd6efe65e86639bed7a

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

​a0f9517Added some documentation to cocharge.

comment:7 Changed 7 years ago by tscrim

Done.

comment:8 Changed 7 years ago by git

  • Commit changed from a0f9517360ceb18aea9dcdd6efe65e86639bed7a to 208ee57b45eb850917a9f55200d8d70622168117

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

​208ee57Merge branch 'public/rigged_configurations/fixes-16876' of trac.sagemath.org:sage into public/rigged_configurations/fixes-16876

comment:9 Changed 7 years ago by aschilling

  • Keywords rigged configurations added
  • Reviewers set to Anne Schilling
  • Status changed from needs_review to positive_review

comment:10 Changed 7 years ago by vbraun

  • Branch changed from public/rigged_configurations/fixes-16876 to 208ee57b45eb850917a9f55200d8d70622168117
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.