search

khaledosman / terasoft-12

Automatically exported from code.google.com/p/terasoft-12
0 stars 0 forks source link

4.07: As a user, I can test the velocity and angle first before throwing #242

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Tasks:
1-Doing refinements to the way the angle was calculated in stories 4.06 and old 
4.07
http://code.google.com/p/terasoft-12/issues/detail?id=11&can=3
http://code.google.com/p/terasoft-12/issues/detail?id=17&can=3

Original issue reported on code.google.com by mohamedra2fat on 12 May 2012 at 11:22

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 12 May 2012 at 11:24

GoogleCodeExporter commented 9 years ago 
Forget to add Priority label.
priority label is now done

Original comment by mohamedra2fat on 12 May 2012 at 11:26

GoogleCodeExporter commented 9 years ago 
Started working on it,,by re factoring the whole code of the two other methods 
of the other two stories that I linked.

Original comment by mohamedra2fat on 17 May 2012 at 1:13

GoogleCodeExporter commented 9 years ago 
Started Working

Original comment by mohamedra2fat on 18 May 2012 at 12:24

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1095.

Calculating the angle with "Plane and vector Calculation" Done.

Original comment by mohamedra2fat on 18 May 2012 at 12:26

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1104.

Calculating the angle with "Plane and vector Calculation" Done.

Original comment by mohamedra2fat on 18 May 2012 at 10:08

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1106.

modified some instance variables

Original comment by mohamedra2fat on 18 May 2012 at 12:57

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1119.

Handled the logic that will get the angle and the time in the last few frames

Original comment by mohamedra2fat on 18 May 2012 at 3:56

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1121.

Status in progress
Done with the angle calculation using new algorithm

Original comment by mohamedra2fat on 18 May 2012 at 4:41

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1125.

edited some line codes

Original comment by mohamedra2fat on 18 May 2012 at 5:34

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1148.

Status In progress
Added setter and getters for all instance variables,
finished the method MeasureVelocity()

Original comment by mohamedra2fat on 19 May 2012 at 11:35

GoogleCodeExporter commented 9 years ago 
Implementation and Documentation on this story is ready

Original comment by mohamedra2fat on 19 May 2012 at 11:38

GoogleCodeExporter commented 9 years ago 
Documentation Verified

Original comment by mohamedra2fat on 19 May 2012 at 11:38

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1163.

Status In progress
Changed if(shooting == true) -> if(shooting)

Original comment by mohamedra2fat on 19 May 2012 at 7:58

GoogleCodeExporter commented 9 years ago 
You dont follow Rules of changing names of methods and variables that we all 
agreed before.Depreciation implemented in C# and must be used to tell what is 
the old method name and what is the new method to be used ! And after you get 
sure that others have done the change : You delete the old method.

because what you did is not software engineering.You changed the method names 
and ignored other developers that are using it which resulted in alot of 
compile errors + wasting other developers time !.

**Most of your recent commits are marked now as negative because of this.

Original comment by hollaxy...@gmail.com on 19 May 2012 at 10:03

GoogleCodeExporter commented 9 years ago 
Well, Tamer I asked Abdelazim today, and he told me no one suffered from the 
changes that I did in the class user2 because I did not change any signature of 
any method, I just added instance variables that I use , and changed the logic 
of the Methods, which will not affect compilation error.
SO there was no need for the negative things.
Thank you.

Original comment by mohamedra2fat on 20 May 2012 at 7:50

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 20 May 2012 at 7:51

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 20 May 2012 at 7:51

GoogleCodeExporter commented 9 years ago 
Your code is doing nothing.It does not measure anything at all.here is 
screenShot of what happens and does not change : 

http://i.imgur.com/y0PeV.png

Please next time test your code first before making Scenario-Ready.

Original comment by hollaxy...@gmail.com on 20 May 2012 at 10:40

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1241.

Removed unused Fields in the class

Original comment by mohamedra2fat on 20 May 2012 at 10:54

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1308.

Status in progress
Just removed the new code, and added the old one because the new code still has 
some small bugs to be fixed. So I did this for the sake of the demo

Original comment by mohamedra2fat on 22 May 2012 at 6:14

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 22 May 2012 at 9:46

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1370.

Status In Progress
Initialized the Button Positions.
Loaded the Simulation and Button Positions
Done withe Instance Variables and Constructor

Original comment by mohamedra2fat on 23 May 2012 at 7:06

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1371.

Status In Progress
Corrected Compilation error, due to Deprecated methods

Original comment by mohamedra2fat on 23 May 2012 at 7:25

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1372.

Status In Progress
Corrected Compilation error, due to Deprecated methods

Original comment by mohamedra2fat on 23 May 2012 at 7:50

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1388.

Status In progress
Deleted class StatisticScreen2.cs AS it should be moved to Exp2

Original comment by mohamedra2fat on 23 May 2012 at 10:15

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1390.

Status In progress
Moved class StatisticScreen2.cs to Exp2

Original comment by mohamedra2fat on 23 May 2012 at 10:16

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1464.

Added button textures created by Hegazy as I needed these buttons

Original comment by mohamedra2fat on 23 May 2012 at 4:18

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1619.

Finished Documentation

Original comment by mohamedra2fat on 24 May 2012 at 8:14

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 24 May 2012 at 8:15

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 24 May 2012 at 8:15

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 24 May 2012 at 9:31

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 24 May 2012 at 4:51

GoogleCodeExporter commented 9 years ago 
It dont work at all and showing exception.

Original comment by hollaxy...@gmail.com on 24 May 2012 at 6:00

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 24 May 2012 at 11:10

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 25 May 2012 at 11:16

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1846.

Status ready
Fixed measuring angle and velocity

Original comment by mohamedra2fat on 25 May 2012 at 1:05

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 25 May 2012 at 1:33

GoogleCodeExporter commented 9 years ago

Original comment by 3abdelazim on 25 May 2012 at 1:39

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 25 May 2012 at 4:23

GoogleCodeExporter commented 9 years ago 
Ok as we agreed in the meeting of yesterday and today that measuring angle and 
velocity  is not 100 % correct and that we will leave it that since there is no 
time to go for new logic for it. and that it will be fixed in sprint 3.

Original comment by hollaxy...@gmail.com on 25 May 2012 at 4:54

GoogleCodeExporter commented 9 years ago 
design notes:
typo in the name of the story
++counter%3 ????
there are lots of arrows coming back to the Actor .. what do they represent .. 
and why so many?

Original comment by asafty2...@gmail.com on 25 May 2012 at 8:08

GoogleCodeExporter commented 9 years ago 
These are Asynchronous  methods that doesn't return anything,,so I think they 
should return empty dotted line
I will change the typo

Original comment by mohamedra2fat on 25 May 2012 at 9:20

GoogleCodeExporter commented 9 years ago

Original comment by mohamedra2fat on 25 May 2012 at 9:29