-
Notifications
You must be signed in to change notification settings - Fork 212
feat: add Green's Open Problem 82 #1897
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
base: main
Are you sure you want to change the base?
feat: add Green's Open Problem 82 #1897
Conversation
Signed-off-by: krrish175-byte <[email protected]>
| $A \subset \mathbb{Z}$ with $|A| = n$. | ||
| -/ | ||
| @[category research open, AMS 11 42] | ||
| theorem green_82 (n : ℕ) (hn : 1 ≤ n) : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
n probably shouldnt be in the context as usual
| ⨅ (A : Finset ℤ) (_ : A.card = n), | ||
| ({θ : ℝ | θ ∈ Ico 0 1 ∧ ∑ a ∈ A, cos (2 * π * a * θ) = 0} : Set ℝ).ncard = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better write this as an inequality and not take sup?
|
The green problem list mentions known bounds, please include them. |
| @[category research open, AMS 11 42] | ||
| theorem green_82 (n : ℕ) (hn : 1 ≤ n) : | ||
| ⨅ (A : Finset ℤ) (_ : A.card = n), | ||
| ({θ : ℝ | θ ∈ Ico 0 1 ∧ ∑ a ∈ A, cos (2 * π * a * θ) = 0} : Set ℝ).ncard = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not what the problem is asking. Why do you have to limit Ico 0 1?
Signed-off-by: krrish175-byte <[email protected]>
|
Thanks for the review. Sorry, I missed those. I have updated the code as you asked for:
Regarding Ico 0 1: Since the trigonometric polynomial is 1-periodic, counting zeros in the fundamental domain [0, 1) seemed like the most direct way to represent the count on R / Z. Let me know if you would prefer using UnitAddCircle instead. Thanks for the review again! |
Thank you ChatGPT. I mistook what R / Z meant in this context, yeah. |
| ∃ c > 0, ∀ n ≥ 2, ∀ A : Finset ℤ, A.card = n → | ||
| c * log n / log (log n) ≤ | ||
| ({θ : ℝ | θ ∈ Ico 0 1 ∧ ∑ a ∈ A, cos (2 * π * a * θ) = 0} : Set ℝ).ncard := by |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where did you get this estimate from? Its not mentioned on the website....
If you include known results, also make sure to reference them properly.
In theory, it is fine if you use AI assistance (sorry if im mistaken, but im pretty sure), but it is crucial you check the output.
On a nother note, there are also bounds in the other direction as mentioned in the problem list, which should be included as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really really apologize for the confusion.
I went back to the original problem statement in Green's paper and I see now that I misread/misunderstood the bounds mentioned. The paper doesn't actually specify concrete bounds for Problem 82, so I shouldn't have included that theorem. I accept its my mistake.
Regarding the Ico 0 1 restriction, you make a good point. I was thinking about the periodicity, but the problem is really asking about the minimum number of zeros as we vary over all possible finite sets A, which is indeed a different question than counting in a fundamental domain.
I will update the PR to remove that lower bound theorem entirely, reformulate without the [0,1) restriction, keep it as an inequality as you suggested
Thanks again for the review, this is my first contribution to this project and I'm still learning the conventions so I request your patience and assistance. Thanks again...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Ico thing should be fine though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah got it. Thanks for clarifying. Then I will keep the Ico 0 1 restriction then. So before I push any changes, can you please confirm if this approach is right or you need any more changes :
- Ico 0 1 kept (as you just confirmed)
- The incorrect lower bound removed
- n ≥ 2 instead of n ≥ 1
- The upper bound n^{5/6} included
let me know if there's anything else that needs adjustment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the problem list, n^ 5/6 isnt the best known bound anymore.
I dont know why you need 1 leq n or 2 leq n. Everything is in Landau O notation, so this shouldnt matter.
There are known results both on the upper and lower bound. A complete foramilasation ought to nention both of them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the problem list, n^ 5/6 isnt the best known bound anymore.
I dont know why you need 1 leq n or 2 leq n. Everything is in Landau O notation, so this shouldnt matter.
There are known results both on the upper and lower bound. A complete foramilasation ought to nention both of them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated the code with the current best known bounds:
Lower bound: (log log n)^c for sufficiently large n
- wReference: Bedert (2025), arXiv:2407.16075
Upper bound: C(n log n)^(2/3)
- Reference: Juškevičius & Sahasrabudhe (2020), arXiv:2005.01695
I've formalized both bounds as separate theorems as you suggested. The lower bound uses ∃ n0 to handle the asymptotic nature properly (since log log n isn't well-defined for small n).
Let me know if it is correct now.
Closes #1737
This PR formalizes Green's Open Problem 82, which asks: for a finite set A ⊂ ℤ of size n, what is the minimum number of zeros that ∑_{a∈A} cos(2πaθ) must have on ℝ/ℤ?