Conversation
|
@JSMassmann FYI |
|
@prabau How do you show that if |
|
Agree. We only have Kol(X) is CWH ⇒ X is CWH. The converse is surely false since X × S4 is always CWH (¬P107 → CWH) |
|
Ah, thank you @prabau for doing this. I'll take a look. (My Internet connection right now isn't great, so I might not be able to talk so actively...) |
|
LGTM. I don't have an opinion on whether we should use P19 or P21. |
|
@StevenClontz We are having a disagreement about T921 (see above) and need your opinion. My point is that, as a general rule, if a theorem can be stated in two equivalent ways, it is preferable to use the version that uses the better known properties. Specifically, suppose the combination of properties (Even if the proof would be marginally shorter with one set of hypotheses, what matters more is the statement itself, which will be read and used (in traits deductions for example) countless times compared with the proof itself, which can be viewed as a black box after it is understood.) |
|
Personally I don't have a strong opinion about the specific situation here, and see merit in both perspectives. In this situation where the proposal is correct, and the reviewer and proposer only disagree over style, and there's nothing in our style guide to refer to, I think it's best to approve and merge the original proposal. Of course, the reviewer is welcome to open a new PR to make their suggested change in style. This also reminds me of a thought I had that we should probably move our style guide into the project itself. Right now anyone can edit the wiki and "declare" style conventions, but we have enough community members where our style guide really should be subject to the same peer review as any other change. (I feel like I may have even made this comment before, but never did anything about it. That should probably change.) |
@StevenClontz In your opinion, what is the other perspective, as I am confused about what it would be? |
StevenClontz
left a comment
There was a problem hiding this comment.
LGTM other than consideration of one suggestion.
I think there are actually three non-equivalent and reasonable perspectives here:
I don't feel I know offhand which is best, and don't recall if we established a convention. If we did, I didn't find it in the wiki, which is what I usually turn to. (I reviewed the substance of the PR after my previous comment, and only saw the most recent T921 where I made my suggestion.) |
|
Oh, we have this: https://github.com/pi-base/data/wiki/Reviewing#requirements-for-a-new-theorem
I think this lines up with my earlier take: the contributor gets to make this kind of call. |
Co-authored-by: Steven Clontz <steven.clontz@gmail.com>
|
Yeah I was too harsh with my critique, either way would be fine as long as its mentioned the two are equivalent |
|
@Moniker1998 If you are satisfied with the latest version, can this be approved? |
|
@StevenClontz I think your change was taken care of, so I'll dismiss it |
I believe this change was already taken care of
As proposed in #1808.