« first day (1870 days earlier)      last day (1369 days later) » 

4:53 AM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208263
 
 
5 hours later…
9:59 AM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208419
 
 
1 hour later…
11:02 AM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208448
 
11:42 AM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208458
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208471 | Edited by reviewer
 
 
3 hours later…
2:18 PM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208594 | Approved by OP
 
2:28 PM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2208596
 
Don't know enough about the subject to review that^ one
 
Sam
Looks fine
 
Yeah it doesn't look bad, but the first rejection reason makes me unsure.
 
Sam
It was mostly reformatting the answer.
 
> No; steps 1 and 2 are under "delete the state" but the rest is not; this edit obscures the logic
 
Sam
2:32 PM
Ohhh
I see
He should've improved the edit in that case.
But no, people like complaining.
 
I'm going to change EditMonitor's internals a bit: it would be much more useful if we can catch edits which are at 2 approvals + 2 rejections in the queue, so we can take action before they are actually approved, if necessary. (And the 'Approved with ...' reason stays for edits it didn't catch in time)
 
Sam
\o/
 
Also I still need to write up the document about how modes work... :P
 
Sam
Yeah, I've been meaning to ask what they mean.
 
Let's do that first.
goes to GitHub wiki
 
Sam
2:42 PM
The Fox scurries off on another adventure...
 
2:56 PM
Was kinda hard to produce clear sentences for that, let me know if it's unclear at some point :P
 
Sam
Oh, cool.
It reads fine to me.
 
yay :p
 
Sam
And I've just finished off a core part of Pham's classifier \o/
 
Okay, moving to changing EditMonitor's catching thing.
@Sam :D
 
Sam
So what actually happened (happening?) with Smokey's Bayesian thing?
@ProgramFOX :D
 
3:03 PM
Nothing. A few weeks ago we had a discussion and the consensus was that it probably wouldn't be a bad idea to remove it...
but nothing happened with it ever since
 
Sam
Aww, that's a shame.
 
I don't think it would be that useful for Smokey since most things are caught by the regexes anyway.
 
Sam
I always used to wonder you what guys were doing with that. lol
 
It was an experiment.
 
Sam
@ProgramFOX Yeah, fair point.
 
3:05 PM
Let's just poke the other people again to ask whether I could safely remove it :P
And the gibberish classifier, same thing with that, it wasn't very accurate because of the great amount of short posts on SE.
A length check wouldn't be an option there since all gibberish posts are short.
 
Sam
:|
So you've abandoned that project?
 
@Sam No, not at all. It's just not useful for Smokey.
 
Sam
Gotcha.
 
The gibberish classifier was built independently of Smokey, and I thought it would be a nice experiment to run it on Smokey a bit.
But it's been mostly overtaken by the "few unique characters in post" reason anyway.
 
Sam
Right
 
3:22 PM
!>stop
 
EditMonitor started.
 
I'll run it locally to figure out if there are any bugs now, and if not, I'll push and run on my cloud.
 
[ EditMonitor ] In the queue with 2 rejection votes (mode: 63): 2208659
 
Sam
ok
 
@FOX9000 That does need a reformat, but not as it's done in that edit...
I started with trying it, but... I lost track of what belonged where pretty quickly.
 
Sam
3:27 PM
lol
 
Anyway, the new feature is working \o/
 
Sam
\o/
 
Now I'll wait to see that it doesn't break the old "Approved with..." reason.
 
3:44 PM
[ EditMonitor ] In the queue with 2 rejection votes (mode: 63): 2208692
 
Sam
Meh
 
3:58 PM
I'll also have to add "... with 2 approval votes ..." to the message to make it clearer.
... I should also avoid rejected edits to be checked because that isn't necessary.
 
Sam
Yeah, probably
 
4:24 PM
EditMonitor started.
 
Okay, now it does not check rejected edits anymore. Better for my network traffic :P
 
Sam
:p
 
It will still do more network requests anyway, as it now checks all ongoing edits.
 
Sam
Better than nothing though
 
Yeah, it does result in a better detection.
Now I just need an already-approved edit to verify that's working too :p
And I just found another thing to lighten network traffic :P
 
4:33 PM
EditMonitor started.
 
5:04 PM
[ EditMonitor ] Approved with 2 rejection votes (mode: 63): 2208770 | Approved by OP | Edited by reviewer
 
Yay!
!>stop
 
Sam
:D
 
EditMonitor started.
 
5:37 PM
[ EditMonitor ] In the queue with 2 approval votes and 2 rejection votes (mode: 47): 2208773
[ EditMonitor ] In the queue with 2 approval votes and 2 rejection votes (mode: 47): 2208779
 
@FOX9000 ... two rejection votes?
And where are those two approve votes...?
weird bug
 
Sam
Seems fine to me.
 
O_O
I just got a... totally different edit.
 
Sam
:o
Fianlly
Finished
DAH CORE
And CI build failed. -_-
 
[ EditMonitor ] In the queue with 2 approval votes and 2 rejection votes (mode: 47): 2208784
 
 
2 hours later…
8:21 PM
[ EditMonitor ] In the queue with 2 approval votes and 2 rejection votes (mode: 47): 2208946
 
 
2 hours later…
9:51 PM
[ EditMonitor ] In the queue with 2 approval votes and 2 rejection votes (mode: 47): 2209035
 
10:37 PM
[ EditMonitor ] Approved with 2 rejection votes (mode: 47): 2209050 | Edited by reviewer
 
11:08 PM
[ EditMonitor ] In the queue with 2 approval votes and 2 rejection votes (mode: 47): 2209053
 

« first day (1870 days earlier)      last day (1369 days later) »