Skip to content

Conversation

@zstone1
Copy link
Contributor

@zstone1 zstone1 commented Apr 19, 2023

Motivation for this change

The last of the general theory required for #834. This adds definitions for basis, second_countable, and two lemmas that I need. It's nowhere near complete in terms of the theory of different kind of basis. But it gets the job done.

Things done/to do
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers
Compatibility with MathComp 2.0
  • I added the label TODO: HB port to make sure someone ports this PR to
    the hierarchy-builder branch or I already opened an issue or PR (please cross reference).
Automatic note to reviewers

Read this Checklist and put a milestone if possible.

@zstone1 zstone1 added enhancement ✨ This issue/PR is about adding new features enhancing the library TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. labels Apr 19, 2023
@affeldt-aist
Copy link
Member

I committed to fix a compilation error and I linted a bit but it deserves more attention.

@zstone1 zstone1 force-pushed the second_countable branch from 52f8b9d to 31ad61d Compare May 3, 2023 01:14
@zstone1 zstone1 merged commit e3cbb98 into math-comp:master May 3, 2023
affeldt-aist added a commit that referenced this pull request May 4, 2023
* adding second countable stuff

* fix, lint

* merging

---------

Co-authored-by: Reynald Affeldt <[email protected]>
@proux01 proux01 removed the TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. label May 30, 2023
IshiguroYoshihiro pushed a commit to IshiguroYoshihiro/analysis that referenced this pull request Sep 7, 2023
* adding second countable stuff

* fix, lint

* merging

---------

Co-authored-by: Reynald Affeldt <[email protected]>
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.

4 participants