Modify

Opened 6 years ago

Closed 5 years ago

#9515 closed enhancement (fixed)

Scheduling logic redesign

Reported by: dinu Owned by: Chris Nelson
Priority: normal Component: TracJsGanttPlugin
Severity: normal Keywords:
Cc: Ryan J Ollos, falkb Trac Release: 0.12

Description

I made a change for the scheduling logic to make it useful for real world planning. The planned start and end fields should be authoritative: I may know that the person the bug is assigned to only works half-time, or also has other tasks, or is going away for a number of days. So a 2-hour ticket may be planned to 2 days. Redesigned the logic to only compute start and finish if not specified.

Attachments (1)

schedulingLogic.patch (3.0 KB) - added by dinumarina@… 6 years ago.

Download all attachments as: .zip

Change History (23)

Changed 6 years ago by dinumarina@…

Attachment: schedulingLogic.patch added

comment:1 Changed 6 years ago by Ryan J Ollos

Cc: Ryan J Ollos added; anonymous removed

comment:2 Changed 6 years ago by falkb

Cc: falkb added

comment:3 Changed 6 years ago by anonymous

Owner: changed from Chris Nelson to anonymous
Status: newassigned

comment:4 Changed 6 years ago by Chris Nelson

Owner: changed from anonymous to Chris Nelson
Status: assignednew

comment:5 Changed 6 years ago by Chris Nelson

(In [10904]) Honor user-specified start and finish date. Refs #9515.

This seems useful in some cases and harmless in others. I tested with an 8-hour task with 2 hours done and a week between start and finish and it showed 25% of the task filled in.

Based on a patch from dinumarina. Thanks.

comment:6 Changed 5 years ago by Chris Nelson

Status: newassigned

comment:7 Changed 5 years ago by Chris Nelson

(In [11069]) Handle implicit dates in parent tickets. Refs #9042, #9515.

Need to carry around an indication of whether the date is implicit or explicit.

comment:8 Changed 5 years ago by Chris Nelson

(In [11071]) Refactor PM stuff out of Gantt class. Refs #9024, #9515, #8982.

comment:9 Changed 5 years ago by Chris Nelson

(In [11072]) Refactor scheduling out of PM class. Refs #8982, #9042, #9515.

comment:10 Changed 5 years ago by Chris Nelson

(In [11073]) Move Project Management stuff into its own module. Refs #8982, #9042, #9515.

comment:11 Changed 5 years ago by Chris Nelson

(In [11074]) Make successor, predecessors consistent with parent, children. Refs #9042, #9515.

predecessors() and successors() returned lists of strings (e.g., ['123', '234']) but parent() returns an integer and children() returns a list of integers. Now all handle integers.

comment:12 Changed 5 years ago by Chris Nelson

(In [11076]) Clean up start and finish handling. Refs #9042, #9515.

  • Refactor start and finish parse routines to set time to midnight.
  • Don't set time to midnight in values returned from parse routines.
  • When defaulting to today, set time to midnight (including seconds, useconds)

comment:13 Changed 5 years ago by Chris Nelson

(In [11077]) Rename percent() to percentComplete() for clarity. Refs #9042, #9515.

Some other refactoring of percent complete calculations.

comment:14 Changed 5 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:15 Changed 5 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:16 Changed 5 years ago by Chris Nelson

(In [11084]) Handle explicit finish with implicit start in ASAP schedule. Refs #9042, #9515.

And vice versa in ALAP.

Also removed unnecessary fixup of explicit start; parseStart() already zeros out those values.

comment:17 Changed 5 years ago by dinu

Could you remove my email please; seems it's getting crawled :) Thanks

comment:18 in reply to:  17 Changed 5 years ago by Chris Nelson

Replying to dinu:

Could you remove my email please; seems it's getting crawled :) Thanks

Regrettably, I don't have permission to do that. Ryan -- who is Cc'd on this ticket -- knows people, perhaps he can pass the word.

comment:19 Changed 5 years ago by Ryan J Ollos

Reporter: changed from dinumarina@… to dinu

comment:20 Changed 5 years ago by Ryan J Ollos

I could delete the patch and comments by dinu, but I don't see a way I can change the Reporter field without having a comment that shows the old and new Reporter's name, thereby leaving the email address in the ticket.

I'll have full admin permissions on t-h.o soon, and then might be able to fix the issue by direct database edit.

comment:21 in reply to:  20 Changed 5 years ago by anonymous

Ask osimons on IRC... he knows a secret guy with admin permissions who can edit the database directly...

A friend

comment:22 Changed 5 years ago by Chris Nelson

Resolution: fixed
Status: assignedclosed

This seems to be working now.

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.