-
Notifications
You must be signed in to change notification settings - Fork 12
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
Document external/ directory #55
Comments
Also, we should document where the procedure we follow for scale factors comes from. |
i prefer adding to wiki (https://github.com/CPLUOS/nano/wiki) |
I guess this issue can be closed? unless there're more things you want it to be added. |
Well, once its documented where the files came from. We have a wiki for data but not yet externals or? |
is this finished? can we close? |
No one ever updated the wiki with externals information @YeChanKang @DanielLee1004 |
First strike on the documentation: See [1]. Only lumiTools and pileupTool has been documented. The remainings will be filled. [1] https://github.com/CPLUOS/nano/wiki/Usage-of-external-codes#how-to-set-the-numbers-in-this-module |
Can we add a README or wiki page that documents where the code we've taken comes from, and if/what we changed (mostly I think this was just splitting the header and cxx)
The text was updated successfully, but these errors were encountered: