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

format doc #1130

Merged
merged 5 commits into from
Jan 7, 2024
Merged

format doc #1130

merged 5 commits into from
Jan 7, 2024

Conversation

affeldt-aist
Copy link
Member

Motivation for this change

The goal of this PR is to format the documentation of topology.v to bring it
one step closer to the format expected for the HB release (ideally, we should
only need to update locally a few subsections when moving it to HB).
Ref: https://github.com/math-comp/math-comp/wiki/How-to-document
Note that this will still need to be updated soon to fit the format of coq2html
(this will be taken care of when rebasing PR #1108 ).

It happens that this PR also fixes a few minor issues with the documentation
(that was not properly updated) and it restores the ordering
(identifiers appear in the documentation in an order close to the
order in which they appear in the file, this makes it easier to edit it now
and should make it easier to split the file in the future).

@t6s This is the first pass of the first pass we talked about during the last MathComp-Analysis dev meeting.

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.

@affeldt-aist affeldt-aist added documentation 📝 This issue/PR is about documentation of the library / repository TODO: MC2 port This PR must be ported to mathcomp 2 now that the. Remove this label when the port is done. labels Jan 2, 2024
@affeldt-aist affeldt-aist added this to the 0.6.7 milestone Jan 2, 2024
@affeldt-aist affeldt-aist requested review from t6s and proux01 January 2, 2024 08:51
Copy link
Collaborator

@proux01 proux01 left a comment

Choose a reason for hiding this comment

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

A few nitpicking but this huge works looks great and should be merged.

Comment on lines +1491 to +1492
Definition monotonous d (T : porderType d) (pT : predType T) (A : pT) (f : T -> T) :=
{in A &, {mono f : x y / (x <= y)%O}} \/ {in A &, {mono f : x y /~ (x <= y)%O}}.
Copy link
Collaborator

Choose a reason for hiding this comment

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

According to WP, this should rather be monotonic and I would expect it to be non strict (with homo instead of mono). Maybe we should also have a strict version. I would also expect it to use nondecreasing_fun and the like from realfun.v.
All in all, maybe this change should come in a separate PR so as to not delay the remaining of the current PR.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I will issue about it to avoid mixing topics.

theories/topology.v Outdated Show resolved Hide resolved
theories/topology.v Outdated Show resolved Hide resolved
theories/topology.v Outdated Show resolved Hide resolved
theories/topology.v Show resolved Hide resolved
theories/topology.v Show resolved Hide resolved
theories/topology.v Show resolved Hide resolved
@proux01
Copy link
Collaborator

proux01 commented Jan 6, 2024

This CI failure is obviously unrelated (and a sign that we should really drop MC1 to avoid having to maintain two branches).

@affeldt-aist affeldt-aist merged commit 437d12c into math-comp:master Jan 7, 2024
26 of 30 checks passed
@t6s
Copy link
Member

t6s commented Jan 8, 2024

The changes look good (sorry for reviewing after the merger).
I remember the discussion at the last meeting that I could add some description about filteredTypes in terms of convergence spaces.
I will put it in another PR after experimenting some definitions of the convergence space.

affeldt-aist added a commit to affeldt-aist/analysis that referenced this pull request Jan 9, 2024
* format doc
---------
Co-authored-by: Pierre Roux <[email protected]>
proux01 pushed a commit that referenced this pull request Jan 9, 2024
* format doc
---------
Co-authored-by: Pierre Roux <[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 Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📝 This issue/PR is about documentation of the library / repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants