Skip to content

[Merged by Bors] - feat: InnerProductSpace (SeparationQuotient E)#17576

Closed
eric-wieser wants to merge 2 commits intomasterfrom
eric-wieser/InnerProductSpace-separation
Closed

[Merged by Bors] - feat: InnerProductSpace (SeparationQuotient E)#17576
eric-wieser wants to merge 2 commits intomasterfrom
eric-wieser/InnerProductSpace-separation

Conversation

@eric-wieser
Copy link
Copy Markdown
Member


Open in Gitpod

@eric-wieser eric-wieser added the t-analysis Analysis (normed *, calculus) label 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
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ADedecker for pointing out that this map and .eq API existed!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 9, 2024

PR summary e347b5a8eb

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ Inseparable.inner_eq_inner
+ inner_mk_mk
+ instance : Inner 𝕜 (SeparationQuotient E)
+ instance : InnerProductSpace 𝕜 (SeparationQuotient E)

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.

@ADedecker
Copy link
Copy Markdown
Member

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>
@eric-wieser
Copy link
Copy Markdown
Member Author

My thought here is that #16707 could probably cover the equivalence between SeparationQuotient and the quotient module by the null-space submodule; but that we shouldn't take that as the definition, and should just directly lift the inner operation through the quotient as I did for norm in #17452, and as we already did for dist.

@ADedecker
Copy link
Copy Markdown
Member

Ah, I see. My opinion on this is "just use SeparationQuotient", so I'm not even sure we really need to state the other construction. But we can discuss that somewhere else.

bors d+
(In case you want to wait on @yoh-tanimoto's review)

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 9, 2024

✌️ eric-wieser can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Oct 9, 2024
@eric-wieser
Copy link
Copy Markdown
Member Author

bors merge

so I'm not even sure we really need to state the other construction. But we can discuss that somewhere else.

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.

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Oct 9, 2024
mathlib-bors bot pushed a commit that referenced this pull request Oct 9, 2024
Co-authored-by: Eric Wieser <efw@google.com>
@yoh-tanimoto
Copy link
Copy Markdown
Collaborator

yoh-tanimoto commented Oct 9, 2024

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 lift of toDualMap?) are needed, but implementing lifts as (continuous) linear maps is necessary, I think. For example, if you have an automorphism of a C^*-algebra preserving the state, in the GNS representation it is represented by a unitary operator. That should be the norm completion of the lift.

Should I use #16707 for this purpose, or open a new one?

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Oct 9, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: InnerProductSpace (SeparationQuotient E) [Merged by Bors] - feat: InnerProductSpace (SeparationQuotient E) Oct 9, 2024
@mathlib-bors mathlib-bors bot closed this Oct 9, 2024
@mathlib-bors mathlib-bors bot deleted the eric-wieser/InnerProductSpace-separation branch October 9, 2024 14:28
yoh-tanimoto added a commit that referenced this pull request Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). ready-to-merge This PR has been sent to bors. t-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants