Parametrize initial rating/RD instead of hardcoding it.

[foosball] / foosball.pm

diff --git a/foosball.pm b/foosball.pm
index f87f7637d2039dd407db7aa6eca2d12af1f53cdc..b5750b26bda09db32f750af5d8a3ea8458906eba 100644 (file)
--- a/foosball.pm
+++ b/foosball.pm
@@ -1,28 +1,52 @@
 use strict;
 use warnings;
 use DBI;
+use POSIX;
 
 package foosball;
 
+our $initial_rating = 1500.0;
+our $initial_rd = 350.0;
+
 sub db_connect {
        my $dbh = DBI->connect("dbi:Pg:dbname=foosball;host=127.0.0.1", "foosball", "cleanrun", {AutoCommit => 0});
        $dbh->{RaiseError} = 1;
        return $dbh;
 }
 
+sub round {
+       my $x = shift;
+       return -round(-$x) if ($x < 0.0);
+       return POSIX::floor($x + 0.5);
+}
+
 sub find_single_rating {
        my ($dbh, $username, $limit) = @_;
+       $limit = "" if (!defined($limit));
        my ($age, $rating, $rd) = $dbh->selectrow_array('SELECT EXTRACT(EPOCH FROM (CURRENT_TIMESTAMP-ratetime)), rating, rd FROM single_rating WHERE username=? '.$limit.' ORDER BY ratetime DESC LIMIT 1',
                undef, $username);
        $rd = apply_aging($rd, $age / 86400.0);
+
+       if (!defined($rating)) {
+               $rating = $initial_rating;
+               $rd = $initial_rd;
+       }
+
        return ($rating, $rd);
 }
 
 sub find_double_rating {
        my ($dbh, $username, $limit) = @_;
+       $limit = "" if (!defined($limit));
        my ($age, $rating, $rd) = $dbh->selectrow_array('SELECT EXTRACT(EPOCH FROM (CURRENT_TIMESTAMP-ratetime)), rating, rd FROM double_rating WHERE username=? '.$limit.'ORDER BY ratetime DESC LIMIT 1',
                undef, $username);
        $rd = apply_aging($rd, $age / 86400.0);
+       
+       if (!defined($rating)) {
+               $rating = $initial_rating;
+               $rd = $initial_rd;
+       }
+
        return ($rating, $rd);
 }
 
@@ -33,10 +57,10 @@ sub create_user_if_not_exists {
        return if ($count > 0);
        $dbh->do('INSERT INTO users (username) VALUES (?)',
                undef, $username);
-       $dbh->do('INSERT INTO single_rating (username,ratetime,rating,rd) VALUES (?,CURRENT_TIMESTAMP,1500.0,350.0)',
-               undef, $username);
-       $dbh->do('INSERT INTO double_rating (username,ratetime,rating,rd) VALUES (?,CURRENT_TIMESTAMP,1500.0,350.0)',
-               undef, $username);
+       $dbh->do('INSERT INTO single_rating (username,ratetime,rating,rd) VALUES (?,CURRENT_TIMESTAMP,?,?)',
+               undef, $username, $initial_rating, $initial_rd);
+       $dbh->do('INSERT INTO double_rating (username,ratetime,rating,rd) VALUES (?,CURRENT_TIMESTAMP,?,?)',
+               undef, $username, $initial_rating, $initial_rd);
        return $dbh;
 }
 
@@ -46,13 +70,24 @@ our $c = 8;
 sub apply_aging {
        my ($rd, $age) = @_;
        $rd = sqrt($rd*$rd + $c * $c * ($age / 86400.0));
-       $rd = 350.0 if ($rd > 350.0);
+       $rd = $initial_rd if ($rd > $initial_rd);
        return $rd;
 }
 
 sub calc_rating {
        my ($rating1, $rd1, $rating2, $rd2, $score1, $score2) = @_;
-       my $result = `/srv/foosball.sesse.net/foorank $rating1 $rd1 $rating2 $rd2 $score1 $score2`;
+       my $result = `/srv/foosball.sesse.net/foosrank $rating1 $rd1 $rating2 $rd2 $score1 $score2`;
+       chomp $result;
+       my ($newr1, $newrd1, $likelihood) = split / /, $result;
+
+       $newrd1 = 30.0 if ($newrd1 < 30.0);
+
+       return ($newr1, $newrd1, $likelihood);
+}
+
+sub calc_rating_double {
+       my ($rating1, $rd1, $rating2, $rd2, $rating3, $rd3, $rating4, $rd4, $score1, $score2) = @_;
+       my $result = `/srv/foosball.sesse.net/foosrank $rating1 $rd1 $rating2 $rd2 $rating3 $rd3 $rating4 $rd4 $score1 $score2`;
        chomp $result;
        my ($newr1, $newrd1) = split / /, $result;
 
@@ -61,4 +96,5 @@ sub calc_rating {
        return ($newr1, $newrd1);
 }
 
+
 1;