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

[Merged by Bors] - chore(analysis/special_functions/trigonometric): review continuity of tan #5429

Closed
wants to merge 8 commits into from

Conversation

urkud
Copy link
Member

@urkud urkud commented Dec 18, 2020

  • prove that tan is discontinuous at x whenever cos x = 0;
  • turn continuous_at_tan and differentiable_at_tan into iff lemmas;
  • reformulate various lemmas in terms of cos x = 0 instead of ∃ k, x = ...;

@github-actions github-actions bot added blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Dec 18, 2020
@github-actions
Copy link

🎉 Great news! Looks like all the dependencies have been resolved:

💡 To add or remove a dependency please update this issue/PR description.

Brought to you by Dependent Issues (:robot: ). Happy coding!

@urkud urkud added the awaiting-review The author would like community review of the PR label Dec 19, 2020
@PatrickMassot PatrickMassot self-assigned this Dec 21, 2020
@PatrickMassot
Copy link
Member

I'm still skeptical that reformulating everything in terms of cos x ≠ 0 is a natural thing to do. It's especially true for lemmas like

tendsto_abs_tan_of_cos_eq_zero {x : ℂ} (hx : cos x = 0) :
  tendsto (λ x, abs (tan x)) (𝓝[{x}ᶜ] x) at_top 

that could be

tendsto_abs_tan_at_top (k : ℤ) :  tendsto (λ x, abs (tan x)) (𝓝[{(2 * k + 1) * π / 2}ᶜ] x) at_top 

that seems pretty clearly less convoluted. But I won't insist on this if you used the lemmas and found it more convenient that way.
bors d+

@bors
Copy link

bors bot commented Dec 22, 2020

✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@PatrickMassot PatrickMassot added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Dec 22, 2020
@urkud
Copy link
Member Author

urkud commented Dec 22, 2020

I've added tendsto_abs_tan_at_top. My main concerns about formulating lemmas like

lemma continuous_at_tan : continuous_at tan x ↔ ∀ k : ℤ, x ≠ (2 * k + 1) * π / 2 := sorry

are:

  1. They're longer.
  2. It's not clear why we use (2 * k + 1) * π / 2 and not, e.g., π * k + π / 2, or k * π + π / 2, or π / 2 + π * k.
  3. Almost every time lemmas like these were used in special_functions/trigonometric, the first thing to do was to rewrite on cos_eq_zero_iff or cos_ne_zero_iff.

@urkud urkud added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Dec 22, 2020
@PatrickMassot
Copy link
Member

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Dec 22, 2020
bors bot pushed a commit that referenced this pull request Dec 22, 2020
… `tan` (#5429)

* prove that `tan` is discontinuous at `x` whenever `cos x = 0`;
* turn `continuous_at_tan` and `differentiable_at_tan` into `iff` lemmas;
* reformulate various lemmas in terms of `cos x = 0` instead of `∃ k, x = ...`;



Co-authored-by: Patrick Massot <patrickmassot@free.fr>
@bors
Copy link

bors bot commented Dec 22, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(analysis/special_functions/trigonometric): review continuity of tan [Merged by Bors] - chore(analysis/special_functions/trigonometric): review continuity of tan Dec 22, 2020
@bors bors bot closed this Dec 22, 2020
@bors bors bot deleted the tan-cont2 branch December 22, 2020 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants