3.11: As a user, I should see a graph with both my performance and my opponent's performance with respect to Displacement, as well as an optimal Graph

GoogleCodeExporter commented 9 years ago

Tasks:
1-story sequence diagram & artifact
2-generating a graph depicting both players' and the optimum displacement 
versus time
3-documentation of method

Original issue reported on code.google.com by asamg...@gmail.com on 13 Apr 2012 at 7:03

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 17 Apr 2012 at 7:31

Changed state: In-Progress

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r369.

Status: In Progress

Updated the PerformanceGraph class to draw the displacements of the competitors 
when the included functions are called in the class Game

Original comment by asamg...@gmail.com on 23 Apr 2012 at 9:16

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r382.

Status: In Progress

Added images and a Spritefont to be used in experiment 1

Original comment by asamg...@gmail.com on 23 Apr 2012 at 10:29

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 24 Apr 2012 at 10:28

Added labels: Implementation-Ready, Design-Ready, Documentation-Ready

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r504.

Status: In Progress

Updated the PerformanceGraph class to accept any number of Displacement frames  
from the users

Original comment by asamg...@gmail.com on 25 Apr 2012 at 5:37

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r507.

Status: In Progress

Updated the PerformanceGraph to accept 12 fps

Original comment by asamg...@gmail.com on 25 Apr 2012 at 5:57

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r508.

Status: In Progress

Updated the test values to match 12 fps

Original comment by asamg...@gmail.com on 25 Apr 2012 at 5:58

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r522.

Status: In Progress

Divided the function responsible for drawing the axis and the labels into 4 
functions

Original comment by asamg...@gmail.com on 25 Apr 2012 at 8:41

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r523.

Status: In Progress

Altered the Game class to call the new functions drawing the axis

Original comment by asamg...@gmail.com on 25 Apr 2012 at 8:42

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r536.

Status: In Progress

Divided the drawEnvironment function to two more functions: drawXLabels and 
drawYLabels

Original comment by asamg...@gmail.com on 25 Apr 2012 at 10:42

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

Scenario review: The optimum displacement is not correctly calculated in case 
the optimum player reached the destination before the end of the race

Original comment by asamg...@gmail.com on 26 Apr 2012 at 12:08

GoogleCodeExporter commented 9 years ago

Scenario review: Bugs detected in getOptimum

Original comment by asamg...@gmail.com on 26 Apr 2012 at 2:24

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r541.

Status: In Progress

Fixed all bugs in the getOptimum function

Original comment by asamg...@gmail.com on 26 Apr 2012 at 4:18

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r542.

Status: In Progress

Added new testing commands after fixing the getOptimum() function bugs

Original comment by asamg...@gmail.com on 26 Apr 2012 at 4:19

GoogleCodeExporter commented 9 years ago

All bugs in the getOptimum() function have been resolved

Original comment by asamg...@gmail.com on 26 Apr 2012 at 4:19

GoogleCodeExporter commented 9 years ago

Documentation review: Documentation required for the new functions

Original comment by asamg...@gmail.com on 26 Apr 2012 at 4:21

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r543.

Status: In Progress

Updated the documentation of the PerformanceGraph class

Original comment by asamg...@gmail.com on 26 Apr 2012 at 6:41

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r544.

Status: In Progress

Updated the class "Game"s documentation

Original comment by asamg...@gmail.com on 26 Apr 2012 at 6:52

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 26 Apr 2012 at 7:04

Added labels: Documentation-Verified
Removed labels: Documentation-Ready

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r563.

Status: In Progress

Added graphs to the screen manager, changed performance graph and countdown to 
accept width/height, commented a line in game to fix an error

Original comment by asamg...@gmail.com on 26 Apr 2012 at 10:57

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r565.

Status: In Progress

Added graphics device ,

Original comment by asamg...@gmail.com on 26 Apr 2012 at 11:11

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r582.

Status: In Progress

Fixed all logic errors in calculating the optimal values

Original comment by asamg...@gmail.com on 26 Apr 2012 at 1:32

GoogleCodeExporter commented 9 years ago

Docmentation review: Fix the documentation for the new and fixed functions

Original comment by asamg...@gmail.com on 26 Apr 2012 at 3:27

