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

helper lemmas for contra (PR #1119) #1136

Merged
merged 2 commits into from
Jan 8, 2024

Conversation

Tragicus
Copy link
Collaborator

@Tragicus Tragicus commented Jan 7, 2024

Motivation for this change

This is the (hopefully) easily mergeable subset of PR #1119.

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.

@Tragicus Tragicus added 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 7, 2024
@affeldt-aist affeldt-aist requested review from affeldt-aist and removed request for affeldt-aist January 8, 2024 08:05
@affeldt-aist
Copy link
Member

I tentatively removed pdegen because it was not used and because PropB looks more practical. Maybe PropB deserves a piece of documentation in the header but that can be done later (after PR #1108 is merged?) as another PR that revises the documentation of boolp.v which is incomplete.

@affeldt-aist affeldt-aist added this to the 0.6.7 milestone Jan 8, 2024
@Tragicus
Copy link
Collaborator Author

Tragicus commented Jan 8, 2024

Thank you for the review and improvements. Concerning the documentation, the goal of this PR was to be able to merge before the next release, so if you plan to merge PR #1108 before then, we can also wait and update the documentation of boolp here.

@affeldt-aist affeldt-aist merged commit 9dafe6d into math-comp:master Jan 8, 2024
26 of 30 checks passed
@Tragicus Tragicus deleted the boolp_contra branch January 8, 2024 13:56
Tragicus added a commit to Tragicus/analysis that referenced this pull request Jan 8, 2024
affeldt-aist added a commit to affeldt-aist/analysis that referenced this pull request Jan 9, 2024
* helper lemmas for contra (PR math-comp#1119)

* rm pdegen, use more PropB

---------

Co-authored-by: Reynald Affeldt <[email protected]>
proux01 pushed a commit that referenced this pull request Jan 9, 2024
* helper lemmas for contra (PR #1119)

* rm pdegen, use more PropB

---------

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 Jan 9, 2024
Tragicus added a commit to Tragicus/analysis that referenced this pull request Jan 17, 2024
Tragicus added a commit to Tragicus/analysis that referenced this pull request Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants