New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Weyl groups are mplemented in rootsystem.py #2864
Comments
comment:1
Attachment: weylgroup3.patch.gz |
comment:2
I realize this is "not ready for review" but I tested it out anyway. It down not apply to sage-3.0.alpga3 but can be applied "manually" (namely, read the man page for "patch" ...). I just checked that the above linked-to email does what it claims this patch does:
Very nice so far! I'm posting the rebased patch to |
comment:3
Replying to @wdjoyner:
It does apply directly to 3.0.alpha3 since it's a patch against 3.0.alpha3. From the combinat directory "patch <weylgroup3.patch" applies cleanly. I'm unsure why Michael Abshoff described it as not ready for review. One possible reason is that Mike Hanson intends reorganization of root_system.py. |
comment:4
"From the combinat directory "patch <weylgroup3.patch" applies cleanly." I would say that is the definition of "applies manually", so i agree with you!
Did you create your patch using hg_sage.commit, hg_sage.bundle, hg_sage.log, hg_sage.export, as described (well, more-or-less described :-) on http://www.sagemath.org/doc/html/prog/node72.html ? If yes, then maybe my download was corrupted or something. |
comment:5
Replying to @dwbump:
Hi Dan, this was the precise reason I marked it this way and I assume Mike will take care of this later. Cheers, Michael |
comment:6
Replying to @wdjoyner: The patch was made by running "hg diff". Michael Abshoff told me on IRC not to make patches that way so thanks for the doc link. But to be clear, the patch didn't need to be rebased since it is a patch against 3.0-alpha3. |
comment:7
Attachment: 2864.patch.gz I've done some work on the the patch (for example, caching the Weyl group, adding doctests to all the functions, etc.) I've also put all the root_system code in combinat/root_system/ which will be necessary as it grows. This patch is made against 3.0.alpha3. |
comment:8
It's a big patch. I'll look at it today. Maybe Justin Walker should be added to the copyright in one file since he did a lot of work to code the exceptional root systems. |
comment:9
Attachment: 2864a.patch.gz Although the patch appears large, it is mostly reorganization. Apart from moving files around, a hash method is added to CartanType_simple and Weyl Groups are cached in improvement over the original patch. Two minor problems: (1) the G2 patch (track #2808) was accidentally reverted. These are addressed by 2864a.patch which goes on top of 2864.patch. My recommendation would be to merge 2864.patch + 2864a.patch. Dan |
Attachment: 2864.2.patch.gz |
comment:11
Attachment: 2864b.patch.gz Merged 2864.patch, 2864a.patch and 2864b.patch in Sage 3.0.alpha4 |
The original Weyl group patch was described here:
http://groups.google.com/group/sage-combinat-devel/browse_thread/thread/80515528dc9648fd?hl=en
No trac ticket was made, and the patch no longer applies without modification so I
revised it and am uploading it as this ticket.
Mike Hansen told me he wants to revise the patch in connection with a reorganization
in which case this ticket might be superfluous. However I added one thing, which is
instructions for obtaining the character tables.
CC: @sagetrac-sage-combinat
Component: combinatorics
Issue created by migration from https://trac.sagemath.org/ticket/2864
The text was updated successfully, but these errors were encountered: