Modify

Opened 13 years ago

Closed 13 years ago

#9533 closed defect (fixed)

Ignoring weekends!

Reported by: Robert Inder Owned by: Chris Nelson
Priority: normal Component: TracJsGanttPlugin
Severity: major Keywords:
Cc: Ryan J Ollos Trac Release: 0.12

Description

I have a handful of tickets which have start and end dates, and estimated hours.

The chart is ignoring/over-riding the start date to calculate its own. But in doing so, it ignores weekends.

So I have a ticket with start date 2012-01-09 and finish date 2012-02-17 and 150 hours estimated. But when I draw a chard with hoursPerDay=7, it shows up on the chart as end date 2012-02-17, start date 2010-01-27.

So the duration has been reduced to 22 days elapsed, which is 149 hours if you work seven days a week.

If I reduce "hoursPerDay" to 5, the start day is pushed to the 19th of January. 30 days at 5 hours per day, 7 days per week.

So contrary to what it says at http://trac-hacks.org/wiki/TracJsGanttPlugin this is taking no account weekends.

Robert.

Attachments (0)

Change History (9)

comment:1 Changed 13 years ago by Robert Inder

Trac Release: 0.110.12

Ooops! Forgot to change the Trac release. Sorry.

comment:2 Changed 13 years ago by Ryan J Ollos

Cc: Ryan J Ollos added; anonymous removed

comment:3 Changed 13 years ago by Chris Nelson

(In [11079]) Cleaner date adjustment using calendarOffset. Refs #9533, #9300, #9042, #9515.

Scheduler used to assume work week. Now let calendarOffset() do that.

Also, it seems rounding errors would creep in and some datetimes would have non-zero microseconds. This tests for beginning and end of day with a little delta.

comment:4 Changed 13 years ago by Chris Nelson

(In [11080]) More flexible calendar offset calculation. Refs #9533, #9300.

comment:5 Changed 13 years ago by Chris Nelson

(In [11081]) Copy some dates before returning them. Refs #9042, #9515, #9533, #9300.

I'm not 100% sure why this is necessary but the wrong things seemed to get updated if I didn't do this and give demonstrably wrong results.

comment:6 Changed 13 years ago by Chris Nelson

(In [11082]) Add calendar handling to scheduler. Refs #9533, #9300.

comment:7 Changed 13 years ago by Chris Nelson

(In [11083]) Rework calendar offset adjustment to not go outside working hours. Refs #9648, #9533, #9300.

comment:8 Changed 13 years ago by Chris Nelson

(In [11142]) Correct end-of-day adjustments for ASAP and < 100% effort. Refs #9533, #9300.

When working forward to build an ASAP schedule, partial takes caused the tasks to shift forward more than their scheduled length.

If one resource can work 8 hours per day and another can only work 4, the second resource takes twice as long to complete the same task. But without this clean up, ALAP scheduling took an 8-hour task at 4 hours per day from the end of day 2 to the middle of day 1. With this change, if all available hours are used up, the rest of the day is consumed so that 8-hour task goes from the end of day 2 to the start of day 1.

comment:9 Changed 13 years ago by Chris Nelson

Resolution: fixed
Status: newclosed

The calendar-based scheduler handles weekends by default. Ambitious users can plug in their own calendar for other needs.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Chris Nelson.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.