-
Notifications
You must be signed in to change notification settings - Fork 614
[PWGHF] runMassFitter: add main() function and missing header, provide Readme #11611
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
Conversation
|
O2 linter results: ❌ 1 errors, |
|
Hi @lubynets , what exactly are you trying to do? |
Hi @vkucera, |
|
@lubynets OK. I think the use of the fitter should be documented somewhere. Do we not have an existing document for this? |
Hi @vkucera, |
That would be very useful indeed. If the |
[MegaLinter] Apply linters automatic fixes to AliceO2Group#11611
vkucera
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @lubynets , looks good to me.
Just please fix the minor typo.
Done |
…e Readme (AliceO2Group#11611) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…e Readme (AliceO2Group#11611) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…e Readme (AliceO2Group#11611) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…e Readme (AliceO2Group#11611) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…e Readme (AliceO2Group#11611) Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
main()function is added as an entry point to enable compilation as CMake project.[Update]
3. Added
CMakeLists_HFInvMassFitter.txtandHFInvMassFitterLinkDef.hfor fitter compilation as a standalone project.4. Added Readme with instructions how to compile and run the fitter.