Skip to content

metric structure #1589

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

affeldt-aist
Copy link
Member

Motivation for this change

@zstone1

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Merge policy

As a rule of thumb:

  • PRs with several commits that make sense individually and that
    all compile are preferentially merged into master.
  • PRs with disorganized commits are very likely to be squash-rebased.
Reminder to reviewers

@affeldt-aist affeldt-aist added this to the 1.11.0 milestone Apr 24, 2025
@affeldt-aist affeldt-aist added the enhancement ✨ This issue/PR is about adding new features enhancing the library label Apr 24, 2025
@affeldt-aist affeldt-aist modified the milestones: 1.11.0, 1.12.0 May 1, 2025
affeldt-aist and others added 2 commits May 5, 2025 12:10
Co-authored-by: Daisuke Kimura <kmr@is.sci.toho-u.ac.jp>
@affeldt-aist
Copy link
Member Author

affeldt-aist commented May 9, 2025

@zstone1 could this be the first step towards addressing issue #1437 ?
This PR introduces a definition of metric space, move the lemma cvg_nbhsP from realfun.v to the directory topology_theory, incidentally together with at_left/at_right, which are now easier to move that normedtype.v has been split and reordered.

I propose to generalize a few more lemmas in the same manner, eventually moving a few things from normedtype_theory to topology_theory, so that we can arrive at a minimal PR introducing the metric structure without much disturbance for users. A complete generalization work can be carried incrementally. I am afraid that doing all the generalizations as a condition for merging will create a monster PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ This issue/PR is about adding new features enhancing the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant