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] - feat: MvPolynomial.X_ne_zero #11071

Closed
wants to merge 1 commit into from

Conversation

BoltonBailey
Copy link
Collaborator

Proves that a variable is not the zero polynomial, analogous to Polynomial.X_ne_zero.


Open in Gitpod

@BoltonBailey BoltonBailey added awaiting-CI easy < 20s of review time. See the lifecycle page for guidelines. and removed easy < 20s of review time. See the lifecycle page for guidelines. labels Feb 29, 2024
@BoltonBailey BoltonBailey added the awaiting-review The author would like community review of the PR label Feb 29, 2024
@BoltonBailey
Copy link
Collaborator Author

Is there any reason this shouldn't be simp? The single-variable version is not, but it seems useful if it's allowed.

@riccardobrasca
Copy link
Member

Is there any reason this shouldn't be simp? The single-variable version is not, but it seems useful if it's allowed.

How can simp use a lemma?

@riccardobrasca
Copy link
Member

Is there any reason this shouldn't be simp? The single-variable version is not, but it seems useful if it's allowed.

How can simp use a lemma?

Well, it can of course close the goal X ≠ 0...

@Ruben-VandeVelde
Copy link
Collaborator

maintainer merge

Copy link

github-actions bot commented Mar 1, 2024

🚀 Pull request has been placed on the maintainer queue by Ruben-VandeVelde.

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

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot 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 Mar 1, 2024
@riccardobrasca
Copy link
Member

Can you add the simp attribute also to one variable case? In another PR it's fine.

mathlib-bors bot pushed a commit that referenced this pull request Mar 1, 2024
Proves that a variable is not the zero polynomial, analogous to `Polynomial.X_ne_zero`.
@mathlib-bors
Copy link

mathlib-bors bot commented Mar 1, 2024

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Mar 1, 2024
Proves that a variable is not the zero polynomial, analogous to `Polynomial.X_ne_zero`.
@mathlib-bors
Copy link

mathlib-bors bot commented Mar 1, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: MvPolynomial.X_ne_zero [Merged by Bors] - feat: MvPolynomial.X_ne_zero Mar 1, 2024
@mathlib-bors mathlib-bors bot closed this Mar 1, 2024
@mathlib-bors mathlib-bors bot deleted the BoltonBailey/x-ne-zero branch March 1, 2024 17:22
mathlib-bors bot pushed a commit that referenced this pull request Mar 2, 2024
kbuzzard pushed a commit that referenced this pull request Mar 12, 2024
Proves that a variable is not the zero polynomial, analogous to `Polynomial.X_ne_zero`.
kbuzzard pushed a commit that referenced this pull request Mar 12, 2024
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
Proves that a variable is not the zero polynomial, analogous to `Polynomial.X_ne_zero`.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
utensil pushed a commit that referenced this pull request Mar 26, 2024
Proves that a variable is not the zero polynomial, analogous to `Polynomial.X_ne_zero`.
utensil pushed a commit that referenced this pull request Mar 26, 2024
Louddy pushed a commit that referenced this pull request Apr 15, 2024
Proves that a variable is not the zero polynomial, analogous to `Polynomial.X_ne_zero`.
Louddy pushed a commit that referenced this pull request Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer-merge 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

5 participants