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

Implement Similarity Class Types #14907

Closed
amritanshu-prasad mannequin opened this issue Jul 18, 2013 · 15 comments
Closed

Implement Similarity Class Types #14907

amritanshu-prasad mannequin opened this issue Jul 18, 2013 · 15 comments

Comments

@amritanshu-prasad
Copy link
Mannequin

amritanshu-prasad mannequin commented Jul 18, 2013

Implement similarity class types for square matrices with entries in a finite field.


Apply: attachment: trac_14907-similarity_classes-ap.patch

Depends on #10630

CC: @sagetrac-sage-combinat @tscrim

Component: combinatorics

Keywords: similarity class type finite field

Author: Amritanshu Prasad

Reviewer: Travis Scrimshaw

Merged: sage-5.12.beta2

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

@amritanshu-prasad amritanshu-prasad mannequin added this to the sage-5.11 milestone Jul 18, 2013
@amritanshu-prasad amritanshu-prasad mannequin self-assigned this Jul 18, 2013
@tscrim
Copy link
Collaborator

tscrim commented Jul 18, 2013

comment:2

Looks good to me. Thanks.

@tscrim
Copy link
Collaborator

tscrim commented Jul 18, 2013

Changed reviewer from tscrim to Travis Scrimshaw

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Jul 20, 2013

comment:3

Does this patch have a dependency on #10630 and maybe something else? Something is wrong because I get errors.
One of the tests from sage.combinat.matrix_type import primitives I get the error:
ImportError: No module named matrix_type

I don't even know where to look for these. Can someone point me to the right patches and maybe how to even look this up myself?

@tscrim
Copy link
Collaborator

tscrim commented Jul 20, 2013

Attachment: trac_14907-review-ts.patch.gz

@tscrim
Copy link
Collaborator

tscrim commented Jul 20, 2013

comment:4

I was working with Armi on this patch and we forgot to run sync-build after changing the name of the file (hence this was still there as a compiled file). Here's the small review patch which fixes this.

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Jul 20, 2013

Dependencies: #10630

@zabrocki
Copy link
Mannequin

zabrocki mannequin commented Jul 20, 2013

comment:5

And the dependency? I am right about this, right?

@tscrim
Copy link
Collaborator

tscrim commented Jul 20, 2013

comment:6

I don't think so...but it's better safe than sorry.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Jul 21, 2013
@amritanshu-prasad
Copy link
Mannequin Author

amritanshu-prasad mannequin commented Jul 22, 2013

comment:8

I will be adding some new methods and functions which get used frequently with these classes.

@amritanshu-prasad
Copy link
Mannequin Author

amritanshu-prasad mannequin commented Jul 24, 2013

comment:9

Attachment: trac_14907-similarity_classes-ap.patch.gz

I think I am done with it for now.

@tscrim
Copy link
Collaborator

tscrim commented Jul 24, 2013

comment:10

Looks good to me. Thanks.

@jdemeyer
Copy link

jdemeyer commented Aug 2, 2013

comment:11

Please clarify which patches should be applied.

@tscrim
Copy link
Collaborator

tscrim commented Aug 2, 2013

comment:12

Forgot to add that. Sorry.

For patchbot:

Apply: trac_14907-similarity_classes-ap.patch

@tscrim

This comment has been minimized.

@jdemeyer
Copy link

Merged: sage-5.12.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

2 participants