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

[PWGJE] Add histograms for data-driven methods #8965

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

hahassan7
Copy link
Collaborator

Adding a THnSparse with jetpT score, jet mass, SV mass, and SVfE histograms wich will be used for data-driven approaches for efficiency and purity estimation.

@nzardosh
Copy link
Collaborator

Hi Hadi, why not use a table?

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for f9ced2e at 2024-12-12 17:08:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:17: error: 'TracksQA_002' is not a member of 'o2::aod'; did you mean 'TracksQA_001'?
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:29: error: template argument 1 is invalid
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:26:19: error: expression cannot be used as a function
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:60:19: error: expression cannot be used as a function
ninja: build stopped: subcommand failed.

Full log here.

@hahassan7
Copy link
Collaborator Author

Hi Nima, tables might be extremely large. I want to minimize the output file as much as possible.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for c570115 at 2024-12-12 17:21:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:17: error: 'TracksQA_002' is not a member of 'o2::aod'; did you mean 'TracksQA_001'?
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:29: error: template argument 1 is invalid
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:26:19: error: expression cannot be used as a function
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:60:19: error: expression cannot be used as a function
ninja: build stopped: subcommand failed.

Full log here.

@nzardosh
Copy link
Collaborator

have you checked how large they would be? it seems you already have most of this information in trees anyways?

@hahassan7
Copy link
Collaborator Author

The Trees are in the TreeCreator, there are no trees here. I also think THnSparse since it has binnings it will be much smaller than TTrees.

@nzardosh
Copy link
Collaborator

ok fair enough

@nzardosh
Copy link
Collaborator

At some point we should arrange a discussion on zoom about this HFtagging code and how to organise it. In general to me it seems the better way to organise it would be something like in HF where an ML selector task is ran which produces a tree with one score coloumn that can be joined to the jet table. Then this can be fed to the other tasks as a selection

@hahassan7
Copy link
Collaborator Author

Yes, I totally agree with this. It will make tagging much easier with just taking the score from the extended jet table. I think we edit the one from Hanseo and add one score colunm to it and a function that evaluates the model and gives the score it gives. Though Hanseo's code needs so much cleaning.

@nzardosh
Copy link
Collaborator

Ok that sounds great then, lets go in that direction :) Yeah I agree the code needs alot of cleaning. I think this would be a good thing to spend a week or so on since the code needs to be used in many workflows. I had discussed this at some point also with Hanseo that for Hard probes I approved alot of code changes that were urgent but with the understanding that they would be tidied up after so its a good moment perhaps to do this.

@hahassan7
Copy link
Collaborator Author

I will try to do this maybe during the Christmas break, or a little bit after it.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for c35118b at 2024-12-12 17:59:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:17: error: 'TracksQA_002' is not a member of 'o2::aod'; did you mean 'TracksQA_001'?
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:21:29: error: template argument 1 is invalid
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:26:19: error: expression cannot be used as a function
/sw/SOURCES/O2Physics/8965-slc9_x86-64/0/Common/TableProducer/Converters/trackQA002Converter.cxx:60:19: error: expression cannot be used as a function
ninja: build stopped: subcommand failed.

Full log here.

@vkucera vkucera changed the title [PWGJE] Addinfg histograms for data-driven methods [PWGJE] Add histograms for data-driven methods Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants