Skip to content

TMCRootManager: implemented RNTuple writer#28

Open
karabowi wants to merge 3 commits into
vmc-project:masterfrom
karabowi:rntupleRootManager
Open

TMCRootManager: implemented RNTuple writer#28
karabowi wants to merge 3 commits into
vmc-project:masterfrom
karabowi:rntupleRootManager

Conversation

@karabowi

Copy link
Copy Markdown

Allows the TMCRootManager store data in TTree or RNTuple format.

Comment on lines +21 to +22
#include "TFile.h"
#include "TTree.h"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These includes should be removed from .cxx

Comment thread source/include/TMCRootManager.h Outdated
Comment on lines +102 to +103
TFile *fFile {0}; // Root output file
TTree *fTree {0}; // Root output tree

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use nullptr

Comment thread source/src/TMCRootManager.cxx Outdated
Comment on lines +150 to +153
TString fileName = Form("T%s",projectName);
if (fStorageMode == kRNTuple) {
fileName = Form("R%s",projectName);
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The file name should not be modified here; this should be removed.

Then in E03 example, we should better duplicate run_g4.C in run_g4a,b,c,d.C to avoid overwrite the output from each example.

@karabowi

Copy link
Copy Markdown
Author

I have applied the requested changes.
Thank you for pointing them out.

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 this pull request may close these issues.

2 participants