Skip to content
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(analysis/normed_space/SemiNormedGroup): has_cokernels #7628

Closed
wants to merge 29 commits into from

Conversation

semorrison
Copy link
Collaborator

@semorrison semorrison commented May 17, 2021

Cokernels in SemiNormedGroup₁ and SemiNormedGroup

We show that SemiNormedGroup₁ has cokernels
(for which of course the cokernel.π f maps are norm non-increasing),
as well as the easier result that SemiNormedGroup has cokernels.

So far, I don't see a way to state nicely what we really want:
SemiNormedGroup has cokernels, and cokernel.π f is norm non-increasing.
The problem is that the limits API doesn't promise you any particular model of the cokernel,
and in SemiNormedGroup one can always take a cokernel and rescale its norm
(and hence making cokernel.π f arbitrarily large in norm), obtaining another categorical cokernel.


Open in Gitpod

@github-actions github-actions bot added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label May 17, 2021
Co-authored-by: Riccardo Brasca <riccardo.brasca@gmail.com>
@github-actions github-actions bot added merge-conflict Please `git merge origin/master` then a bot will remove this label. and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels May 18, 2021
@github-actions
Copy link

🎉 Great news! Looks like all the dependencies have been resolved:

💡 To add or remove a dependency please update this issue/PR description.

Brought to you by Dependent Issues (:robot: ). Happy coding!

@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label May 18, 2021
@semorrison semorrison added the awaiting-review The author would like community review of the PR label May 19, 2021
@github-actions github-actions bot added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label May 19, 2021
@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label May 20, 2021
Comment on lines 495 to 498
lemma lift_bound_by {N : Type*} [semi_normed_group N] (S : add_subgroup M)
(f : normed_group_hom M N) (hf : ∀ s ∈ S, f s = 0)
{c : ℝ≥0} (fb : f.bound_by c) :
(lift S f hf).bound_by c :=
Copy link
Member

Choose a reason for hiding this comment

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

I realize that I was just being completely ignorant with this bound_by definition. Would you mind phrasing this in terms of the operator norm?

@semorrison semorrison added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jun 9, 2021
@semorrison semorrison added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jul 16, 2021
@jcommelin
Copy link
Member

Thanks 🎉

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Jul 27, 2021
bors bot pushed a commit that referenced this pull request Jul 27, 2021
# Cokernels in SemiNormedGroup₁ and SemiNormedGroup

We show that `SemiNormedGroup₁` has cokernels
(for which of course the `cokernel.π f` maps are norm non-increasing),
as well as the easier result that `SemiNormedGroup` has cokernels.

So far, I don't see a way to state nicely what we really want:
`SemiNormedGroup` has cokernels, and `cokernel.π f` is norm non-increasing.
The problem is that the limits API doesn't promise you any particular model of the cokernel,
and in `SemiNormedGroup` one can always take a cokernel and rescale its norm
(and hence making `cokernel.π f` arbitrarily large in norm), obtaining another categorical cokernel.



Co-authored-by: Patrick Massot <patrickmassot@free.fr>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link

bors bot commented Jul 27, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(analysis/normed_space/SemiNormedGroup): has_cokernels [Merged by Bors] - feat(analysis/normed_space/SemiNormedGroup): has_cokernels Jul 27, 2021
@bors bors bot closed this Jul 27, 2021
@bors bors bot deleted the SemiNormedGroup_cokernel branch July 27, 2021 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants