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(algebra/pointwise): make instances global #3240

Closed
wants to merge 2 commits into from

Conversation

fpvandoorn
Copy link
Member

@fpvandoorn fpvandoorn commented Jun 30, 2020

add image2 and image3, the images of binary and ternary functions
cleanup in algebra/pointwise
make many variables implicit
make many names shorter
add some lemmas
add more simp lemmas
add type set_semiring as alias for set, with semiring instance using union as "addition"


@fpvandoorn fpvandoorn added the WIP Work in progress label Jun 30, 2020
src/algebra/pointwise.lean Outdated Show resolved Hide resolved
src/algebra/pointwise.lean Outdated Show resolved Hide resolved
src/algebra/pointwise.lean Show resolved Hide resolved
src/algebra/pointwise.lean Outdated Show resolved Hide resolved
src/algebra/pointwise.lean Outdated Show resolved Hide resolved
@fpvandoorn
Copy link
Member Author

fpvandoorn commented Jul 1, 2020

I'm currently simplifying {1} to (1 : set _), which occasionally breaks something that specifically uses singletons. Though simplifying in the other direction requires some extra simp lemmas like s * {1} = s (and {1} is not "simpler"). Thoughts?

I'm haven't yet done anything specifically for sub (which has no multiplicative analogue, right?), but I probably want to simplify images to preimages there, too.

I think I can finish this tomorrow (depending on the number of errors left).

@fpvandoorn fpvandoorn added awaiting-review The author would like community review of the PR and removed WIP Work in progress labels Jul 1, 2020
@urkud
Copy link
Member

urkud commented Jul 1, 2020

Please fix merge conflicts.

@fpvandoorn
Copy link
Member Author

I will, but feel free to already review the current version.

much cleanup
make many variables implicit
make many names shorter
add some lemmas
add type set_semiring as alias for set, with appropriate instance
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 🎉

Now that this compiles, I'm happy to kick it on the queue!

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 Jul 3, 2020
bors bot pushed a commit that referenced this pull request Jul 3, 2020
add image2 and image3, the images of binary and ternary functions
cleanup in algebra/pointwise
make many variables implicit
make many names shorter
add some lemmas
add more simp lemmas
add type set_semiring as alias for set, with semiring instance using union as "addition"
@bors
Copy link

bors bot commented Jul 3, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(algebra/pointwise): make instances global [Merged by Bors] - feat(algebra/pointwise): make instances global Jul 3, 2020
@bors bors bot closed this Jul 3, 2020
@bors bors bot deleted the algebra-pointwise branch July 3, 2020 22:00
cipher1024 pushed a commit to cipher1024/mathlib that referenced this pull request Mar 15, 2022
…3240)

add image2 and image3, the images of binary and ternary functions
cleanup in algebra/pointwise
make many variables implicit
make many names shorter
add some lemmas
add more simp lemmas
add type set_semiring as alias for set, with semiring instance using union as "addition"
bors bot pushed a commit that referenced this pull request May 16, 2022
Move `set_semiring` to a new file `data.set.semiring`.

Crediting Floris for #3240



Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
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

4 participants