[Merged by Bors] - feat: InnerProductSpace (SeparationQuotient E)#17576
[Merged by Bors] - feat: InnerProductSpace (SeparationQuotient E)#17576eric-wieser wants to merge 2 commits intomasterfrom
InnerProductSpace (SeparationQuotient E)#17576Conversation
eric-wieser
commented
Oct 9, 2024
| theorem Inseparable.inner_eq_inner {x₁ x₂ y₁ y₂ : E} | ||
| (hx : Inseparable x₁ x₂) (hy : Inseparable y₁ y₂) : | ||
| inner x₁ y₁ = (inner x₂ y₂ : 𝕜) := | ||
| ((hx.prod hy).map continuous_inner).eq |
There was a problem hiding this comment.
Thanks @ADedecker for pointing out that this map and .eq API existed!
PR summary e347b5a8ebImport changes for modified filesNo significant changes to the import graph Import changes for all files
|
|
Sorry, could you explain quickly what is the relation between this and #16707 ? Also, @yoh-tanimoto, I see you have the GNS in mind, do you already have some working code ? I was planning on doing this before the summer, but I don't want to step on your toes. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
My thought here is that #16707 could probably cover the equivalence between |
|
Ah, I see. My opinion on this is "just use bors d+ |
|
✌️ eric-wieser can now approve this pull request. To approve and merge a pull request, simply reply with |
|
bors merge
I think it would be nice to state simply to show equivalence, but probably if @yoh-tanimoto has further goals it isn't actually necessary to reach them. |
|
you two are very quick, and the code is so short, I'm impressed! @ADedecker No, I don't have a single line of code of GNS. I'm not sure what I will work on next, so if you wish, please go ahead. I don't know whether other constructions (by Should I use #16707 for this purpose, or open a new one? |
|
Pull request successfully merged into master. Build succeeded: |
InnerProductSpace (SeparationQuotient E)InnerProductSpace (SeparationQuotient E)