Re: [Planner] Cost calculations

[Waldemar Augustyn <waldemar nxp com>]

Thanks!

Alvaro del Castillo wrote:

Hi!

I am sure you think we have forgotten your email but ... no :)

Here goes the patch to correct the bug you have found. I will send
another email out of this thread in order all the people read it.

Cheers and thanks 

El mié, 14-01-2004 a las 01:12, Waldemar Augustyn escribió:
  

I have just noticed, Planner's cost calculation is not quite right.  It 
appears, it misinterprets the resource "units" value. I thought the 
"units" figure is really the percentage of time a resource is expected 
(allowed) to work on a task.  The default is 100, which I think should 
be labeled with a %.

Here is what happens:

A  Task takes 3 days to finish.  There is one resource allocated to it 
at 50% time and 45 cost.  Planner properly calculates the task to take 
six days (because of 50% resource), but the cost is calculated at 540.   
I would think, it should be 3*8*45= 1080  (three days * 8 hours * 45 
$/hour).  Or, perhaps, 6*4*45=1080 (six days * 4 hours * 45 $/h).

Same thing with another task.  It takes 5 days to finish, the allocated 
resource works 5% of time at cost of 30.  Planner properly calculates 
the task to take 100 days to finish, but the cost is calculated at 60.  
I would think, it should be 5*8*30=1200.  Or,  100*(8/20)*30=1200.

I am using version 0.11.


_______________________________________________
Planner mailing list
Planner lists imendio com
http://lists.imendio.com/mailman/listinfo/planner
    

Index: libplanner/mrp-task.c
===================================================================
RCS file: /cvs/gnome/planner/libplanner/mrp-task.c,v
retrieving revision 1.2
diff -u -b -B -p -r1.2 mrp-task.c
--- libplanner/mrp-task.c	17 Feb 2004 20:00:17 -0000	1.2
+++ libplanner/mrp-task.c	6 Apr 2004 07:38:24 -0000
@@ -1569,7 +1569,7 @@ mrp_task_get_cost (MrpTask *task)
 		resource = mrp_assignment_get_resource (l->data);
 
 		mrp_object_get (resource, "cost", &cost, NULL);
-		total += mrp_assignment_get_units (l->data) * priv->work * cost / (3600.0 * 100);
+		total += priv->work * cost / 3600.0;
 	}
 
 	return total;
  

_______________________________________________
Planner mailing list
Planner lists imendio com
http://lists.imendio.com/mailman/listinfo/planner