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

feat(ring_theory/matrix): category of free modules; scalar matrices #375

Closed
wants to merge 7 commits into from

Conversation

jcommelin
Copy link
Member

Do we want to define the identity matrix in terms of scalar?

TO CONTRIBUTORS:

Make sure you have:

  • reviewed and applied the coding style: coding, naming
  • make sure definitions and lemmas are put in the right files
  • make sure definitions and lemmas are not redundant

For reviewers: code review check list

@johoelzl
Copy link
Collaborator

johoelzl commented Oct 2, 2018

These are finitely generated free modules, aren't they?
Another option would be to use finsupp, and index by arbitrary types.
scalar is quiet unrelated to the category itself.

@jcommelin
Copy link
Member Author

Yes, these are indeed finitely generated free modules. For arbitrary types we wouldn't have matrices as morphisms. (At least currently, matrices are only indexed by fintypes.)
I agree that scalar is unrelated. I do think it is useful. I was just trying to get stuff from kbb into mathlib. If thought these were too trivial to split.

@johoelzl
Copy link
Collaborator

johoelzl commented Oct 2, 2018

So should it be called different?

@jcommelin
Copy link
Member Author

Well, it is in the matrix namespace, so in that case I think it is fine. But we could call it finite_free_module or something like that...

@semorrison
Copy link
Collaborator

semorrison commented Oct 2, 2018 via email

@jcommelin
Copy link
Member Author

Sure, it's your code anyway (-; Feel free to reclaim it and push to the matrix branch on the community fork.

@digama0
Copy link
Member

digama0 commented Oct 8, 2018

Superceded by #406 and #404. Try again later for the free_module stuff when we have a clearer idea of what to do with it.

@digama0 digama0 closed this Oct 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants