Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LazyTree construction errors : UndefVarError: TLeafS / UndefVarError: TBasket not defined #160

Open
Momox357 opened this issue Apr 7, 2022 · 4 comments · Fixed by #161

Comments

@Momox357
Copy link

Momox357 commented Apr 7, 2022

Hello, first of all thanks for this amazing package and the fast reaction times (I've started looking into it on monday and experienced the compling error as my first obstacle)!

I've managed to read in the sample file in the quick start sections and compare that data / a generated histogram with one generated from ROOT / with the TBrowser.

Now I'm trying to do that with .root files from ATLAS OpenData (since they have the notebooks for comparison), but for the files I've tried it reads the files but doesn't display it in a "tree-like fashion" though I'm not sure if that limited / just not done for files too large for example.
My main issue is that I'm getting an error when trying to contruct a LazyTree.

For my first case (as far as I understand it "grabs" the whole tree, while ftree=LazyTree(f, "mini", ["lep_n"]) is the specific branch)

Using UnROOT

f=ROOTFile("http://opendata.atlas.cern/release/samples/MC/mc_105987.WZ.root")
ftree=LazyTree(f, "mini")

I get "UndefVarError: TLeafS not defined" .

For two others (https://atlas-opendata.web.cern.ch/atlas-opendata/samples/2020/GamGam/Data/data_A.GamGam.root and https://atlas-opendata.web.cern.ch/atlas-opendata/samples/2020/1largeRjet1lep/MC/mc_361106.Zee.1largeRjet1lep.root )
I get "UndefVarError: TBasket not defined" in that case.

I've found this issue while looking through the github.
That said I'm not sure if it's a potential bug, a file issue or an user issue (since I'm new to Root and Julia :) ).

@Moelf
Copy link
Member

Moelf commented Apr 7, 2022

thanks for the report, I think it's the same problem with:

@Moelf Moelf mentioned this issue Apr 7, 2022
@Moelf
Copy link
Member

Moelf commented Apr 7, 2022

can you try the PR, that fixes your first issue.

@Moelf
Copy link
Member

Moelf commented Apr 7, 2022

https://opendata.cern.ch/record/12341

this is an opendata in a widely used format (nanoAOD), we have extensive tests with nanoAOD format so everything should just work

@Moelf Moelf closed this as completed in #161 Apr 7, 2022
@Moelf Moelf reopened this Apr 7, 2022
@Momox357
Copy link
Author

Momox357 commented Apr 7, 2022

can you try the PR, that fixes your first issue.

Yes it works, thanks! That was really fast.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants