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] - refactor(order/filter/bases): turn is_countably_generated into a class #9838

Closed
wants to merge 6 commits into from

Conversation

urkud
Copy link
Member

@urkud urkud commented Oct 21, 2021

API changes

Filters

  • turn filter.is_countably_generated into a class, turn some lemmas into instances;
  • remove definition/lemmas (all were in the filter.is_countably_generated namespace): generating_set, countable_generating_set, eq_generate, countable_filter_basis, filter_basis_filter, has_countable_basis, exists_countable_infi_principal, exists_seq;
  • rename filter.is_countably_generated.exists_antitone_subbasis to filter.has_basis.exists_antitone_subbasis;
  • rename filter.is_countably_generated.exists_antitone_basis to filter.exists_antitone_basis;
  • rename filter.is_countably_generated.exists_antitone_seq' to filter.exists_antitone_seq;
  • rename filter.is_countably_generated.exists_seq to filter.exists_antitone_eq_infi_principal;
  • rename filter.is_countably_generated.tendsto_iff_seq_tendsto to filter.tendsto_iff_seq_tendsto;
  • rename filter.is_countably_generated.tendsto_of_seq_tendsto to filter.tendsto_of_seq_tendsto;
  • slightly generalize is_countably_generated_at_top and is_countably_generated_at_bot;
  • add filter.generate_eq_binfi;

Topology

  • merge is_closed.is_Gδ with is_closed.is_Gδ';
  • merge is_Gδ_set_of_continuous_at_of_countably_generated_uniformity with is_Gδ_set_of_continuous_at;
  • merge is_lub.exists_seq_strict_mono_tendsto_of_not_mem' with is_lub.exists_seq_strict_mono_tendsto_of_not_mem;
  • merge is_lub.exists_seq_monotone_tendsto' with is_lub.exists_seq_monotone_tendsto;
  • merge is_glb.exists_seq_strict_anti_tendsto_of_not_mem' with is_glb.exists_seq_strict_anti_tendsto_of_not_mem;
  • merge is_glb.exists_seq_antitone_tendsto' with is_glb.exists_seq_antitone_tendsto;
  • drop emetric.first_countable_topology, turn uniform_space.first_countable_topology into an instance;
  • drop emetric.second_countable_of_separable, use uniform_space.second_countable_of_separable instead;
  • drop metric.compact_iff_seq_compact, use uniform_space.compact_iff_seq_compact instead;
  • drop metric.compact_space_iff_seq_compact_space, use uniform_space.compact_space_iff_seq_compact_space instead;

Measure theory and integrals

Various lemmas assume [is_countably_generated l] instead of (h : is_countably_generated l).


@PatrickMassot Do you want me to restore some of the deleted definitions/lemmas? Should I move lemmas that now automatically work for metric spaces (uniform_space.second_countable_of_separable etc) to the root namespace?

Open in Gitpod

@urkud urkud added needs-documentation This PR is missing required documentation awaiting-review The author would like community review of the PR labels Oct 21, 2021
@PatrickMassot
Copy link
Member

What's the motivation for removing lemmas?

@urkud
Copy link
Member Author

urkud commented Oct 21, 2021

Do you mean lemmas in the is_countably_generated namespace? None of them were used in mathlib and I think that filter.exists_antitone_basis and filter.has_basis.exists_antitone_subbasis should be the main APIs for countably generated filters. I can restore them back in the filter.is_countably_generated NS if you want; can't move them to the filter NS because of name clashes.
As for lemmas in the (e)metric namespace, earlier they provided the is_countably_generated argument to their uniform_space counterparts but now typeclass instances can do it.

@PatrickMassot
Copy link
Member

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 Oct 23, 2021
bors bot pushed a commit that referenced this pull request Oct 23, 2021
…ass (#9838)

## API changes

### Filters

* turn `filter.is_countably_generated` into a class, turn some lemmas into instances;
* remove definition/lemmas (all were in the `filter.is_countably_generated` namespace): `generating_set`, `countable_generating_set`, `eq_generate`, `countable_filter_basis`, `filter_basis_filter`, `has_countable_basis`, `exists_countable_infi_principal`, `exists_seq`;
* rename `filter.is_countably_generated.exists_antitone_subbasis` to `filter.has_basis.exists_antitone_subbasis`;
* rename `filter.is_countably_generated.exists_antitone_basis` to `filter.exists_antitone_basis`;
* rename `filter.is_countably_generated.exists_antitone_seq'` to `filter.exists_antitone_seq`;
* rename `filter.is_countably_generated.exists_seq` to `filter.exists_antitone_eq_infi_principal`;
* rename `filter.is_countably_generated.tendsto_iff_seq_tendsto` to `filter.tendsto_iff_seq_tendsto`;
* rename `filter.is_countably_generated.tendsto_of_seq_tendsto` to `filter.tendsto_of_seq_tendsto`;
* slightly generalize `is_countably_generated_at_top` and `is_countably_generated_at_bot`;
* add `filter.generate_eq_binfi`;

### Topology

* merge `is_closed.is_Gδ` with `is_closed.is_Gδ'`;
* merge `is_Gδ_set_of_continuous_at_of_countably_generated_uniformity` with `is_Gδ_set_of_continuous_at`;
* merge `is_lub.exists_seq_strict_mono_tendsto_of_not_mem'` with `is_lub.exists_seq_strict_mono_tendsto_of_not_mem`;
* merge `is_lub.exists_seq_monotone_tendsto'` with `is_lub.exists_seq_monotone_tendsto`;
* merge `is_glb.exists_seq_strict_anti_tendsto_of_not_mem'` with `is_glb.exists_seq_strict_anti_tendsto_of_not_mem`;
* merge `is_glb.exists_seq_antitone_tendsto'` with `is_glb.exists_seq_antitone_tendsto`;
* drop `emetric.first_countable_topology`, turn `uniform_space.first_countable_topology` into an instance;
* drop `emetric.second_countable_of_separable`, use `uniform_space.second_countable_of_separable` instead;
* drop `metric.compact_iff_seq_compact`, use `uniform_space.compact_iff_seq_compact` instead;
* drop `metric.compact_space_iff_seq_compact_space`, use `uniform_space.compact_space_iff_seq_compact_space` instead;

## Measure theory and integrals

Various lemmas assume `[is_countably_generated l]` instead of `(h : is_countably_generated l)`.
@sgouezel
Copy link
Collaborator

bors r+

bors bot pushed a commit that referenced this pull request Oct 23, 2021
…ass (#9838)

## API changes

### Filters

* turn `filter.is_countably_generated` into a class, turn some lemmas into instances;
* remove definition/lemmas (all were in the `filter.is_countably_generated` namespace): `generating_set`, `countable_generating_set`, `eq_generate`, `countable_filter_basis`, `filter_basis_filter`, `has_countable_basis`, `exists_countable_infi_principal`, `exists_seq`;
* rename `filter.is_countably_generated.exists_antitone_subbasis` to `filter.has_basis.exists_antitone_subbasis`;
* rename `filter.is_countably_generated.exists_antitone_basis` to `filter.exists_antitone_basis`;
* rename `filter.is_countably_generated.exists_antitone_seq'` to `filter.exists_antitone_seq`;
* rename `filter.is_countably_generated.exists_seq` to `filter.exists_antitone_eq_infi_principal`;
* rename `filter.is_countably_generated.tendsto_iff_seq_tendsto` to `filter.tendsto_iff_seq_tendsto`;
* rename `filter.is_countably_generated.tendsto_of_seq_tendsto` to `filter.tendsto_of_seq_tendsto`;
* slightly generalize `is_countably_generated_at_top` and `is_countably_generated_at_bot`;
* add `filter.generate_eq_binfi`;

### Topology

* merge `is_closed.is_Gδ` with `is_closed.is_Gδ'`;
* merge `is_Gδ_set_of_continuous_at_of_countably_generated_uniformity` with `is_Gδ_set_of_continuous_at`;
* merge `is_lub.exists_seq_strict_mono_tendsto_of_not_mem'` with `is_lub.exists_seq_strict_mono_tendsto_of_not_mem`;
* merge `is_lub.exists_seq_monotone_tendsto'` with `is_lub.exists_seq_monotone_tendsto`;
* merge `is_glb.exists_seq_strict_anti_tendsto_of_not_mem'` with `is_glb.exists_seq_strict_anti_tendsto_of_not_mem`;
* merge `is_glb.exists_seq_antitone_tendsto'` with `is_glb.exists_seq_antitone_tendsto`;
* drop `emetric.first_countable_topology`, turn `uniform_space.first_countable_topology` into an instance;
* drop `emetric.second_countable_of_separable`, use `uniform_space.second_countable_of_separable` instead;
* drop `metric.compact_iff_seq_compact`, use `uniform_space.compact_iff_seq_compact` instead;
* drop `metric.compact_space_iff_seq_compact_space`, use `uniform_space.compact_space_iff_seq_compact_space` instead;

## Measure theory and integrals

Various lemmas assume `[is_countably_generated l]` instead of `(h : is_countably_generated l)`.
@bors
Copy link

bors bot commented Oct 23, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(order/filter/bases): turn is_countably_generated into a class [Merged by Bors] - refactor(order/filter/bases): turn is_countably_generated into a class Oct 23, 2021
@bors bors bot closed this Oct 23, 2021
@bors bors bot deleted the countably-gen-class branch October 23, 2021 19:26
@eric-wieser eric-wieser added the hacktoberfest-accepted Without this label hacktoberfest is scared off by bors label Oct 26, 2021
ericrbg pushed a commit that referenced this pull request Nov 9, 2021
…ass (#9838)

## API changes

### Filters

* turn `filter.is_countably_generated` into a class, turn some lemmas into instances;
* remove definition/lemmas (all were in the `filter.is_countably_generated` namespace): `generating_set`, `countable_generating_set`, `eq_generate`, `countable_filter_basis`, `filter_basis_filter`, `has_countable_basis`, `exists_countable_infi_principal`, `exists_seq`;
* rename `filter.is_countably_generated.exists_antitone_subbasis` to `filter.has_basis.exists_antitone_subbasis`;
* rename `filter.is_countably_generated.exists_antitone_basis` to `filter.exists_antitone_basis`;
* rename `filter.is_countably_generated.exists_antitone_seq'` to `filter.exists_antitone_seq`;
* rename `filter.is_countably_generated.exists_seq` to `filter.exists_antitone_eq_infi_principal`;
* rename `filter.is_countably_generated.tendsto_iff_seq_tendsto` to `filter.tendsto_iff_seq_tendsto`;
* rename `filter.is_countably_generated.tendsto_of_seq_tendsto` to `filter.tendsto_of_seq_tendsto`;
* slightly generalize `is_countably_generated_at_top` and `is_countably_generated_at_bot`;
* add `filter.generate_eq_binfi`;

### Topology

* merge `is_closed.is_Gδ` with `is_closed.is_Gδ'`;
* merge `is_Gδ_set_of_continuous_at_of_countably_generated_uniformity` with `is_Gδ_set_of_continuous_at`;
* merge `is_lub.exists_seq_strict_mono_tendsto_of_not_mem'` with `is_lub.exists_seq_strict_mono_tendsto_of_not_mem`;
* merge `is_lub.exists_seq_monotone_tendsto'` with `is_lub.exists_seq_monotone_tendsto`;
* merge `is_glb.exists_seq_strict_anti_tendsto_of_not_mem'` with `is_glb.exists_seq_strict_anti_tendsto_of_not_mem`;
* merge `is_glb.exists_seq_antitone_tendsto'` with `is_glb.exists_seq_antitone_tendsto`;
* drop `emetric.first_countable_topology`, turn `uniform_space.first_countable_topology` into an instance;
* drop `emetric.second_countable_of_separable`, use `uniform_space.second_countable_of_separable` instead;
* drop `metric.compact_iff_seq_compact`, use `uniform_space.compact_iff_seq_compact` instead;
* drop `metric.compact_space_iff_seq_compact_space`, use `uniform_space.compact_space_iff_seq_compact_space` instead;

## Measure theory and integrals

Various lemmas assume `[is_countably_generated l]` instead of `(h : is_countably_generated l)`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Without this label hacktoberfest is scared off by bors needs-documentation This PR is missing required documentation 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