Project

General

Profile

Bug #312

xboa tests fail

Added by Tunnell, Christopher almost 12 years ago. Updated almost 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Build System
Target version:
Start date:
02 February 2011
Due date:
% Done:

100%

Estimated time:
Workflow:

Description

currently the xboa unit tests fail for maus. see attached. these should pass before v1.0


Files

std (26 KB) std Tunnell, Christopher, 02 February 2011 13:35
err (14.1 KB) err Tunnell, Christopher, 02 February 2011 13:35
#1

Updated by Tunnell, Christopher almost 12 years ago

seems to happen when

Canvas.Update()

is called within Bunch.py

#2

Updated by Rogers, Chris almost 12 years ago

How did you do the build? When I try to import xboa tests, I get error
because numpy isn't found in third_party/build/python ...

#3

Updated by Tunnell, Christopher almost 12 years ago

So I build it with numpy-1.5.1. There's now a build script for both numpy and xboa in third_party.

#4

Updated by Rogers, Chris almost 12 years ago

Oops, noticed numpy now!

Okay, looks like there is a problem with display in the ROOT
installation. I do at python command line:

import ROOT
c = ROOT.TCanvas('hello')
hist = ROOT.TH2D
hist.Draw()

and get the same error. In fact, get the same error in ROOT/cint. Will
investigate further.

Chri

#5

Updated by Tunnell, Christopher almost 12 years ago

If you add a 'test runner' or just some script that outputs the results of the scripts, I can add it to Hudson. If it spits out XML reports, those can be displayed.

#6

Updated by Rogers, Chris almost 12 years ago

Looks like this could be many things:-
  • Bad QT install
  • Bad ROOTSYS etc

Probably to do with the way root build happens, but not sure what has
broken it. I notice env.csh doesn't set ROOTSYS for example.

#7

Updated by Tunnell, Christopher almost 12 years ago

Ah.... I'll set $ROOTSYS then (I just manually looked for libraries) and retest.

#8

Updated by Tunnell, Christopher almost 12 years ago

I had been trying to avoid as many environmental variables as possible to make syncing bash and C shell scripts easier and just not pollute stuff.

But I need to just set it (and also check if it exists rather than forcing people to grab ROOT). Ill add an issue but it's a low priority at the moment (unless somebody else wants to do it?)

#9

Updated by Rogers, Chris almost 12 years ago

I just tried setting rootsys (to third_party/install and third_party/build/root_5.28.00) and it didn't help...

#10

Updated by Tunnell, Christopher almost 12 years ago

I just commit a patch that I think will fix it. Waiting for the nightlies to let me know. Main thing? I don't do make install with ROOT and just leave it in its build directory, then source that.

#11

Updated by Tunnell, Christopher almost 12 years ago

it's a little nasty though because you then need to setup your environment twice. ie. after you install ROOT, you need to restart bash so it sees root. I don't like the fix, but I can plot stuff at least for the moment.

If that's the issue, I'll add a test to catch this type of ROOT failure in the future.

#12

Updated by Tunnell, Christopher almost 12 years ago

https://savannah.cern.ch/bugs/?78845

reported ROOT bug, though think I fixed it here

#13

Updated by Tunnell, Christopher almost 12 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

Applied in changeset r352.

Also available in: Atom PDF