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

Seed recoording functionality #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aburke1605
Copy link

Added TNamed object to write the seed which was used to randomly sample the ntuple to the output files (rather than having to remember the seed).

Added TNamed object to write the seed which was used to randomly sample the ntuple to the output files (rather than having to remember the seed used to generate).
@cofitzpa
Copy link
Owner

Need to have a think about this one:

  • Typically I store the seed by piping the stdout to a text file. The current tool prints all its input arguments and output info for this reason. I consider the seed to be part of the metadata rather than the data itself.
  • If you sample the output of a previously sampled ntuple, this loses the prior seed and adds the new one. Is that intended?

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