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] - refactor(*): rename some declarations ending with '' #9504
Conversation
semorrison
commented
Oct 2, 2021
|
||
-- from now on, we don't want to use the following instance anymore | ||
attribute [instance, priority 0] algebra.to_has_scalar | ||
|
||
lemma smul_def (r : R) (x : A) : r • x = algebra_map R A r * x := |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might be important that this appears after attribute [instance, priority 0] algebra.to_has_scalar
, as it affects which instance appears in the statement.
Perhaps the solution is just to make the double-primed version private?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure how it could be important, if it has no effect downstream. I think historically this did matter, but the definitions have changed since. In fact, I think we can just remove the priority 0
statement here, so I've pushed this change for CI to check. If it works I'll mention this in the PR description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. Leaving this instance in places causes painful deterministic timeouts downstream, that I don't want to debug. Instead I've made this private
as you suggest, and left some comments. This is pretty gross. :-(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🎉
bors merge
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Build failed: |
Failure seems spurious. |
bors somehow crashed. |
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Pull request successfully merged into master. Build succeeded: |
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>