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

Change Organization of AnaProcessors #26

Open
cbravo135 opened this issue Nov 13, 2019 · 2 comments
Open

Change Organization of AnaProcessors #26

cbravo135 opened this issue Nov 13, 2019 · 2 comments
Assignees

Comments

@cbravo135
Copy link
Collaborator

We would like to use the python configuration to map collections into multiple instantiations of the same histogram class. The python configuration file should name a json file which will configure which histograms it will fill. Multiple instances of this set of histograms can be used for different sets of input collections, which should be set in the python configuration script.

@mholtrop
Copy link
Collaborator

I think we should discuss this, and how you plan to implement it.

@pbutti
Copy link
Collaborator

pbutti commented Nov 14, 2019

Hi Maurik,
Thank you for opening up the discussion. We were already worked on this issue (we actually started in the morning but then got side-tracked in the afternoon on other topics)
There is already a commit in iss26 which implemented the C++ for reading json files. This will improve the turn-around of plot production as basic plot definitions will be set at run-time instead at compilation time. Machinery tested and working. Planning to extend a bit in the next days.

Perhaps the title of the issue is too long. What it boils down to is to:

  • Add configuration to the processors from config.py
  • Add configuration files for making TH1/TH2.

Quite basic and nothing really fancy as might sound at first! =)

Thanks!
Cheers,
PF

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

No branches or pull requests

3 participants