search

elaird / supy

analyze events stored in TTrees in parallel
8 stars 7 forks source link

Clearing issue #90

Closed betchart closed 12 years ago

betchart commented 12 years ago

How about this?

elaird commented 12 years ago

Hi Burt,

This looks fine to me. An alternative might be to do self.canvas.Clear("D") [1]. I sign off on either method (though haven't tested them).

Ted

[1] http://root.cern.ch/root/html/TCanvas.html#TCanvas:Clear

elaird commented 12 years ago

Hi Burt,

Actually, how about a clearCanvas(self) which is called by uponAcceptance? Then it is completely general.

Ted

betchart commented 12 years ago

I like the latter proposal. An alternative name for the function could be reset(), which doesn't implicate canvas at all (which I like, since my displayer offshoot doesn't actually clear the canvas).

elaird commented 12 years ago

Sounds good to me.

betchart commented 12 years ago

See if that works for you, and if it's good, pull it into the display branch. Before we pull display into master, I'd like to see if there are any other tweaks I need in order to inherit it in the displays_ttbar class.

elaird commented 12 years ago

Thanks. I will merge this into display now, and then test display and comment on that pull request.