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
[Merged by Bors] - feat: define reflexive modules and prove basics of perfect pairings #4989
Conversation
ocfnash
commented
Jun 12, 2023
•
edited
edited
…nd prove basics of perfect pairings
@ocfnash It would be really cool if we could generalize the statement of Maybe something like this, but it's not fully fleshed out:
Note that, in functional analysis, the condition you provided for |
@j-loreaux Thanks for the remarks, I like the idea but I don't see an easy way to implement it (I had actually thought a bit about this while making this PR). For me the issue is that I don't see an easy way to abstract over the type of dual we're talking about. For example, the type (R : Type u_1) → (M : Type u_2) → [CommSemiring R] → [AddCommMonoid M] → [Module R M] → Type (max u_2 u_1) I expect we could unify things using our category theory library but I don't think it would be worth it. I think it is better just to have separate theories for this purely algebraic concept (which is called reflexive) and the analytic versions (with lemmas connecting them in, say, finite dimensions). This is also what we do elsewhere, e.g., we have purely algebraic tensor products. |
This branch is quite old so probably a good idea to merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors d+
I gave up on ULift
because I think we're missing some machinery, and it's not interesting enough to bother with here.
✌️ ocfnash can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
…4989) Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
…4989) Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
…4989) Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
…4989) Co-authored-by: Eric Wieser <wieser.eric@gmail.com>