Skip to content

[Merged by Bors] - fix: to_additive now translates projections#952

Closed
fpvandoorn wants to merge 2 commits intomasterfrom
toAdditiveProjection
Closed

[Merged by Bors] - fix: to_additive now translates projections#952
fpvandoorn wants to merge 2 commits intomasterfrom
toAdditiveProjection

Conversation

@fpvandoorn
Copy link
Copy Markdown
Member

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Dec 12, 2022

bors merge

bors bot pushed a commit that referenced this pull request Dec 12, 2022
* Projections can occur in terms that use the structure notation `{ ... with ... }`.
* Nobody submitted a MWE, so I didn't add a test (I don't know exactly how to ensure that a projection is included in a term). I did check that this fixes the issues in #944 and #912
* Closes #939, it fixes both comments mentioned there.
@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Dec 12, 2022
@bors
Copy link
Copy Markdown

bors bot commented Dec 12, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix: to_additive now translates projections [Merged by Bors] - fix: to_additive now translates projections Dec 12, 2022
@bors bors bot closed this Dec 12, 2022
@bors bors bot deleted the toAdditiveProjection branch December 12, 2022 00:17
bors bot pushed a commit that referenced this pull request Dec 14, 2022
mathlib3 SHA: 3342d1b2178381196f818146ff79bc0e7ccd9e2d

- [x] depends on: #952 

Co-authored-by: Kevin Buzzard <k.buzzard@imperial.ac.uk>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Frédéric Dupuis <31101893+dupuisf@users.noreply.github.com>
Co-authored-by: Scott Morrison <scott@tqft.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

to_additive interacts poorly with default fields of structures

2 participants