Skip to content

[Merged by Bors] - feat: 8 ∣ a ^ 2 - 1 for each odd a#28790

Closed
yuanyi-350 wants to merge 13 commits intoleanprover-community:masterfrom
yuanyi-350:eight_dvd_sq_sub_one_of_odd-
Closed

[Merged by Bors] - feat: 8 ∣ a ^ 2 - 1 for each odd a#28790
yuanyi-350 wants to merge 13 commits intoleanprover-community:masterfrom
yuanyi-350:eight_dvd_sq_sub_one_of_odd-

Conversation

@yuanyi-350
Copy link
Collaborator

@yuanyi-350 yuanyi-350 commented Aug 23, 2025

In this PR, we proved 8 ∣ a ^ 2 - 1 for each odd natural numbers and integers a

Open in Gitpod

@yuanyi-350
Copy link
Collaborator Author

easy

@github-actions github-actions bot added the easy < 20s of review time. See the lifecycle page for guidelines. label Aug 23, 2025
@yuanyi-350 yuanyi-350 requested a review from joneugster August 23, 2025 04:55
@yuanyi-350 yuanyi-350 changed the title eight_dvd_sq_sub_one_of_odd eight_dvd_sq_sub_one_of_odd Aug 23, 2025
@github-actions github-actions bot added the t-number-theory Number theory (also use t-algebra or t-analysis to specialize) label Aug 23, 2025
@github-actions
Copy link

github-actions bot commented Aug 23, 2025

PR summary 3d73639872

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ Int.eight_dvd_sq_sub_one_of_odd
+ Nat.eight_dvd_sq_sub_one_of_odd
++ two_dvd_mul_add_one

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@euprunin
Copy link
Collaborator

It looks like the PR is missing two_dvd_mul_add_one?

@yuanyi-350
Copy link
Collaborator Author

Thank you for your reminder. I think this issue is now resolved.

It looks like the PR is missing two_dvd_mul_add_one?

@b-mehta b-mehta added the awaiting-author A reviewer has asked the author a question or requested changes. label Aug 24, 2025
yuanyi-350 and others added 6 commits August 25, 2025 06:44
Co-authored-by: themathqueen <23701951+themathqueen@users.noreply.github.com>
Co-authored-by: themathqueen <23701951+themathqueen@users.noreply.github.com>
Co-authored-by: themathqueen <23701951+themathqueen@users.noreply.github.com>
Co-authored-by: themathqueen <23701951+themathqueen@users.noreply.github.com>
Co-authored-by: themathqueen <23701951+themathqueen@users.noreply.github.com>
Co-authored-by: themathqueen <23701951+themathqueen@users.noreply.github.com>
@yuanyi-350 yuanyi-350 removed the awaiting-author A reviewer has asked the author a question or requested changes. label Aug 24, 2025
@MichaelStollBayreuth
Copy link
Collaborator

  1. Please read the explanation of the "easy" label. I don't think it applies here.
  2. Read this and edit the PR title accordingly.
  3. Add a PR description.

@yuanyi-350
Copy link
Collaborator Author

-easy

@github-actions github-actions bot removed the easy < 20s of review time. See the lifecycle page for guidelines. label Sep 1, 2025
@yuanyi-350 yuanyi-350 changed the title eight_dvd_sq_sub_one_of_odd feat: 8 ∣ a ^ 2 - 1 for each odd aeight_dvd_sq_sub_one_of_odd Sep 1, 2025
@yuanyi-350 yuanyi-350 changed the title feat: 8 ∣ a ^ 2 - 1 for each odd aeight_dvd_sq_sub_one_of_odd feat: 8 ∣ a ^ 2 - 1 for each odd a Sep 1, 2025
@MichaelStollBayreuth
Copy link
Collaborator

MichaelStollBayreuth commented Sep 1, 2025

You could mention in the PR description that this is for natural numbers and integers a specifically.

@MichaelStollBayreuth MichaelStollBayreuth added the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 1, 2025
yuanyi-350 and others added 2 commits September 1, 2025 21:20
Co-authored-by: Michael Stoll <99838730+MichaelStollBayreuth@users.noreply.github.com>
Co-authored-by: Michael Stoll <99838730+MichaelStollBayreuth@users.noreply.github.com>
@yuanyi-350 yuanyi-350 removed the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 1, 2025
@MichaelStollBayreuth MichaelStollBayreuth added the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 1, 2025
@MichaelStollBayreuth MichaelStollBayreuth removed the awaiting-author A reviewer has asked the author a question or requested changes. label Sep 1, 2025
@MichaelStollBayreuth
Copy link
Collaborator

MichaelStollBayreuth commented Sep 1, 2025

I think this is OK now (regarding the naming of the two_dvd_... results, see this Zulip thread).

maintainer merge

@github-actions
Copy link

github-actions bot commented Sep 1, 2025

🚀 Pull request has been placed on the maintainer queue by MichaelStollBayreuth.

@ghost ghost added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Sep 1, 2025
@github-actions
Copy link

github-actions bot commented Sep 1, 2025

🚀 Pull request has been placed on the maintainer queue by MichaelStollBayreuth.

@kbuzzard
Copy link
Member

kbuzzard commented Sep 1, 2025

bors merge

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Sep 1, 2025
mathlib-bors bot pushed a commit that referenced this pull request Sep 1, 2025
In this PR, we proved `8 ∣ a ^ 2 - 1` for each odd natural numbers and integers `a`
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Sep 1, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: 8 ∣ a ^ 2 - 1 for each odd a [Merged by Bors] - feat: 8 ∣ a ^ 2 - 1 for each odd a Sep 1, 2025
@mathlib-bors mathlib-bors bot closed this Sep 1, 2025
@yuanyi-350 yuanyi-350 deleted the eight_dvd_sq_sub_one_of_odd- branch September 2, 2025 00:01
CBirkbeck pushed a commit to CBirkbeck/mathlib4 that referenced this pull request Sep 8, 2025
In this PR, we proved `8 ∣ a ^ 2 - 1` for each odd natural numbers and integers `a`
yuanyi-350 added a commit to yuanyi-350/yuanyi_mathlib4 that referenced this pull request Sep 10, 2025
In this PR, we proved `8 ∣ a ^ 2 - 1` for each odd natural numbers and integers `a`
joelriou pushed a commit to joelriou/mathlib4 that referenced this pull request Oct 2, 2025
In this PR, we proved `8 ∣ a ^ 2 - 1` for each odd natural numbers and integers `a`
mathlib-bors bot pushed a commit that referenced this pull request Dec 20, 2025
- [x] depends on: #28788
- [x] depends on: #28790
- [x] depends on: #28829
kim-em pushed a commit to kim-em/mathlib4 that referenced this pull request Jan 6, 2026
goliath-klein pushed a commit to PrParadoxy/mathlib4 that referenced this pull request Jan 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-number-theory Number theory (also use t-algebra or t-analysis to specialize)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants