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] - chore(geometry/manifold/complex): extract some theory of locally constant functions #16019

Closed
wants to merge 2 commits into from

Conversation

hrmacbeth
Copy link
Member

After #16015, I realised that we might as well deduce the rest of the lemmas in the file from the is_locally_constant lemma by appealing to general theory about locally constant functions. I have added three such lemmas to the general theory of locally constant functions:

  • is_locally_constant.apply_eq_of_preconnected_space
  • is_locally_constant.eq_const
  • is_locally_constant.exists_eq_const

All are direct copies of the lemmas for the bundled version (locally_constant). Then I reworked the holomorphic function theory to pass through these, which cuts a few lines from the file.


Open in Gitpod

@hrmacbeth hrmacbeth added awaiting-review The author would like community review of the PR awaiting-CI The author would like to see what CI has to say before doing more work. t-differential-geometry Manifolds, etc. labels Aug 11, 2022
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Aug 12, 2022
@ocfnash
Copy link
Collaborator

ocfnash commented Aug 19, 2022

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 Aug 19, 2022
bors bot pushed a commit that referenced this pull request Aug 19, 2022
…tant functions (#16019)

After #16015, I realised that we might as well deduce the rest of the lemmas in the file from the `is_locally_constant` lemma by appealing to general theory about locally constant functions.  I have added three such lemmas to the general theory of locally constant functions:
* `is_locally_constant.apply_eq_of_preconnected_space`
* `is_locally_constant.eq_const`
* `is_locally_constant.exists_eq_const`

All are direct copies of the lemmas for the bundled version (`locally_constant`).  Then I reworked the holomorphic function theory to pass through these, which cuts a few lines from the file.
@bors
Copy link

bors bot commented Aug 19, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(geometry/manifold/complex): extract some theory of locally constant functions [Merged by Bors] - chore(geometry/manifold/complex): extract some theory of locally constant functions Aug 19, 2022
@bors bors bot closed this Aug 19, 2022
@bors bors bot deleted the hrmacbeth_holo_mfld_cleanup branch August 19, 2022 13:53
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+`.) t-differential-geometry Manifolds, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants