AI-comp / wpcs

2 stars 2 forks source link

Fix score board and submission history to show acquired scores. Close #9 #128

Closed exKAZUu closed 11 years ago

exKAZUu commented 11 years ago

9

coveralls commented 11 years ago

Coverage Status

Coverage remained the same when pulling a13f762925f084597fa0583606c80ff6b43e6204 on 9_score into d33453455f10307b96a9571546a24694ff5ac298 on master.

exKAZUu commented 11 years ago

This commits also deal with #126

coveralls commented 11 years ago

Coverage Status

Coverage remained the same when pulling 40815a0d756153413f267f862672ebacce03df65 on 9_score into d33453455f10307b96a9571546a24694ff5ac298 on master.

ss0201 commented 11 years ago

As I've pointed out in #126, I don't think showing the absolute time for each problem is reasonable. It's better to show the elapsed time from the start of the contest.

Also, I think it's better to describe somewhere in the scoreboard that the top of the cell is for "small" and the bottom is for "large." Maybe at the right of the "Score" column? I want to discuss it in #126.

ss0201 commented 11 years ago

And I thought that #9 means showing like "150/200" in the "Scores" field in the problem list, which indicates that you'll get 150 points if you solve this problem right now, and that the maximum score of this problem is 200. At least that's why I closed #55.

exKAZUu commented 11 years ago

I see. I will complete #9.

exKAZUu commented 11 years ago

I fixed them to show max score.

ss0201 commented 11 years ago

Really? I don't think it's working. Did you push your commit?

exKAZUu commented 11 years ago

Sorry, I just now pushed my commits!

coveralls commented 11 years ago

Coverage Status

Changes Unknown when pulling 6bb7b872167b7c3331c125bb5724c26fa144f303 on 9_score into \ on master**.

exKAZUu commented 11 years ago

And more!

exKAZUu commented 11 years ago

I finished to push all my commits.

coveralls commented 11 years ago

Coverage Status

Changes Unknown when pulling a4a85eff77b2af3bcaa990aad5044f6473aa01e8 on 9_score into \ on master**.

ss0201 commented 11 years ago

Ah... you've actually done just as I wrote in #126. OK. For now let's use this lame table, but I'll make an issue to make it cool :)