A couple of suggestions:

First, please include the entire program.  What you've supplied is apparently only subroutines; there is no use Tk, nor use Tk::Graph, and no program main from which any of the subroutines are called.

Secondly, please provide a program which *works*.  When I make the assumption that you're trying to call reportg(), I get the following:

#!/usr/bin/perl

use Tk;
use Tk::Graph;

reportg();

# The rest of your code as you provided it...

#
# Output
#
Tk::Error: Can't set -variable to `undef' for Tk::Graph=HASH(0x84c9780
+): Tie::Watch::new(): -variable is required. at /usr/lib/perl5/site_p
+erl/5.8.0/Tk/Graph.pm line 1088
 at /usr/lib/perl5/site_perl/5.8.0/i386-linux-thread-multi/Tk/Derived.
+pm line 294
 Tk callback for .
 Tk::Derived::configure at /usr/lib/perl5/site_perl/5.8.0/i386-linux-t
+hread-multi/Tk/Derived.pm line 306
 main::reportg at x line 72
ERROR in Tk::Graph:Tk::Graph::set #947: Option '-type' is incorrect! (
+) at /usr/lib/perl5/site_perl/5.8.0/Tk/Graph.pm line 1802.
Can't set -variable to `undef' for Tk::Graph=HASH(0x84c9780): Tie::Wat
+ch::new(): -variable is required. at /usr/lib/perl5/site_perl/5.8.0/T
+k/Graph.pm line 1088
 at /usr/lib/perl5/site_perl/5.8.0/i386-linux-thread-multi/Tk/Derived.
+pm line 294
[download]

Thirdly, please put the following statements at or near the top of your code:

use strict;
use warnings;
[download]

They will alert you to variables which have not been declared, which is a GOOD THING!

For example, when I did that to your program, I needed to then define the following variables (under the section "Global data"):

#!/usr/bin/perl

# Use Strict!  Use Warnings!
#use strict;
#use warnings;

# Libraries
use Tk;
use Tk::Graph;

# Global data
#my $data;
#my $MW;
#my $total_user_job_count;
#my $usage;
#my $ca;
#my $typ;
#my $field;

# Main program
reportg();
[download]

Once you provide a working example which anyone else can run, to get the errors you describe, it will be much easier to identify your problem!

Unfortunatley, I can't really supply the entire program because you will need the data files to run it, which I'm not allowed to give out. Sorry about that, I know it makes it harder to find the problem. I have put in the statements use Tk; and use Tk::Graph at the start of the program and the program does display a graph with the correct values, so I know that part of the program works ok. It's just when I tried changing the type of the graph by using menus, it just doesn't change. It displays a pie chart to start off with, with the correct values, but when I chose a selection on the menu the graph doesn't change and I can see the error that i mentioned before in the console. I know the variable for the type of graph is correct because I'm printing it out when that function is called.

We don't WANT to see the propritary stuff! WHat we need you do is make us up a compact test case that shows what doesn't work, then we can run it and figure out how to help you.

In this case what you need to do is give us some dummy data built-into the code.

jdtoronto

So I can't run your program because I'm missing the data, but the first thing to try with things like this is calling $MW->update; after you change something. Sometimes that's all it takes.

Hays

Update: Why don't you try:

$ca->clear();
$setdisplay(whatever);
$ca->redraw();
$mw->update;
[download]

Also, the below posts are right, you should post your complete code, or at least a working version, so we can help you better.

I tried adding that in the function setdisplay, underneath $ca->configure(-type => $typ); but it didnt' work. The console behind my graph, prints the message

ERROR in Tk::Graph:Tk::Graph::set #947: Option '-type' is incorrect! (
+) at /home/others/it/itwi/perl/lib/perl5/site_perl/5.6.1/Tk/Graph.pm 
+line 1802, <IN> line 9318.
Bars
[download]

The last line which says'Bars' is the contents of the $typ variable being passed to the setdisplay function. Which whould be correct, I don't know why it is reporting -type is incorrect?

use Tk;
use Tk::Graph;

my $mw = MainWindow->new;

my $data = {
   Sleep   => 51,
   Work    => 135,
   Access  => 124,
   mySQL   => 5
};

my $ca = $mw->Graph(
   -title => 'Example',
   -type  => 'BARS'
)->pack(qw/-expand 1 -fill both/);
   
$ca->set($data);

$mw->Button(-text => 'Change', -command => sub {
   $ca->configure(-type => 'Circle');
   $ca->set($data);
})->pack;

MainLoop;
[download]

Update: The suggestion to use redraw was a good one -- that should work, but there is a bug within Graph.pm. The redraw method calls set with no value - the implication is that if no data is provided then the existing data should be used, however that is not what happens, and ultimately results in the display not getting redrawn. The workaround I provided will work. I'll report the bug and send the patch to the Author.

Within the set method, there is the following line:

If you change this to:

Then redraw will work as you'd expect, and would support the following version of your setdisplay sub.

sub setdisplay {
   $typ = shift;
   $ca->configure(-type => $typ);
   $ca->redraw;
}
[download]

I managed to get it fixed by putting

$ca->configure(-variable => $data);
[download]

in my subdisplay function, below the line

 $ca->configure(-type => $typ);
[download]

and it seems to work. Thanks for everybody's help.

Yep, under the hood that will also call the set method. The advantage of using variable is that if you update your data, then the changes should be reflected automatically within your Graph.

Rob

jdtoronto