Skip to content

chore:Clean up Blueprint#75

Open
JonBannon wants to merge 10 commits intomasterfrom
BlueprintCleanupMar17
Open

chore:Clean up Blueprint#75
JonBannon wants to merge 10 commits intomasterfrom
BlueprintCleanupMar17

Conversation

@JonBannon
Copy link
Copy Markdown
Collaborator

@JonBannon JonBannon commented Mar 17, 2026

Mild blueprint cleanup. (Not strictly dependent on #74, but if we wait for that to merge, I can also include it.)

@JonBannon JonBannon changed the title feat:ClenupBlueprint chore:CleanupBlueprint Mar 17, 2026
@JonBannon JonBannon changed the title chore:CleanupBlueprint chore: Clean up Blueprint Mar 17, 2026
@JonBannon JonBannon changed the title chore: Clean up Blueprint chore:Clean up Blueprint Mar 17, 2026
% The space $\Omega$ is called the \textbf{spectrum space} of $\mathcal{A}$.
% \end{corollary}

We state the following without proof, as it is available in Mathlib.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Was this meant to be commented out too?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, because we (so far) haven't needed nonunital Gelfand, and Jireh advised against putting in the cfc_n because it's already in mathlib. So I think it's ok to get rid of all of this. I commented it out in case we weren't sure...

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh yes, and also the last theorem commented out here points at the Gelfand duality. I was leaving all this here and commented out since we seem to be able to get away with CFC arguments...and we are probably going to keep getting away with them...

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What I meant was that this line isn't commented out, was it not meant to be commented out?
But, yeah, I don't know, you can keep things commented out I guess, just in case.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I see. Yeah, that seemed silly there. Must have missed it.

@themathqueen
Copy link
Copy Markdown
Collaborator

themathqueen commented Mar 23, 2026

Can you merge master and add the label for 1.5.3? Edit: I meant 1.3.2!

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