Added labels: Documentation-ReOpened
Removed labels: Documentation-Verified

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 26 Apr 2012 at 4:19

Added labels: Scenario-Ready

GoogleCodeExporter commented 9 years ago

Scenario review:
The graphs are not compatible with the new screen manager and the values 
obtained from the kinect.

Original comment by asamg...@gmail.com on 26 Apr 2012 at 9:20

Added labels: Scenario-ReOpened
Removed labels: Scenario-Ready

GoogleCodeExporter commented 9 years ago

1- change the variables' names in class PerformanceGraph.cs to use Camel-case 
naming convention

2- change the variables' names to be a bit more comprehensive (i.e: a1, a2, a3, 
...etc have no meaning and can not be comprehended)

3- make all the instance variables private and create getters and setters for 
them except those that might be used publicly or statically

4- the method drawLabels have some unused parameters(GraphicsDevice and Font2) 
and also drawAxis, drawArrows, drawXLabels and drawYLabels

5- the loops in method OptimumIncreasingAcceleration can be combined together 
and also OptimumDecreasingAcceleration

Original comment by michel.n...@gmail.com on 26 Apr 2012 at 10:08

Added labels: Implementation-ReOpened
Removed labels: Implementation-Ready

GoogleCodeExporter commented 9 years ago

@Michel...The first four points will be done today, but please check carefully 
the differences between the functions OptimumIncreasingAcceleration and 
OptimumDecreasingAcceleration to know why I could not combine them, thanks.

Original comment by asamg...@gmail.com on 27 Apr 2012 at 11:33

GoogleCodeExporter commented 9 years ago

It's about about combining the methods themselves it's that there's two or 
three loops loops inside the first method that got the same head so these loops 
should be combined together in one loop, and the second method also got the 
same issue, that's it.

Original comment by michel.n...@gmail.com on 27 Apr 2012 at 11:36

GoogleCodeExporter commented 9 years ago

@Michel, Ill try.

Original comment by asamg...@gmail.com on 27 Apr 2012 at 11:52

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r678.

Status: In Progress

Updated the documentation.

Original comment by asamg...@gmail.com on 27 Apr 2012 at 2:27

GoogleCodeExporter commented 9 years ago

Implementation Review: 5- one more thing, in the if conditions each statement 
should be surrounded by (). (i.e: if(A && B) -> if((A) && (B)) and the same 
with the ||)

Original comment by michel.n...@gmail.com on 27 Apr 2012 at 3:04

GoogleCodeExporter commented 9 years ago

if this is a generated UML please change it .. thank you

Original comment by asafty2...@gmail.com on 27 Apr 2012 at 4:56

Added labels: Design-ReOpened
Removed labels: Design-Ready

GoogleCodeExporter commented 9 years ago

This issue was updated by revision r743.

Changed the visibility of the fields to private

Original comment by asamg...@gmail.com on 27 Apr 2012 at 8:07

Changed state: Inprogress

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 27 Apr 2012 at 8:18

Added labels: Implementation-Ready
Removed labels: Implementation-Verified

GoogleCodeExporter commented 9 years ago

@Michel 3amalt kol el points bta3tak bas a5er wa7da mesh 3aref a3melha wel 
experiment metlassama aslan w lesaly el umls

Original comment by asamg...@gmail.com on 27 Apr 2012 at 8:38

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 27 Apr 2012 at 9:59

Added labels: Design-Ready, Documentation-Verified
Removed labels: Design-ReOpened, Documentation-ReOpened

GoogleCodeExporter commented 9 years ago

bos heya a5er no2ta dih sahla awy heya el fekra en heya heya nafs el loop bs 
elly bye7sal gowaha mo5talef fa momken yeb2o m3 b3d w 5las, it's easy sada2ny 
bs bos lw enta mota2aked ennaha hattala3 error olly.

Original comment by michel.n...@gmail.com on 27 Apr 2012 at 10:03

GoogleCodeExporter commented 9 years ago

Original comment by asamg...@gmail.com on 27 Apr 2012 at 10:09

Added labels: Scenario-Verified
Removed labels: Scenario-ReOpened

ssafty / terasoft-12

3.11: As a user, I should see a graph with both my performance and my opponent's performance with respect to Displacement, as well as an optimal Graph #31