Skip to content

[Merged by Bors] - feat: port LinearAlgebra.Matrix.Circulant#3465

Closed
ymonbru wants to merge 8 commits intomasterfrom
port/LinearAlgebra.Matrix.Circulant
Closed

[Merged by Bors] - feat: port LinearAlgebra.Matrix.Circulant#3465
ymonbru wants to merge 8 commits intomasterfrom
port/LinearAlgebra.Matrix.Circulant

Conversation

@ymonbru
Copy link
Copy Markdown
Contributor

@ymonbru ymonbru commented Apr 16, 2023


Open in Gitpod

ymonbru added 3 commits April 16, 2023 13:33
Mathbin -> Mathlib
fix certain import statements
move "by" to end of line
add import to Mathlib.lean
@ymonbru ymonbru added the WIP Work in progress label Apr 16, 2023
@Parcly-Taxel Parcly-Taxel changed the title Port/linear algebra.matrix.circulant feat: port LinearAlgebra.Matrix.Circulant Apr 16, 2023
@ymonbru
Copy link
Copy Markdown
Contributor Author

ymonbru commented Apr 16, 2023

Most of the errors are from the tactic "decide", i don't know how to manage this

In " Fin.circulant_isSymm_iff", the type is not accepted, i tryed to remove the "fun v => " from the proof and then lean tels that the terminasion of the term is not proven

It's not a mistake but in "circulant_mul_comm" there was a case disjunction that i deleted while solving an error (i am not shure if it's a good thing or not)

@ymonbru ymonbru added the help-wanted The author needs attention to resolve issues label Apr 16, 2023
@int-y1 int-y1 added the mathlib-port This is a port of a theory file from mathlib. label Apr 17, 2023
@kim-em kim-em added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Apr 20, 2023
@kim-em kim-em removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Apr 23, 2023
@int-y1 int-y1 added awaiting-review and removed WIP Work in progress help-wanted The author needs attention to resolve issues labels Apr 23, 2023
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Apr 24, 2023

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Apr 24, 2023
bors bot pushed a commit that referenced this pull request Apr 24, 2023
Co-authored-by: Jason Yuen <jason_yuen2007@hotmail.com>
Co-authored-by: int-y1 <jason_yuen2007@hotmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Apr 24, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port LinearAlgebra.Matrix.Circulant [Merged by Bors] - feat: port LinearAlgebra.Matrix.Circulant Apr 24, 2023
@bors bors bot closed this Apr 24, 2023
@bors bors bot deleted the port/LinearAlgebra.Matrix.Circulant branch April 24, 2023 06:30
kim-em pushed a commit that referenced this pull request May 10, 2023
Co-authored-by: Jason Yuen <jason_yuen2007@hotmail.com>
Co-authored-by: int-y1 <jason_yuen2007@hotmail.com>
hrmacbeth pushed a commit that referenced this pull request May 10, 2023
Co-authored-by: Jason Yuen <jason_yuen2007@hotmail.com>
Co-authored-by: int-y1 <jason_yuen2007@hotmail.com>
hrmacbeth pushed a commit that referenced this pull request May 11, 2023
Co-authored-by: Jason Yuen <jason_yuen2007@hotmail.com>
Co-authored-by: int-y1 <jason_yuen2007@hotmail.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.

3 participants