Please see https://stackoverflow.com/review/suggested-edits/5881383
The editor removed the snippet, yet, 3/4 of the reviewers approved it. On a side note, they didn't even fix my typo.
Why was this approved?
Please see https://stackoverflow.com/review/suggested-edits/5881383
The editor removed the snippet, yet, 3/4 of the reviewers approved it. On a side note, they didn't even fix my typo.
Why was this approved?
Because the edit review system isn't working well. The main flaw is that the system relies on the human edit reviewers to be responsible, careful and actually understand the edit policies.
Look at the statistics from the 4 people who reviewed the edit:
Accept_reviewer1 has approved 224 edit suggestions and rejected 14 edit suggestions
Accept_reviewer2 has approved 159 edit suggestions and rejected 14 edit suggestions
Reject_reviewer1 has approved 370 edit suggestions and rejected 201 edit suggestions
Accept_reviewer3 has approved 190 edit suggestions and rejected 34 edit suggestions
Experience from the flood of crap edits that the site is bombarded with suggest that a careful suggested edits reviewer should roughly have an accept/reject ratio of 50/50 (plus minus some depending on how picky they are).
You can tell that the "reject reviewer" is doing his job well by the realistic number of rejects. Similarly, you can tell that the three others are robo-reviewers, who accept pretty much anything.
As for suggestions about how to solve this, that has been discussed in detail elsewhere.
(approve%)*(reject%) * log_2(total reviews)
, with a dynamic amount of total "vote power" required to accept an edit (that grows as the size of the edit approval queue approach 0). (accept%)*(reject%)
is the amount of information your vote gives (someone who always accepts doesn't produce information when they vote), and the log_2 portion gives more weight to people with more expertise.
Commented
Sep 30, 2014 at 17:46
The edit review system makes rejecting edits harder than approving them.
The result is that those who reject edits are discouraged, and review slower. Those who approve edits are encouraged, and review faster.
This results in people doing edits getting told "good job, do another like that" with a review success, even if the edit in question was poor. This creates a second order effect of more poor edits.
Approve is a one click step. Lots of encouragement. You do have to spot blatantly horrible random spam "tests" and say "that looks like a test". I have yet to see a test that looks merely like a bad post.
The next two are a pair:
Improve Edit requires editing the post, and counts as an approve.
Reject and Edit requires editing the post from scratch.
So if the poster fixes 7/20 errors in the post, you either have to approve, or redo the 7 fixes. So for these two paths, approve is clearly the one the website encourages you to use if the edit contained anything useful. Incomplete/lazy edits are approved more often than they should be, encouraging more of same.
Next we have Reject. If you click on it, you now have to look at a restricted set of options, all of which are narrowly constructed in what seems to be an attempt to discourage you from picking any of them. You either need to do research to justify the rejection, consider the edit attempt to be a misclick, random spam, or prove to yourself that the edit was sufficient to make the original post incomprehensible. Or you can go with "other" and put forward a custom argument why the edit should be rejected, which by association with the other reasons is implied that you should only be done in extreme circumstances.
To vote Copied Content honestly, you'd have to do research and find that content if you are being honest. We are talking orders of magnitude more work than approve. And it doesn't apply here anyhow.
Invalid Edit seems aimed at someone clicking the wrong button, or thinking that edit is the same as comment. Your post doesn't seem to match this -- it looks like an edit, if a crappy one. These happen, but this is again a narrow option from its description.
Radical Change could apply. By the letter of the description, it only applies if the edit would lose the original meaning of the post. While the edit in question does do serious damage, the original meaning of the post remains: "use instead" -- the example is damaged, but the meaning is not lost, as required by the reading of the "refuse edit" choice. Even to detect this damage, you have to either mentally parse the markdown, or click on the render markdown, and determine that the editor had made a mistake. So while I would pick this, by the plain words it doesn't even apply to the edit in question! And it is, again, much harder to justify to oneself than "meh, approve".
Vandalism might apply, but it is not defacing, it is not spam, and it isn't "inappropriate" in the context of defacing/spam.
Which leaves Other, where the person rejecting the edit has to type out "the person who did the edit removed screwed up and caused a code sample to disappear". Again, orders of magnitude more work than Approve.
The powers at be at stack overflow have regularly made the Reject reasons narrower and added hoops to jump through in order to reject an edit, while Approve remains easy, and the tests are good enough to stop a "drinking bird". The result is that the suggested edit queue is full of people who keep on getting rewarded for nearly mindlessly Approving anything that seems vaguely reasonable (ie, isn't random gibberish). Those that Reject reasonably often end up taking so long that even if they matched the number of mass Approvers, they would lose out on most votes.
So the Rejectors do orders of magnitude more work, and see their Rejects getting overridden. Over time they find the options for rejecting made more narrow, and extra hoops added, discouraging them even more.
In short, it was approved because stack overflow edit queue causes bad edits to be approved.
Skip
, I am sure they dont even see it.
skip
matters. It doesn't generate an approve nor a reject: it is "abstain", roughly the same as closing the browser window. The edit remains in the queue until certain number of people approve or reject it, skip
except insofar as it makes a biased ratio of otherwise approvers/rejectors skip rather than vote does not impact this. I have no idea what ratio of otherwise approvers/rejectors it causes to skip rather than vote, but it serves an important purpose: "I have no idea, so will not speak".
Commented
Jan 19, 2015 at 15:10