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] - [Merged by Bors] - feat: port Analysis.PSeries #4182

Closed
wants to merge 7 commits into from

Conversation

mcdoll
Copy link
Member

@mcdoll mcdoll commented May 22, 2023

Removed the simp attribute on one lemma because simpNF was timing out trying to reduce the LHS. See Zulip for the problem. Once this error is corrected with simp, we should be able to add the attribute again.


Open in Gitpod

Mathbin -> Mathlib
fix certain import statements
move "by" to end of line
add import to Mathlib.lean
@mcdoll mcdoll added WIP Work in progress mathlib-port This is a port of a theory file from mathlib. help-wanted The author needs attention to resolve issues labels May 22, 2023
@j-loreaux j-loreaux added awaiting-review The author would like community review of the PR and removed help-wanted The author needs attention to resolve issues WIP Work in progress labels May 25, 2023
Comment on lines +268 to +271
section pow_macro
-- porting note: without these local macro rules Lean fails to elaborate `^` properly.
-- See https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/!4.234085
local macro_rules | `($x ^ $y) => `(HPow.hPow $x $y)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@digama0 Do you understand what the correct solution is here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🎉

bors merge

@semorrison semorrison added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels May 25, 2023
bors bot pushed a commit that referenced this pull request May 25, 2023
Removed the `simp` attribute on one lemma because `simpNF` was timing out trying to reduce the LHS. See [Zulip](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/looping.20in.20.60simp.60.20set) for the problem. Once this error is corrected with `simp`, we should be able to add the attribute again.



Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
@bors
Copy link

bors bot commented May 25, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title feat: port Analysis.PSeries [Merged by Bors] - feat: port Analysis.PSeries May 25, 2023
@bors bors bot closed this May 25, 2023
@bors bors bot deleted the port/Analysis.PSeries branch May 25, 2023 20:03
@bors
Copy link

bors bot commented May 25, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title [Merged by Bors] - feat: port Analysis.PSeries [Merged by Bors] - [Merged by Bors] - feat: port Analysis.PSeries May 25, 2023
qawbecrdtey pushed a commit to qawbecrdtey/greedoid-mathlib4 that referenced this pull request Jun 12, 2023
Removed the `simp` attribute on one lemma because `simpNF` was timing out trying to reduce the LHS. See [Zulip](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/looping.20in.20.60simp.60.20set) for the problem. Once this error is corrected with `simp`, we should be able to add the attribute again.



Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mathlib-port This is a port of a theory file from mathlib. ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants