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.

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

CartanMatrix class #14137

Closed
stumpc5 opened this issue Feb 15, 2013 · 18 comments
Closed

CartanMatrix class #14137

stumpc5 opened this issue Feb 15, 2013 · 18 comments

Comments

@stumpc5
Copy link
Contributor

stumpc5 commented Feb 15, 2013

A class for Cartan matrices

Apply attachment: trac_14137-cartan_matrix_class-ts.patch

Depends on #2023
Depends on #14314
Depends on #14673

CC: @sagetrac-sage-combinat @nthiery erichmond@math.ubc.ca @tscrim

Component: combinatorics

Keywords: cartan matrix, days45, days49

Author: Travis Scrimshaw

Reviewer: Ben Salisbury

Merged: sage-5.11.beta2

Issue created by migration from https://trac.sagemath.org/ticket/14137

@tscrim
Copy link
Collaborator

tscrim commented Feb 19, 2013

Changed keywords from cartan matrix, sagedays45 to cartan matrix, days45

@tscrim
Copy link
Collaborator

tscrim commented Feb 19, 2013

comment:1

Attachment: cartan_matrix.py.gz

@tscrim
Copy link
Collaborator

tscrim commented Mar 7, 2013

comment:2

Are you trying to factor this part out of #11187?

@stumpc5
Copy link
Contributor Author

stumpc5 commented Mar 7, 2013

comment:3

Replying to @tscrim:

Are you trying to factor this part out of #11187?

Yes -- if you want, you are very welcome to take it over!

@tscrim
Copy link
Collaborator

tscrim commented Apr 9, 2013

Dependencies: #2023 #14314

@tscrim
Copy link
Collaborator

tscrim commented Apr 9, 2013

Author: Travis Scrimshaw

@tscrim

This comment has been minimized.

@tscrim
Copy link
Collaborator

tscrim commented Apr 12, 2013

@tscrim

This comment has been minimized.

@tscrim
Copy link
Collaborator

tscrim commented Apr 12, 2013

comment:5

New version of the patch.

For patchbot:

Apply: trac_14137-cartan_matrix_class-ts.2.patch

@tscrim
Copy link
Collaborator

tscrim commented Apr 14, 2013

comment:6

Made some tweaks to fix doctests.

For patchbot:

Apply: trac_14137-cartan_matrix_class-ts.patch

@tscrim

This comment has been minimized.

@tscrim
Copy link
Collaborator

tscrim commented Jun 1, 2013

comment:7

Rebased over #14673.

Apply: trac_14137-cartan_matrix_class-ts.patch

@tscrim
Copy link
Collaborator

tscrim commented Jun 1, 2013

Changed dependencies from #2023 #14314 to #2023 #14314 #14673

@tscrim
Copy link
Collaborator

tscrim commented Jun 17, 2013

comment:8

Attachment: trac_14137-cartan_matrix_class-ts.patch.gz

New version which fixes some bugs Ben found.

Apply: trac_14137-cartan_matrix_class-ts.patch

@tscrim
Copy link
Collaborator

tscrim commented Jun 17, 2013

Changed keywords from cartan matrix, days45 to cartan matrix, days45, days49

@bsalisbury1
Copy link

Reviewer: Ben Salisbury

@jdemeyer
Copy link

Merged: sage-5.11.beta2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants