Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove coxeter matrix implementation for type H (the generic implementation is just as good) #14174

Closed
nthiery opened this issue Feb 24, 2013 · 7 comments
Closed

Remove coxeter matrix implementation for type H (the generic implementation is just as good) #14174

nthiery opened this issue Feb 24, 2013 · 7 comments
Assignees
@sagetrac-sage-combinat
Labels
c: combinatorics p: major / 3 t: enhancement
Milestone
sage-5.8

Comments

@nthiery
Copy link
Contributor

nthiery commented Feb 24, 2013

This is a followup to #12912 where this method was inadvertently not removed.

Depends on #12912

Component: combinatorics

Keywords: coxeter3

Author: Nicolas M. Thiéry

Reviewer: Anne Schilling

Merged: sage-5.8.beta2

Issue created by migration from https://trac.sagemath.org/ticket/14174
@nthiery nthiery added this to the sage-5.8 milestone Feb 24, 2013
@nthiery
Copy link
Contributor Author

nthiery commented Feb 24, 2013

Attachment: trac_14174-coxeter_matrix-type_H.patch.gz

@nthiery

This comment has been minimized.

Sign in to view
@anneschilling
Copy link

Changed keywords from none to coxeter3

@anneschilling
Copy link

Dependencies: #12912

@anneschilling
Copy link

comment:3

If the patchbot turns green, I am happy with this!

Anne

@nthiery
Copy link
Contributor Author

nthiery commented Feb 26, 2013

comment:5

For the record: the patchbot failure comes from it complaining about not knowing how to handle the spkg for #12912

@jdemeyer
Copy link

Merged: sage-5.8.beta2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-sage-combinat sagetrac-sage-combinat

Labels
c: combinatorics p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-5.8
Development

No branches or pull requests
3 participants
@nthiery @anneschilling @jdemeyer