Modify

Opened 3 years ago

Closed 3 years ago

#9533 closed defect (fixed)

Ignoring weekends!

Reported by: robert@… Owned by: ChrisNelson
Priority: normal Component: TracJsGanttPlugin
Severity: major Keywords:
Cc: rjollos 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 3 years ago by robert@…

  • Trac Release changed from 0.11 to 0.12

Ooops!
Forgot to change the Trac release.
Sorry.

comment:2 Changed 3 years ago by rjollos

  • Cc rjollos added

comment:3 Changed 3 years ago by ChrisNelson

(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 3 years ago by ChrisNelson

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

comment:5 Changed 3 years ago by ChrisNelson

(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 3 years ago by ChrisNelson

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

comment:7 Changed 3 years ago by ChrisNelson

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

comment:8 Changed 3 years ago by ChrisNelson

(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 3 years ago by ChrisNelson

  • Resolution set to fixed
  • Status changed from new to closed

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

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.