Quote:
Originally Posted by engunneer
as i am sure you are aware, there is some debate as to rotors spinning in auto counting in teleop. You''l have to wait for the rule update, but i would have a plan for that. The score would be (60*autorotors + (40*(telerotors-autorotors))) in that case
What are the seconds indicators? time to process the fuel? I like that you count high and low processing time separately.
I was confused by the 1s 5s 10s buttons. maybe change text to +/-1 +/-5 +/-10? There is a symbol for +/- , or you could label the button group. I thought at first it was related to seconds.
It took me quite a few seconds to find the output. Is there room to make the text bigger, or use a javascript transition to flash it when it changes?
there seems to be no way to remove fouls? also fouls allow a negative score.
Lastly, the plural of Fuel is Fuel
That may sound like a lot of criticism, but actually that's a very small bug list. Great work. Very useful! You covered nearly all of the test cases i could even think of.
|
Great constructive feedback. Thanks! It took a bit of work to get the calculations correct, especially the left over fuel from auto which still counts as a fractional score.
The README.md file on GitHub makes mention of some of my thought processes regarding some of the issues you brought up.
I'm certainly going to make some changes tonight based off your comments. Much appreciated!