getting it working again after commit avalanche on 30/Dec/2011

Well, that's embarrassing.

(In [11087]) Wrong scope for pseudoticket function. Refs #9653.

[11087] helps for the error described above, thanks!

next error after update to [11087], (very much appreciating your work (!) but you should consider to setup a test environment for a set of cases):

[2012-01-02 09:06:09,565 p5620:t2576] formatter.py:720 ERROR: Macro TracJSGanttChart(milestone=XX X-XXXX1 - XX|XX X-XXXX2 - XX,userMap=0,format=week,resolution!=duplicate|invalid|wontfix|worksforme) failed: 
Traceback (most recent call last):
  File "build\bdist.win32\egg\trac\wiki\formatter.py", line 717, in _macro_formatter
    return macro.process(args, in_paragraph=True)
  File "build\bdist.win32\egg\trac\wiki\formatter.py", line 304, in process
    text = self.processor(text)
  File "build\bdist.win32\egg\trac\wiki\formatter.py", line 291, in _macro_processor
    text)
  File "build\bdist.win32\egg\tracjsgantt\tracjsgantt.py", line 606, in expand_macro
    tasks = self._add_tasks(options)
  File "build\bdist.win32\egg\tracjsgantt\tracjsgantt.py", line 544, in _add_tasks
    self.pm.postQuery(options, self.tickets)
  File "build\bdist.win32\egg\tracjsgantt\tracpm.py", line 542, in postQuery
    self._add_milestones(options, tickets)
  File "build\bdist.win32\egg\tracjsgantt\tracpm.py", line 450, in _add_milestones
    milestoneTicket = self._pseudoTicket(id,
AttributeError: 'TracPM' object has no attribute '_pseudoTicket'

(In [11096]) Typo in pseudoTicket function name. Refs #9653.

Replying to falkb:

[11087] helps for the error described above, thanks! next error after update to [11087], (very much appreciating your work (!) but you should consider to setup a test environment for a set of cases): ...

I actually do have a test environment but had some of the charts disabled while I worked on scheduling algorithms so it was easier to see the changes. (My prime motivation in integrating the multiple-charts-per-page patch was to allow me to have one page with all my test tests on it but sometimes that's just too much information.)

Thanks. next step done :-) Now:

Cannot find an implementation of the "ITaskScheduler" interface named "CalendarScheduler".  Please update the option TracPM.scheduler in trac.ini.

I activated "CalendarScheduler? (tracjsgantt.tracpm.*)" and the chart comes up now. Is it right that I have to activate it? Do I need to activate the other 2 new ones (SimpleCalender?, TracPM) as well?

Replying to falkb:

Do I need to activate the other 2 new ones (SimpleCalender?, TracPM) as well?

I activated all 3 ones now but it makes no visible difference.

Well, sorry for the bad news, now all ticktes are shown with duration 1 day here (which is wrong), start and end date are displayed as today (1/2/2012) and all ticket dependencies are lost in the chart display (no connection lines anymore)...

Replying to falkb:

Thanks. next step done :-) Now: {{{ Cannot find an implementation of the "ITaskScheduler" interface named "CalendarScheduler?". Please update the option TracPM.scheduler in trac.ini. }}} I activated "CalendarScheduler? (tracjsgantt.tracpm.*)" and the chart comes up now.

Great!

Is it right that I have to activate it? Do I need to activate the other 2 new ones (SimpleCalender?, TracPM) as well?

I've struggled with getting ExtensionOption? working. It *should* work as written as I understand the documentation. I've asked on the trac-users (or trac-dev?) list and not gotten much useful help.

Replying to falkb:

Replying to falkb:

Do I need to activate the other 2 new ones (SimpleCalender?, TracPM) as well?

I activated all 3 ones now but it makes no visible difference. Well, sorry for the bad news, now all ticktes are shown with duration 1 day here (which is wrong), start and end date are displayed as today (1/2/2012) and all ticket dependencies are lost in the chart display (no connection lines anymore)...

Is there anything in your trac.log?

Replying to ChrisNelson:

Is there anything in your trac.log?

not with WARNING level. Timeout here for now... I'm gonna come back later with DEBUG level logging if you want to...

Replying to falkb:

Replying to ChrisNelson:

Is there anything in your trac.log?

not with WARNING level. Timeout here for now... I'm gonna come back later with DEBUG level logging if you want to...

Aha! Now I remember. Several of the former Trac-jsGantt settings have moves with the refactoring of the PM stuff into it's own module.

[TracPM]
date_format = %Y-%m-%d
fields.estimate = estimatedhours
fields.finish = userfinish
fields.parent = parents
fields.pred = blockedby
fields.start = userstart
fields.succ = blocking
fields.worked = totalhours
milestone_type = inchpebble

I'll update the documentation on the plugin page (and the in-line documentation, too, if it's wrong).

Replying to ChrisNelson:

... I'll update the documentation on the plugin page (and the in-line documentation, too, if it's wrong).

Plugin page updated.

(In [11097]) Clarify where options are configured. Refs #9653.

Replying to ChrisNelson:

Aha! Now I remember. Several of the former Trac-jsGantt settings have moves with the refactoring of the PM stuff into it's own module. {{{ [TracPM]

Replying to ChrisNelson:

Aha! Now I remember. Several of the former Trac-jsGantt settings have moves with the refactoring of the PM stuff into it's own module. {{{ [TracPM]

