Skip to content

Comments

feat: Hilbert space & unbounded operators on Space#957

Draft
gloges wants to merge 11 commits intolean-phys-community:masterfrom
gloges:hilbert-space
Draft

feat: Hilbert space & unbounded operators on Space#957
gloges wants to merge 11 commits intolean-phys-community:masterfrom
gloges:hilbert-space

Conversation

@gloges
Copy link
Contributor

@gloges gloges commented Feb 21, 2026

Initial progress for single-particle QM on Space d, addressing #851.

SpaceDHilbertSpace and SchwartzSubmodule carry over from 1d with only minor changes (cf. OneDimension/HilbertSpace).
UnboundedOperator is defined using LinearPMap with additional density and closability hypotheses. Nothing in UnboundedOperator depends on the particular Hilbert space and can be easily generalized in the future.

A few sorries remain to be filled in; two I think I can do after understanding how to work with InnerProductSpace/PiL2.

Copy link
Member

@jstoobysmith jstoobysmith left a comment

Choose a reason for hiding this comment

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

Many thanks for this PR

open LinearPMap

-- Nothing here uses any specific properties of this particular Hilbert space
abbrev hilbSp d := SpaceDHilbertSpace d
Copy link
Member

Choose a reason for hiding this comment

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

Why do we need this abbreviation? If it is down to not wanting to write SpaceDHilbertSpace I think we should probably aim for notation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just a temporary abbreviation - all instances of SpaceDHilbertSpace are now removed for unbounded operators.

so that `hilbSp × hilbSp` has the structure of an inner product space.
-/

abbrev hilbSp2 (d : ℕ) := hilbSp d × hilbSp d
Copy link
Member

Choose a reason for hiding this comment

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

Do you mean the tensor product here? If not maybe we could add a doc for why this is needed

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.

2 participants