Zulip Chat Archive
Stream: general
Topic: Trig function values
Heather Macbeth (Jul 20 2020 at 13:02):
An undergraduate student of mine, @Benjamin Davidson , has written up some missing trig function values -- and similar. He has bigger plans, but to me, this already seems like a reasonable first PR. What do others think?
Shing Tak Lam (Jul 20 2020 at 13:13):
I don't know what they already have, or if this is useful, but here are a few lemmas to do with cos
and sin
that I've proven a few weeks ago. Might be worth bundling into one PR? Or should I PR these separately?
https://gist.github.com/shingtaklam1324/4cd8a22113e0c3db0219b7751336fdd3
Heather Macbeth (Jul 20 2020 at 13:15):
I think PRs can be arbitrarily small, so no need to combine them. But to me, this definitely also looks worth PR'ing.
Johan Commelin (Jul 20 2020 at 14:11):
Yup, especially a first PR shouldn't try to be big.
Johan Commelin (Jul 20 2020 at 14:11):
PR's are cheap.
Kevin Buzzard (Jul 20 2020 at 16:47):
If it's of any interest, I remember when I was at school knowing sin(15)=(sqrt(6)-sqrt(2))/4 and sin(18)=(sqrt(5)-1)/4.
Kevin Buzzard (Jul 20 2020 at 16:48):
And I think that when I was at uni I convinced myself that things wouldn't go much further, in the sense that you start needing square roots of square roots after that.
Floris van Doorn (Jul 20 2020 at 16:53):
cos(pi / 4)
and similar values already exists in mathlib:
https://github.com/leanprover-community/mathlib/blob/master/src/data/real/pi.lean#L101
I think this file was forgotten when properties about trigonometric functions were refactored into this file:
https://github.com/leanprover-community/mathlib/blob/master/src/analysis/special_functions/trigonometric.lean
Heather Macbeth (Jul 20 2020 at 17:01):
@Floris van Doorn , thanks for linking. In my opinion, the first half of that file (everything with a sin
or cos
in the name) should be refactored into analysis.special_functions.trigonometric
Heather Macbeth (Jul 20 2020 at 17:02):
And the rest (the facts purely about ) left in data.real.pi
. What do you think?
Floris van Doorn (Jul 20 2020 at 17:03):
I agree. I think everything until line 123 should go into special_functions.trigonometric
: it's more about trig than about pi
.
Heather Macbeth (Jul 20 2020 at 17:05):
That sounds great. Maybe @Benjamin Davidson can do this reorganization. He also proved a few facts that are not in your file (for example, and ), which he could add.
Joseph Myers (Jul 20 2020 at 18:39):
Wikipedia has a very long list of such values, but maybe they wouldn't all be appropriate for mathlib (that page has a warning they might not all be appropriate for Wikipedia either). DLMF has a shorter table for multiples of ; maybe multiples of , and cover most of the cases likely to be useful in practice (along with and as a well-known result even if not one anything else is likely to use).
Jalex Stark (Jul 20 2020 at 18:39):
Getting the whole list may be a nice exercise in automation :)
Kevin Buzzard (Jul 20 2020 at 18:44):
or more homework for @Benjamin Davidson :-)
Jalex Stark (Jul 20 2020 at 18:45):
right, I suppose "automation" could also just mean "proving helper lemmas" rather than "writing tactics"
Kevin Buzzard (Jul 20 2020 at 18:45):
So for me, square roots of square roots was too much (I could see you could go on forever this way), but I didn't know .
Jalex Stark (Jul 20 2020 at 18:47):
that's a pretty number
Kevin Buzzard (Jul 20 2020 at 18:49):
I remember one thing bothering me when I was at school -- there are nice "no square roots of square roots" formulas for both and , but even though came out nicely, I couldn't figure out how to get without taking a square root of a square root. I guess nowadays I can prove it's impossible :-)
Kevin Buzzard (Jul 20 2020 at 18:50):
Thanks for the link Joseph, that Wikipedia page is a work of art.
Benjamin Davidson (Jul 21 2020 at 00:02):
Hi all, I'm Ben and I am more than happy to PR anything deemed useful! May I please have GitHub access so that I can work on making my first PR?
Benjamin Davidson (Jul 21 2020 at 00:04):
@Floris van Doorn Thank you for sharing this file, I hadn't come across it and had only checked analysis.special_functions.trigonometric
for the likes of cos (pi/4)
.
Jalex Stark (Jul 21 2020 at 00:05):
Benjamin Davidson said:
Hi all, I'm Ben and I am more than happy to PR anything deemed useful! May I please have GitHub access so that I can work on making my first PR?
what's your github username?
Floris van Doorn (Jul 21 2020 at 00:41):
Benjamin Davidson said:
Floris van Doorn Thank you for sharing this file, I hadn't come across it and had only checked
analysis.special_functions.trigonometric
for the likes ofcos (pi/4)
.
Makes sense. They are in an unexpected place.
Benjamin Davidson (Jul 21 2020 at 03:10):
@Jalex Stark My GitHub username is benjamindavidson
Scott Morrison (Jul 21 2020 at 03:35):
@Benjamin Davidson: See https://github.com/leanprover-community/mathlib/invitations
Benjamin Davidson (Jul 21 2020 at 03:43):
@Scott Morrison Thank you
Benjamin Davidson (Jul 21 2020 at 17:11):
Last updated: Dec 20 2023 at 11:08 UTC