Yes, that renaming the trac.ini section from trac-jsgantt to TracPM helps to get further.

1. Which options actually remain in trac-jsgantt?
2. for backward-compatibility, isn't it a good idea to keep with trac-jsgantt?
3. If it's not shared by more than one plugin, isn't it a good idea to use the plugin name as section name?

Well, now we can go on and get to a next error:

[2012-01-03 08:37:48,849 p2600:t1092] formatter.py:720 ERROR: Macro TracJSGanttChart(milestone=XX X-XXXX1 - XX|XX X-XXXX2 - XX,userMap=0,format=week,resolution!=duplicate|invalid|wontfix|worksforme) failed: 
Traceback (most recent call last):
  File "build\bdist.win32\egg\trac\wiki\formatter.py", line 717, in _macro_formatter
    return macro.process(args, in_paragraph=True)
  File "build\bdist.win32\egg\trac\wiki\formatter.py", line 304, in process
    text = self.processor(text)
  File "build\bdist.win32\egg\trac\wiki\formatter.py", line 291, in _macro_processor
    text)
  File "build\bdist.win32\egg\tracjsgantt\tracjsgantt.py", line 606, in expand_macro
    tasks = self._add_tasks(options)
  File "build\bdist.win32\egg\tracjsgantt\tracjsgantt.py", line 570, in _add_tasks
    tasks += self._format_ticket(ticket, options)
  File "build\bdist.win32\egg\tracjsgantt\tracjsgantt.py", line 480, in _format_ticket
    task += '"%s",' % self.pm.percentComplete(ticket)
  File "build\bdist.win32\egg\tracjsgantt\tracpm.py", line 308, in percentComplete
    worked = float(ticket[self.fields['worked']])
ValueError: empty string for float()

I'll update the documentation on the plugin page (and the in-line documentation, too, if it's wrong).

The wiki still mentions that old section name.

Replying to falkb:

ValueError: empty string for float()

This patch should help as fix:

Index: tracjsganttplugin/0.11/tracjsgantt/tracpm.py
===================================================================
--- tracjsganttplugin/0.11/tracjsgantt/tracpm.py	(revision 11100)
+++ tracjsganttplugin/0.11/tracjsgantt/tracpm.py	(working copy)
@@ -305,7 +305,11 @@
             if (estimate == 0):
                 percent = 0
             else:
-                worked = float(ticket[self.fields['worked']])
+                workedStr = ticket[self.fields['worked']]
+                if workedStr == '':
+                    worked = 0
+                else:
+                    worked = float(workedStr)
                 percent = '%s/%s' % (worked, estimate)
         # Use percent if provided
         elif self.isCfg('percent'):

Now with that local patch the chart comes up without warnings again!

Replying to falkb:

Replying to falkb:

ValueError: empty string for float()

This patch should help as fix: {{{ #!diff Index: tracjsganttplugin/0.11/tracjsgantt/tracpm.py =================================================================== --- tracjsganttplugin/0.11/tracjsgantt/tracpm.py (revision 11100) +++ tracjsganttplugin/0.11/tracjsgantt/tracpm.py (working copy) @@ -305,7 +305,11 @@ if (estimate == 0): percent = 0 else: - worked = float(ticket[self.fieldsworked?]) + workedStr = ticket[self.fieldsworked?] + if workedStr == : + worked = 0 + else: + worked = float(workedStr) percent = '%s/%s' % (worked, estimate) # Use percent if provided elif self.isCfg('percent'): }}} Now with that local patch the chart comes up without warnings again!

Thanks. I *swear* I've put that patch in before. Maybe it got lost in refactoring.

Replying to falkb:

... Yes, that renaming the trac.ini section from trac-jsgantt to TracPM helps to get further. 1. Which options actually remain in trac-jsgantt?

The site-wide default overrides (e.g., option.userMap=0).

2. for backward-compatibility, isn't it a good idea to keep with trac-jsgantt? 3. If it's not shared by more than one plugin, isn't it a good idea to use the plugin name as section name?

This is a first step toward breaking TracPM out into its own plugin that can support not just jsGantt but also others like a WBS editor, etc.

Well, now we can go on and get to a next error: ... ValueError?: empty string for float()

I'll update the documentation on the plugin page (and the in-line documentation, too, if it's wrong).

The wiki still mentions that old section name.

I see that the configuration section has an example of [TracPM] and the option table says you can put site-wide default overrides in [trac-jsgantt]. What are you referring to?

Replying to ChrisNelson:

I see that the configuration section has an example of [TracPM] and the option table says you can put site-wide default overrides in [trac-jsgantt]. What are you referring to?

I wasn't aware of the difference between site-wide default overrides and that breakout stuff. Now it's clear and fine with me. I think after committing that patch from above you can close this ticket now. Thanks for your fast reply. :-)

(In [11103]) Handle blank worked values. Refs #9653.

Replying to falkb:

Replying to ChrisNelson:

I see that the configuration section has an example of [TracPM] and the option table says you can put site-wide default overrides in [trac-jsgantt]. What are you referring to?

I wasn't aware of the difference between site-wide default overrides and that breakout stuff. Now it's clear and fine with me. I think after committing that patch from above you can close this ticket now. Thanks for your fast reply. :-)

Thanks for your feedback!